feat: core/browser additions (#1175)

* fix: Disable @sentry/browser tests because of flaky travis

* feat: Add setRelease, Make everything a promise

Up test coverage

* fix: tests for fetch api

* ci: Reenable travis tests

* ci: Fix core dep

* test(core): Remove unnecessary asserts

* meta: Bump core/browser

Author: HazAT

.travis.yml

@@ -23,17 +23,16 @@ matrix:
         sauce_connect: true
       script:
         - npm run test && if [ "$TRAVIS_SECURE_ENV_VARS" == "true" ]; then npm run test:ci; else exit 0; fi
+    - env: NAME="@sentry/core"
+      install: true
+      script:
+      - cd packages/core && yarn install && yarn test
     - env: NAME="@sentry/browser"
       install: true
       script:
       - yarn link
       - cd packages/core && yarn link && yarn install && yarn dist
       - cd ../browser
-      - yarn remove @sentry/core # this has to be removed once we released the package
       - yarn link "@sentry/core"
       - yarn link "raven-js"
       - yarn install && yarn test
-    - env: NAME="@sentry/core"
-      install: true
-      script:
-      - cd packages/core && yarn install && yarn test

packages/browser/__tests__/index.ts

@@ -34,7 +34,7 @@ describe('Browser Interface', () => {
 
     page.on('request', async request => {
       // @ts-ignore
-      if (request.resourceType === 'xhr') {
+      if (request.resourceType === 'other' || request.resourceType === 'xhr') {
         const data = JSON.parse(request.postData as any);
         if (data.exception) {
           expect(data.exception).not.toBeUndefined();

packages/browser/lib/SentryBrowser.ts

@@ -23,7 +23,7 @@ export class SentryBrowser implements IAdapter {
     return Raven;
   }
 
-  public setOptions(options: ISentryBrowserOptions) {
+  public async setOptions(options: ISentryBrowserOptions) {
     Object.assign(this.options, options);
     Object.assign(Raven._globalOptions, this.options);
     return this;
@@ -74,23 +74,28 @@ export class SentryBrowser implements IAdapter {
     });
   }
 
-  public setUserContext(user?: IUser) {
+  public async setUserContext(user?: IUser) {
     Raven.setUserContext(user);
     return this;
   }
 
-  public setTagsContext(tags?: { [key: string]: any }) {
+  public async setTagsContext(tags?: { [key: string]: any }) {
     Raven.setTagsContext(tags);
     return this;
   }
 
-  public setExtraContext(extra?: { [key: string]: any }) {
+  public async setExtraContext(extra?: { [key: string]: any }) {
     Raven.setExtraContext(extra);
     return this;
   }
 
-  public clearContext() {
+  public async clearContext() {
     Raven.clearContext();
     return this;
   }
+
+  public async setRelease(release: string) {
+    Raven.setRelease(release);
+    return this;
+  }
 }

packages/browser/package.json

@@ -1,35 +1,35 @@
 {
   "name": "@sentry/browser",
-  "version": "0.0.1",
+  "version": "0.2.1",
   "description": "Offical Sentry SDK for browsers",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
   "typescript": {
     "definition": "dist/index.d.ts"
   },
   "dependencies": {
-    "@sentry/core": "^0.1.0",
-    "raven-js": "^3.18.1"
+    "@sentry/core": "^0.2.1",
+    "raven-js": "^3.21.0"
   },
   "devDependencies": {
-    "@types/jest": "^21.1.6",
-    "@types/node": "^8.0.51",
-    "@types/puppeteer": "^0.12.4",
+    "@types/jest": "^21.1.8",
+    "@types/node": "^8.5.1",
+    "@types/puppeteer": "^0.13.8",
     "jest": "^21.2.1",
     "npm-run-all": "^4.1.2",
-    "prettier": "^1.8.2",
+    "prettier": "^1.9.2",
     "puppeteer": "^0.13.0",
     "rollup": "^0.52.1",
     "rollup-plugin-commonjs": "^8.2.6",
     "rollup-plugin-node-resolve": "^3.0.0",
     "rollup-plugin-uglify": "^2.0.1",
-    "ts-jest": "^21.2.2",
-    "ts-loader": "^3.1.1",
-    "ts-node": "^3.3.0",
+    "ts-jest": "^21.2.4",
+    "ts-loader": "^3.2.0",
+    "ts-node": "^4.0.2",
     "tslint": "^5.8.0",
     "tslint-config-prettier": "^1.6.0",
     "tslint-eslint-rules": "^4.1.1",
-    "typescript": "^2.6.1",
+    "typescript": "^2.6.2",
     "vrsource-tslint-rules": "^5.8.0"
   },
   "engines": {