test(browser-integration): Fix incorrect tag value assertions (#18162)

Fixes faulty test assertions where we asserted for certain properties to
_not_ be in an object but used `toMatchObject` to do so.

Author: Lms24

dev-packages/browser-integration-tests/suites/public-api/setTag/with_non_primitives/test.ts

@@ -3,11 +3,19 @@ import type { Event } from '@sentry/core';
 import { sentryTest } from '../../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest } from '../../../../utils/helpers';
 
-sentryTest('should not accept non-primitive tags', async ({ getLocalTestUrl, page }) => {
+sentryTest('[bug] accepts non-primitive tags', async ({ getLocalTestUrl, page }) => {
+  // this is a bug that went unnoticed due to type definitions and a bad assertion
+  // TODO: We should not accept non-primitive tags. Fix this as a follow-up.
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
 
   expect(eventData.message).toBe('non_primitives');
-  expect(eventData.tags).toMatchObject({});
+
+  // TODO: This should be an empty object but instead, it is:
+  expect(eventData.tags).toEqual({
+    tag_1: {},
+    tag_2: [],
+    tag_3: ['a', {}],
+  });
 });

dev-packages/browser-integration-tests/suites/public-api/setTag/with_primitives/test.ts

@@ -9,7 +9,7 @@ sentryTest('should set primitive tags', async ({ getLocalTestUrl, page }) => {
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
 
   expect(eventData.message).toBe('primitive_tags');
-  expect(eventData.tags).toMatchObject({
+  expect(eventData.tags).toEqual({
     tag_1: 'foo',
     tag_2: 3.141592653589793,
     tag_3: false,