add browser integration test

Author: Lms24

dev-packages/browser-integration-tests/suites/public-api/logger/scopeAttributes/subject.js

@@ -0,0 +1,34 @@
+// only log attribute
+Sentry.logger.info('log_before_any_scope', { log_attr: 'scope_attr_1' });
+
+Sentry.getGlobalScope().setAttribute('global_scope_attr', true);
+
+// global scope, log attribute
+Sentry.logger.info('log_after_global_scope', { log_attr: 'scope_attr_2' });
+
+let isolScope = null;
+let isolScope2 = null;
+
+Sentry.withIsolationScope(isolationScope => {
+  isolScope = isolationScope;
+  isolationScope.setAttribute('isolation_scope_1_attr', { value: 100, unit: 'ms' });
+
+  // global scope, isolation scope, log attribute
+  Sentry.logger.info('log_with_isolation_scope', { log_attr: 'scope_attr_3' });
+
+  Sentry.withScope(scope => {
+    scope.setAttribute('scope_attr', { value: 200, unit: 'ms' });
+
+    // global scope, isolation scope, current scope attribute, log attribute
+    Sentry.logger.info('log_with_scope', { log_attr: 'scope_attr_4' });
+  });
+
+  Sentry.withScope(scope2 => {
+    scope2.setAttribute('scope_2_attr', { value: 300, unit: 'ms' });
+
+    // global scope, isolation scope, current scope attribute, log attribute
+    Sentry.logger.info('log_with_scope_2', { log_attr: 'scope_attr_5' });
+  });
+});
+
+Sentry.flush();

dev-packages/browser-integration-tests/suites/public-api/logger/scopeAttributes/test.ts

@@ -0,0 +1,98 @@
+import { expect } from '@playwright/test';
+import type { LogEnvelope } from '@sentry/core';
+import { sentryTest } from '../../../../utils/fixtures';
+import { getFirstSentryEnvelopeRequest, properFullEnvelopeRequestParser } from '../../../../utils/helpers';
+
+sentryTest('captures logs with scope attributes', async ({ getLocalTestUrl, page }) => {
+  const bundle = process.env.PW_BUNDLE || '';
+  // Only run this for npm package exports
+  if (bundle.startsWith('bundle') || bundle.startsWith('loader')) {
+    sentryTest.skip();
+  }
+
+  const url = await getLocalTestUrl({ testDir: __dirname });
+
+  const event = await getFirstSentryEnvelopeRequest<LogEnvelope>(page, url, properFullEnvelopeRequestParser);
+  const envelopeItems = event[1];
+
+  expect(envelopeItems[0]).toEqual([
+    {
+      type: 'log',
+      item_count: 5,
+      content_type: 'application/vnd.sentry.items.log+json',
+    },
+    {
+      items: [
+        {
+          timestamp: expect.any(Number),
+          level: 'info',
+          body: 'log_before_any_scope',
+          severity_number: 9,
+          trace_id: expect.any(String),
+          attributes: {
+            'sentry.sdk.name': { value: 'sentry.javascript.browser', type: 'string' },
+            'sentry.sdk.version': { value: expect.any(String), type: 'string' },
+            log_attr: { value: 'scope_attr_1', type: 'string' },
+          },
+        },
+        {
+          timestamp: expect.any(Number),
+          level: 'info',
+          body: 'log_after_global_scope',
+          severity_number: 9,
+          trace_id: expect.any(String),
+          attributes: {
+            'sentry.sdk.name': { value: 'sentry.javascript.browser', type: 'string' },
+            'sentry.sdk.version': { value: expect.any(String), type: 'string' },
+            global_scope_attr: { value: true, type: 'boolean' },
+            log_attr: { value: 'scope_attr_2', type: 'string' },
+          },
+        },
+        {
+          timestamp: expect.any(Number),
+          level: 'info',
+          body: 'log_with_isolation_scope',
+          severity_number: 9,
+          trace_id: expect.any(String),
+          attributes: {
+            'sentry.sdk.name': { value: 'sentry.javascript.browser', type: 'string' },
+            'sentry.sdk.version': { value: expect.any(String), type: 'string' },
+            global_scope_attr: { value: true, type: 'boolean' },
+            isolation_scope_1_attr: { value: 100, unit: 'ms', type: 'integer' },
+            log_attr: { value: 'scope_attr_3', type: 'string' },
+          },
+        },
+        {
+          timestamp: expect.any(Number),
+          level: 'info',
+          body: 'log_with_scope',
+          severity_number: 9,
+          trace_id: expect.any(String),
+          attributes: {
+            'sentry.sdk.name': { value: 'sentry.javascript.browser', type: 'string' },
+            'sentry.sdk.version': { value: expect.any(String), type: 'string' },
+            global_scope_attr: { value: true, type: 'boolean' },
+            isolation_scope_1_attr: { value: 100, unit: 'ms', type: 'integer' },
+            scope_attr: { value: 200, unit: 'ms', type: 'integer' },
+            log_attr: { value: 'scope_attr_4', type: 'string' },
+          },
+        },
+        {
+          timestamp: expect.any(Number),
+          level: 'info',
+          body: 'log_with_scope_2',
+          severity_number: 9,
+          trace_id: expect.any(String),
+          attributes: {
+            'sentry.sdk.name': { value: 'sentry.javascript.browser', type: 'string' },
+            'sentry.sdk.version': { value: expect.any(String), type: 'string' },
+            global_scope_attr: { value: true, type: 'boolean' },
+            isolation_scope_1_attr: { value: 100, unit: 'ms', type: 'integer' },
+            scope_2_attr: { value: 300, unit: 'ms', type: 'integer' },
+            log_attr: { value: 'scope_attr_5', type: 'string' },
+          },
+        },
+      ],
+    },
+  ]);
+});