add test

Author: Lms24

packages/node/src/client.ts

@@ -118,9 +118,7 @@ export class NodeClient extends BaseClient<NodeClientOptions> {
    */
   // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
   public eventFromException(exception: any, hint?: EventHint): PromiseLike<Event> {
-    return resolvedSyncPromise(
-      eventFromUnknownInput(this._options.stackParser, exception, hint, this._options.normalizeDepth),
-    );
+    return resolvedSyncPromise(eventFromUnknownInput(this._options.stackParser, exception, hint));
   }
 
   /**

packages/node/src/eventbuilder.ts

@@ -46,12 +46,7 @@ export function exceptionFromError(stackParser: StackParser, error: Error): Exce
  * Builds and Event from a Exception
  * @hidden
  */
-export function eventFromUnknownInput(
-  stackParser: StackParser,
-  exception: unknown,
-  hint?: EventHint,
-  normalizeDepth?: number,
-): Event {
+export function eventFromUnknownInput(stackParser: StackParser, exception: unknown, hint?: EventHint): Event {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   let ex: unknown = exception;
   const providedMechanism: Mechanism | undefined =
@@ -67,7 +62,10 @@ export function eventFromUnknownInput(
       // which is much better than creating new group when any key/value change
       const message = `Non-Error exception captured with keys: ${extractExceptionKeysForMessage(exception)}`;
 
-      getCurrentHub().configureScope(scope => {
+      const hub = getCurrentHub();
+      const client = hub.getClient();
+      const normalizeDepth = client && client.getOptions().normalizeDepth;
+      hub.configureScope(scope => {
         scope.setExtra('__serialized__', normalizeToSize(exception, normalizeDepth));
       });
 

packages/node/test/eventbuilders.test.ts

@@ -0,0 +1,75 @@
+import { Client } from '@sentry/types';
+import { defaultStackParser, Scope } from '../src';
+import { eventFromUnknownInput } from '../src/eventbuilder';
+
+const testScope = new Scope();
+
+jest.mock('@sentry/hub', () => {
+  const original = jest.requireActual('@sentry/hub');
+  return {
+    ...original,
+    getCurrentHub(): {
+      getClient(): Client;
+      getScope(): Scope;
+      configureScope(scopeFunction: (scope: Scope) => void): void;
+    } {
+      return {
+        getClient(): any {
+          return {
+            getOptions(): any {
+              return { normalizeDepth: 6 };
+            },
+          };
+        },
+        getScope(): Scope {
+          return new Scope();
+        },
+        configureScope(scopeFunction): void {
+          scopeFunction(testScope);
+        },
+      };
+    },
+  };
+});
+
+afterEach(() => {
+  jest.resetAllMocks();
+});
+
+describe('eventFromUnknownInput', () => {
+  test('uses normalizeDepth from init options', () => {
+    const deepObject = {
+      a: {
+        b: {
+          c: {
+            d: {
+              e: {
+                f: {
+                  g: 'foo',
+                },
+              },
+            },
+          },
+        },
+      },
+    };
+
+    eventFromUnknownInput(defaultStackParser, deepObject);
+
+    const serializedObject = (testScope as any)._extra.__serialized__;
+    expect(serializedObject).toBeDefined();
+    expect(serializedObject).toEqual({
+      a: {
+        b: {
+          c: {
+            d: {
+              e: {
+                f: '[Object]',
+              },
+            },
+          },
+        },
+      },
+    });
+  });
+});