remove compensation in tests

Author: lobsterkatie

packages/core/test/lib/base.test.ts

@@ -884,29 +884,9 @@ describe('BaseClient', () => {
       const normalizedTransaction = JSON.parse(JSON.stringify(transaction)); // deep-copy
 
       client.captureEvent(transaction);
-
-      // TODO: This is to compensate for a temporary debugging hack which adds data the tests aren't anticipating to the
-      // event. The code can be restored to its original form (the commented-out line below) once that hack is
-      // removed. See https://github.com/getsentry/sentry-javascript/pull/4425 and
-      // https://github.com/getsentry/sentry-javascript/pull/4574
       const capturedEvent = TestClient.instance!.event!;
-      if (capturedEvent.sdkProcessingMetadata?.normalizeDepth) {
-        if (Object.keys(capturedEvent.sdkProcessingMetadata).length === 1) {
-          delete capturedEvent.sdkProcessingMetadata;
-        } else {
-          delete capturedEvent.sdkProcessingMetadata.normalizeDepth;
-        }
-      }
-      if (capturedEvent.sdkProcessingMetadata?.baseClientNormalized) {
-        if (Object.keys(capturedEvent.sdkProcessingMetadata).length === 1) {
-          delete capturedEvent.sdkProcessingMetadata;
-        } else {
-          delete capturedEvent.sdkProcessingMetadata.baseClientNormalized;
-        }
-      }
 
       expect(capturedEvent).toEqual(normalizedTransaction);
-      // expect(TestClient.instance!.event!).toEqual(normalizedTransaction);
     });
 
     test('calls beforeSend and uses original event without any changes', () => {

packages/core/test/mocks/client.ts

@@ -69,6 +69,10 @@ export class TestClient extends BaseClient<TestClientOptions> {
 
   public sendEvent(event: Event, hint?: EventHint): void {
     this.event = event;
+
+    // In real life, this will get deleted as part of envelope creation.
+    delete event.sdkProcessingMetadata;
+
     if (this._options.enableSend) {
       super.sendEvent(event, hint);
       return;

packages/integration-tests/suites/public-api/configureScope/clear_scope/test.ts

@@ -9,18 +9,6 @@ sentryTest('should clear previously set properties of a scope', async ({ getLoca
 
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
 
-  // TODO: This is to compensate for a temporary debugging hack which adds data the tests aren't anticipating to the
-  // event. The code can be restored to its original form (the commented-out line below) once that hack is
-  // removed. See https://github.com/getsentry/sentry-javascript/pull/4425 and
-  // https://github.com/getsentry/sentry-javascript/pull/4574
-  if (eventData.extra) {
-    if (Object.keys(eventData.extra).length === 1) {
-      delete eventData.extra;
-    } else {
-      delete eventData.extra.normalizeDepth;
-    }
-  }
-
   expect(eventData.message).toBe('cleared_scope');
   expect(eventData.user).toBeUndefined();
   expect(eventData.tags).toBeUndefined();

packages/integration-tests/utils/helpers.ts

@@ -70,26 +70,7 @@ async function getMultipleRequests(
       if (urlRgx.test(request.url())) {
         try {
           reqCount -= 1;
-
-          // TODO: This is to compensate for a temporary debugging hack which adds data the tests aren't anticipating to
-          // the request. The code can be restored to its original form (the commented-out line below) once that hack is
-          // removed. See https://github.com/getsentry/sentry-javascript/pull/4425.
-          const parsedRequest = requestParser(request);
-          if (parsedRequest.tags) {
-            if (
-              Object.keys(parsedRequest.tags).length === 0 ||
-              (Object.keys(parsedRequest.tags).length === 1 && 'skippedNormalization' in parsedRequest.tags)
-            ) {
-              delete parsedRequest.tags;
-            } else {
-              delete parsedRequest.tags.skippedNormalization;
-            }
-          }
-          if (parsedRequest.extra && Object.keys(parsedRequest.extra).length === 0) {
-            delete parsedRequest.extra;
-          }
-          requestData.push(parsedRequest);
-          // requestData.push(requestParser(request));
+          requestData.push(requestParser(request));
 
           if (reqCount === 0) {
             if (timeoutId) {

packages/node-integration-tests/suites/public-api/configureScope/clear_scope/test.ts

@@ -8,8 +8,6 @@ test('should clear previously set properties of a scope', async () => {
 
   assertSentryEvent(envelope[2], {
     message: 'cleared_scope',
-    tags: {},
-    extra: {},
   });
 
   expect((envelope[2] as Event).user).not.toBeDefined();

packages/node-integration-tests/suites/public-api/withScope/nested-scopes/test.ts

@@ -11,7 +11,6 @@ test('should allow nested scoping', async () => {
     user: {
       id: 'qux',
     },
-    tags: {},
   });
 
   assertSentryEvent(envelopes[3][2], {
@@ -49,6 +48,5 @@ test('should allow nested scoping', async () => {
     user: {
       id: 'qux',
     },
-    tags: {},
   });
 });