fix: Breadcrumb tests

Author: HazAT

packages/browser/src/integrations/breadcrumbs.ts

@@ -179,6 +179,11 @@ export class Breadcrumbs implements Integration {
       return;
     }
 
+    if (handlerData.fetchData.url.match(/sentry_key/) && handlerData.fetchData.method === 'POST') {
+      // We will not create breadcrumbs for fetch requests that contain `sentry_key` (internal sentry requests)
+      return;
+    }
+
     if (handlerData.error) {
       getCurrentHub().addBreadcrumb(
         {

packages/browser/test/integration/suites/breadcrumbs.js

@@ -84,115 +84,6 @@ describe("breadcrumbs", function() {
     }
   );
 
-  it(
-    optional(
-      "should transform XMLHttpRequests with events to the Sentry store endpoint as sentry.event type breadcrumb",
-      IS_LOADER
-    ),
-    function() {
-      return runInSandbox(sandbox, { manual: true }, function() {
-        var store =
-          document.location.protocol +
-          "//" +
-          document.location.hostname +
-          (document.location.port ? ":" + document.location.port : "") +
-          "/api/1/store/" +
-          "?sentry_key=1337";
-
-        var xhr = new XMLHttpRequest();
-        xhr.open("POST", store);
-        xhr.send('{"message":"someMessage","level":"warning"}');
-        waitForXHR(xhr, function() {
-          Sentry.captureMessage("test");
-          window.finalizeManualTest();
-        });
-      }).then(function(summary) {
-        // The async loader doesn't wrap XHR
-        if (IS_LOADER) {
-          return;
-        }
-        assert.equal(summary.breadcrumbs.length, 1);
-        assert.equal(summary.breadcrumbs[0].category, "sentry.event");
-        assert.equal(summary.breadcrumbs[0].level, "warning");
-        assert.equal(summary.breadcrumbs[0].message, "someMessage");
-      });
-    }
-  );
-
-  it(
-    optional(
-      "should transform XMLHttpRequests with transactions type to the Sentry store endpoint as sentry.transaction type breadcrumb",
-      IS_LOADER
-    ),
-    function() {
-      return runInSandbox(sandbox, { manual: true }, function() {
-        var store =
-          document.location.protocol +
-          "//" +
-          document.location.hostname +
-          (document.location.port ? ":" + document.location.port : "") +
-          "/api/1/store/" +
-          "?sentry_key=1337";
-
-        var xhr = new XMLHttpRequest();
-        xhr.open("POST", store);
-        xhr.send(
-          '{"message":"someMessage","transaction":"wat","level":"warning", "type": "transaction"}'
-        );
-        waitForXHR(xhr, function() {
-          Sentry.captureMessage("test");
-          window.finalizeManualTest();
-        });
-      }).then(function(summary) {
-        // The async loader doesn't wrap XHR
-        if (IS_LOADER) {
-          return;
-        }
-        assert.equal(summary.breadcrumbs.length, 1);
-        assert.equal(summary.breadcrumbs[0].category, "sentry.transaction");
-        assert.equal(summary.breadcrumbs[0].level, "warning");
-        assert.equal(summary.breadcrumbs[0].message, "someMessage");
-      });
-    }
-  );
-
-  it(
-    optional(
-      "should not transform XMLHttpRequests with transactions attribute to the Sentry store endpoint as sentry.transaction type breadcrumb",
-      IS_LOADER
-    ),
-    function() {
-      return runInSandbox(sandbox, { manual: true }, function() {
-        var store =
-          document.location.protocol +
-          "//" +
-          document.location.hostname +
-          (document.location.port ? ":" + document.location.port : "") +
-          "/api/1/store/" +
-          "?sentry_key=1337";
-
-        var xhr = new XMLHttpRequest();
-        xhr.open("POST", store);
-        xhr.send(
-          '{"message":"someMessage","transaction":"wat","level":"warning"}'
-        );
-        waitForXHR(xhr, function() {
-          Sentry.captureMessage("test");
-          window.finalizeManualTest();
-        });
-      }).then(function(summary) {
-        // The async loader doesn't wrap XHR
-        if (IS_LOADER) {
-          return;
-        }
-        assert.equal(summary.breadcrumbs.length, 1);
-        assert.equal(summary.breadcrumbs[0].category, "sentry.event");
-        assert.equal(summary.breadcrumbs[0].level, "warning");
-        assert.equal(summary.breadcrumbs[0].message, "someMessage");
-      });
-    }
-  );
-
   it("should record a fetch request", function() {
     return runInSandbox(sandbox, { manual: true }, function() {
       fetch("/base/subjects/example.json", {