browser: Ported all integration tests from raven-js

Author: kamilogorek

packages/browser/rollup.config.js

@@ -8,6 +8,32 @@ const commitHash = require('child_process')
   .execSync('git rev-parse --short HEAD', { encoding: 'utf-8' })
   .trim();
 
+const bundleConfig = {
+  input: 'src/index.ts',
+  output: {
+    format: 'iife',
+    name: 'Sentry',
+    sourcemap: true,
+  },
+  context: 'window',
+  plugins: [
+    typescript({
+      tsconfig: 'tsconfig.build.json',
+      tsconfigOverride: { compilerOptions: { declaration: false } },
+    }),
+    resolve({
+      jsnext: true,
+      main: true,
+      browser: true,
+    }),
+    commonjs(),
+    license({
+      sourcemap: true,
+      banner: `/*! @sentry/browser <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/raven-js */`,
+    }),
+  ],
+};
+
 export default [
   {
     input: 'src/index.ts',
@@ -31,56 +57,23 @@ export default [
     ],
   },
   {
-    input: 'src/index.ts',
+    ...bundleConfig,
     output: {
+      ...bundleConfig.output,
       file: 'build/bundle.js',
-      format: 'iife',
-      name: 'Sentry',
-      sourcemap: true,
     },
-    context: 'window',
-    plugins: [
-      typescript({
-        tsconfig: 'tsconfig.build.json',
-        tsconfigOverride: { compilerOptions: { declaration: false } },
-      }),
-      resolve({
-        jsnext: true,
-        main: true,
-        browser: true,
-      }),
-      commonjs(),
-      license({
-        sourcemap: true,
-        banner: `/*! @sentry/browser <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/raven-js */`,
-      }),
-    ],
   },
   {
-    input: 'src/index.ts',
+    ...bundleConfig,
     output: {
+      ...bundleConfig.output,
       file: 'build/bundle.min.js',
-      format: 'iife',
-      name: 'Sentry',
-      sourcemap: true,
     },
-    context: 'window',
     plugins: [
-      typescript({
-        tsconfig: 'tsconfig.build.json',
-        tsconfigOverride: { compilerOptions: { declaration: false } },
-      }),
-      resolve({
-        jsnext: true,
-        main: true,
-        browser: true,
-      }),
-      commonjs(),
+      ...bundleConfig.plugins.slice(0, -1),
+       Uglify has to be at the end of compilation, BUT before the license banner
       uglify(),
-      license({
-        sourcemap: true,
-        banner: `/*! @sentry/browser <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/raven-js */`,
-      }),
+      ...bundleConfig.plugins.slice(-1),
     ],
   },
 ];

packages/browser/test/integration/frame.html

@@ -87,9 +87,6 @@
   </script>
   <script src="../../build/bundle.js"></script>
   <script>
-    // stub _makeRequest so we don't actually transmit any data
-    Raven._makeRequest = function () { };
-
     // store references to original, unwrapped built-ins in order to:
     // - get a clean, unwrapped setTimeout (so stack traces don't include
     //   frames from mocha)
@@ -108,12 +105,34 @@
       consoleLog: console.log
     };
 
-    window.ravenData = [];
-    Raven.config('https://[email protected]/1', {
-      dataCallback: function (data) {
-        ravenData.push(data);
+    // expose events so we can access them in our tests
+    window.sentryData = [];
+
+    // stub transport so we don't actually transmit any data
+    function DummyTransport() { }
+    DummyTransport.prototype.send = function (event) {
+      // console.log(JSON.stringify(event, null, 2));
+      sentryData.push(event);
+      return true;
+    }
+
+    Sentry.init({
+      dsn: 'https://[email protected]/1',
+      transport: DummyTransport,
+      shouldAddBreadcrumb: function (breadcrumb) {
+        // Filter internal Karma requests
+        if (
+          breadcrumb.type === 'http' &&
+          (
+            breadcrumb.data.url.indexOf('test.js') !== -1 ||
+            breadcrumb.data.url.indexOf('frame.html') !== -1
+          )
+        ) {
+          return false;
+        }
+        return true;
       }
-    }).install();
+    })
 
     function bar() {
       baz();