test(browser): Add integration tests for withScope (#4342)

onurtemizkan · web-flow · commit 74d27555c92c · 2021-12-22T23:02:20.000Z
diff --git a/packages/integration-tests/suites/public-api/withScope/init.js b/packages/integration-tests/suites/public-api/withScope/init.js
@@ -0,0 +1,7 @@
+import * as Sentry from '@sentry/browser';
+
+window.Sentry = Sentry;
+
+Sentry.init({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+});
diff --git a/packages/integration-tests/suites/public-api/withScope/nested_scopes/subject.js b/packages/integration-tests/suites/public-api/withScope/nested_scopes/subject.js
@@ -0,0 +1,18 @@
+Sentry.setUser({ id: 'qux' });
+Sentry.captureMessage('root_before');
+
+Sentry.withScope(scope => {
+  scope.setTag('foo', false);
+  Sentry.captureMessage('outer_before');
+
+  Sentry.withScope(scope => {
+    scope.setTag('bar', 10);
+    scope.setUser(null);
+    Sentry.captureMessage('inner');
+  });
+
+  scope.setUser({ id: 'baz' });
+  Sentry.captureMessage('outer_after');
+});
+
+Sentry.captureMessage('root_after');
diff --git a/packages/integration-tests/suites/public-api/withScope/nested_scopes/test.ts b/packages/integration-tests/suites/public-api/withScope/nested_scopes/test.ts
@@ -0,0 +1,30 @@
+import { expect } from '@playwright/test';
+
+import { sentryTest } from '../../../../utils/fixtures';
+import { getMultipleSentryRequests } from '../../../../utils/helpers';
+
+sentryTest('should allow nested scoping', async ({ getLocalTestPath, page }) => {
+  const url = await getLocalTestPath({ testDir: __dirname });
+
+  const eventData = await getMultipleSentryRequests(page, 5, url);
+
+  expect(eventData[0].message).toBe('root_before');
+  expect(eventData[0].user).toMatchObject({ id: 'qux' });
+  expect(eventData[0].tags).toBeUndefined();
+
+  expect(eventData[1].message).toBe('outer_before');
+  expect(eventData[1].user).toMatchObject({ id: 'qux' });
+  expect(eventData[1].tags).toMatchObject({ foo: false });
+
+  expect(eventData[2].message).toBe('inner');
+  expect(eventData[2].user).toBeUndefined();
+  expect(eventData[2].tags).toMatchObject({ foo: false, bar: 10 });
+
+  expect(eventData[3].message).toBe('outer_after');
+  expect(eventData[3].user).toMatchObject({ id: 'baz' });
+  expect(eventData[3].tags).toMatchObject({ foo: false });
+
+  expect(eventData[4].message).toBe('root_after');
+  expect(eventData[4].user).toMatchObject({ id: 'qux' });
+  expect(eventData[4].tags).toBeUndefined();
+});
diff --git a/packages/integration-tests/suites/public-api/withScope/template.hbs b/packages/integration-tests/suites/public-api/withScope/template.hbs
@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title></title>
+  <script src="{{htmlWebpackPlugin.options.initialization}}"></script>
+  </head>
+  <body>
+    <script src="{{htmlWebpackPlugin.options.subject}}"></script>
+  </body>
+</html>

-Original file line number
+Diff line change
@@ @@ -0,0 +1,7 @@ @@
 +import * as Sentry from '@sentry/browser';
++
 +window.Sentry = Sentry;
++
 +Sentry.init({
 +  dsn: 'https://[email protected]/1337',
 +});