test(browser): Add integration tests for setExtras (#4347)

onurtemizkan · web-flow · commit 15773ba0ab84 · 2021-12-21T11:50:23.000-05:00
diff --git a/packages/integration-tests/suites/public-api/setExtras/consecutive_calls/subject.js b/packages/integration-tests/suites/public-api/setExtras/consecutive_calls/subject.js
@@ -0,0 +1,11 @@
+Sentry.setExtras({ extra: [] });
+Sentry.setExtras({ null: 0 });
+Sentry.setExtras({
+  obj: {
+    foo: ['bar', 'baz', 1],
+  },
+});
+Sentry.setExtras({ [null]: Infinity });
+Sentry.setExtras({ [Infinity]: 2 });
+
+Sentry.captureMessage('consecutive_calls');
diff --git a/packages/integration-tests/suites/public-api/setExtras/consecutive_calls/test.ts b/packages/integration-tests/suites/public-api/setExtras/consecutive_calls/test.ts
@@ -0,0 +1,13 @@
+import { expect } from '@playwright/test';
+
+import { sentryTest } from '../../../../utils/fixtures';
+import { getSentryRequest } from '../../../../utils/helpers';
+
+sentryTest('should set extras from multiple consecutive calls', async ({ getLocalTestPath, page }) => {
+  const url = await getLocalTestPath({ testDir: __dirname });
+
+  const eventData = await getSentryRequest(page, url);
+
+  expect(eventData.message).toBe('consecutive_calls');
+  expect(eventData.extra).toMatchObject({ extra: [], Infinity: 2, null: null, obj: { foo: ['bar', 'baz', 1] } });
+});
diff --git a/packages/integration-tests/suites/public-api/setExtras/init.js b/packages/integration-tests/suites/public-api/setExtras/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/setExtras/multiple_extras/subject.js b/packages/integration-tests/suites/public-api/setExtras/multiple_extras/subject.js
@@ -0,0 +1,12 @@
+Sentry.setExtras({
+  extra_1: [1, ['foo'], 'bar'],
+  extra_2: 'baz',
+  extra_3: Math.PI,
+  extra_4: {
+    qux: {
+      quux: false,
+    },
+  },
+});
+
+Sentry.captureMessage('multiple_extras');
diff --git a/packages/integration-tests/suites/public-api/setExtras/multiple_extras/test.ts b/packages/integration-tests/suites/public-api/setExtras/multiple_extras/test.ts
@@ -0,0 +1,18 @@
+import { expect } from '@playwright/test';
+
+import { sentryTest } from '../../../../utils/fixtures';
+import { getSentryRequest } from '../../../../utils/helpers';
+
+sentryTest('should record an extras object', async ({ getLocalTestPath, page }) => {
+  const url = await getLocalTestPath({ testDir: __dirname });
+
+  const eventData = await getSentryRequest(page, url);
+
+  expect(eventData.message).toBe('multiple_extras');
+  expect(eventData.extra).toMatchObject({
+    extra_1: [1, ['foo'], 'bar'],
+    extra_2: 'baz',
+    extra_3: 3.141592653589793,
+    extra_4: { qux: { quux: false } },
+  });
+});
diff --git a/packages/integration-tests/suites/public-api/setExtras/template.hbs b/packages/integration-tests/suites/public-api/setExtras/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',
 +});