fix tests

Author: lobsterkatie

packages/core/src/request.ts

@@ -59,15 +59,19 @@ export function eventToSentryRequest(event: Event, api: APIDetails): SentryReque
   const { method: samplingMethod, rate: sampleRate } = transactionSampling || {};
 
   // TODO: Below is a temporary hack in order to debug a serialization error - see
-  // https://github.com/getsentry/sentry-javascript/issues/2809 and
-  // https://github.com/getsentry/sentry-javascript/pull/4425. TL;DR: even though we normalize all events (which should
-  // prevent this), something is causing `JSON.stringify` to throw a circular reference error.
+  // https://github.com/getsentry/sentry-javascript/issues/2809,
+  // https://github.com/getsentry/sentry-javascript/pull/4425, and
+  // https://github.com/getsentry/sentry-javascript/pull/4574.
+  //
+  // TL; DR: even though we normalize all events (which should prevent this), something is causing `JSON.stringify` to
+  // throw a circular reference error.
   //
   // When it's time to remove it:
   // 1. Delete everything between here and where the request object `req` is created, EXCEPT the line deleting
   //    `sdkProcessingMetadata`
   // 2. Restore the original version of the request body, which is commented out
-  // 3. Search for `skippedNormalization` and pull out the companion hack in the browser playwright tests
+  // 3. Search for either of the PR URLs above and pull out the companion hacks in the browser playwright tests and the
+  //    baseClient tests in this package
   enhanceEventWithSdkInfo(event, api.metadata.sdk);
   event.tags = event.tags || {};
   event.extra = event.extra || {};

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

@@ -867,7 +867,21 @@ describe('BaseClient', () => {
 
       client.captureEvent(transaction);
 
-      expect(TestBackend.instance!.event!).toEqual(normalizedTransaction);
+      // 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 = TestBackend.instance!.event!;
+      if (capturedEvent.sdkProcessingMetadata?.normalizeDepth) {
+        if (Object.keys(capturedEvent.sdkProcessingMetadata).length === 1) {
+          delete capturedEvent.sdkProcessingMetadata;
+        } else {
+          delete capturedEvent.sdkProcessingMetadata.normalizeDepth;
+        }
+      }
+
+      expect(capturedEvent).toEqual(normalizedTransaction);
+      // expect(TestBackend.instance!.event!).toEqual(normalizedTransaction);
     });
 
     test('calls beforeSend and uses original event without any changes', () => {