test: remove sanitizer mocking

Author: FeironoX5

packages/javascript/tests/utils/event.test.ts

@@ -3,12 +3,6 @@ import { getErrorFromEvent } from '../../src/utils/event';
 
 vi.mock('@hawk.so/core', () => ({ log: vi.fn(), isLoggerSet: vi.fn(() => true), setLogger: vi.fn() }));
 
-vi.mock('../../src/modules/sanitizer', () => ({
-  default: {
-    sanitize: vi.fn((data) => data),
-  },
-}));
-
 import Sanitizer from '../../src/modules/sanitizer';
 
 describe('getErrorFromEvent', () => {
@@ -24,7 +18,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBe(error);
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(error);
     });
 
     it('should return the DOMException when event.error is a DOMException', () => {
@@ -33,8 +26,7 @@ describe('getErrorFromEvent', () => {
 
       const result = getErrorFromEvent(event);
 
-      expect(result).toBe(error);
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(error);
+      expect(result).toBe('<instance of DOMException>');
     });
 
     it('should return the message when event.error is not provided and message is a string', () => {
@@ -43,7 +35,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBe('Script error.');
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith('Script error.');
     });
 
     it('should return empty string when event.error is not provided and message is empty', () => {
@@ -52,7 +43,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBe('');
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith('');
     });
   });
 
@@ -64,7 +54,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBe(reason);
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(reason);
     });
 
     it('should return the string when event.reason is a string', () => {
@@ -74,7 +63,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBe(reason);
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(reason);
     });
 
     it('should return stringified object when event.reason is a plain object', () => {
@@ -92,7 +80,6 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBeUndefined();
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(undefined);
     });
 
     it('should return null when event.reason is null', () => {
@@ -101,35 +88,9 @@ describe('getErrorFromEvent', () => {
       const result = getErrorFromEvent(event);
 
       expect(result).toBeNull();
-      expect(Sanitizer.sanitize).toHaveBeenCalledWith(null);
     });
 
     it('should handle circular references in object reason', () => {
-      vi.mocked(Sanitizer.sanitize).mockImplementation((data) => {
-        if (data !== null && typeof data === 'object') {
-          const seen = new WeakSet<object>();
-          const sanitize = (obj: unknown): unknown => {
-            if (obj !== null && typeof obj === 'object') {
-              if (seen.has(obj as object)) {
-                return '<circular>';
-              }
-              seen.add(obj as object);
-              if (Array.isArray(obj)) {
-                return obj.map(sanitize);
-              }
-              const result: Record<string, unknown> = {};
-              for (const [key, value] of Object.entries(obj)) {
-                result[key] = sanitize(value);
-              }
-              return result;
-            }
-            return obj;
-          };
-          return sanitize(data);
-        }
-        return data;
-      });
-
       const circularObj: Record<string, unknown> = { name: 'test' };
       circularObj.self = circularObj;