refactor: wip

Author: BioPhoton

packages/utils/src/lib/profiler/profiler-node.unit.test.ts

@@ -294,10 +294,10 @@ describe('NodejsProfiler', () => {
 
       profiler.close();
 
-      expect(() => profiler.setEnabled(true)).toThrowError(
+      expect(() => profiler.setEnabled(true)).toThrow(
         'Profiler already closed',
       );
-      expect(() => profiler.setEnabled(false)).toThrowError(
+      expect(() => profiler.setEnabled(false)).toThrow(
         'Profiler already closed',
       );
 
@@ -317,7 +317,7 @@ describe('NodejsProfiler', () => {
         // This should not throw since we're using the public API correctly
         profiler.setEnabled(false);
         profiler.setEnabled(true);
-      }).not.toThrowError();
+      }).not.toThrow();
     });
   });
 
@@ -399,7 +399,7 @@ describe('NodejsProfiler', () => {
         profiler.measure('error-test', () => {
           throw error;
         });
-      }).toThrowError(error);
+      }).toThrow(error);
     });
 
     it('should propagate errors from measureAsync work function', async () => {
@@ -410,7 +410,7 @@ describe('NodejsProfiler', () => {
         profiler.measureAsync('async-error-test', async () => {
           throw error;
         }),
-      ).rejects.toThrowError(error);
+      ).rejects.toThrow(error);
     });
 
     it('should skip measurement when profiler is not active', () => {
@@ -447,7 +447,7 @@ describe('NodejsProfiler', () => {
 
       expect(() => {
         profiler.marker('inactive-marker');
-      }).not.toThrowError();
+      }).not.toThrow();
     });
 
     it('base Profiler behavior: should always be active in base profiler', () => {
@@ -471,7 +471,7 @@ describe('NodejsProfiler', () => {
 
       expect(() => {
         profiler.marker('base-marker');
-      }).not.toThrowError();
+      }).not.toThrow();
     });
   });
 
@@ -789,7 +789,7 @@ describe('NodejsProfiler', () => {
     });
 
     it('installs exit handlers on construction', () => {
-      expect(() => createProfiler()).not.toThrowError();
+      expect(() => createProfiler()).not.toThrow();
 
       expect(mockSubscribeProcessExit).toHaveBeenCalledWith({
         onError: expect.any(Function),

packages/utils/src/lib/profiler/profiler.int.test.ts

@@ -231,7 +231,7 @@ describe('Profiler Integration', () => {
       profiler.measure('error-test', () => {
         throw error;
       });
-    }).toThrowError(error);
+    }).toThrow(error);
 
     const measures = performance.getEntriesByType('measure');
     expect(measures).toStrictEqual(
@@ -258,7 +258,7 @@ describe('Profiler Integration', () => {
       profiler.measure('custom-error-test', () => {
         throw customError;
       });
-    }).toThrowError(customError);
+    }).toThrow(customError);
 
     const measures = performance.getEntriesByType('measure');
     expect(measures).toStrictEqual(

packages/utils/src/lib/profiler/profiler.unit.test.ts

@@ -154,7 +154,7 @@ describe('Profiler', () => {
         tooltipText: 'Test marker',
         properties: [['key', 'value']],
       });
-    }).not.toThrowError();
+    }).not.toThrow();
 
     const marks = performance.getEntriesByType('mark');
     expect(marks).toStrictEqual([
@@ -181,7 +181,7 @@ describe('Profiler', () => {
       profilerWithColor.marker('test-marker-default-color', {
         tooltipText: 'Test marker with default color',
       });
-    }).not.toThrowError();
+    }).not.toThrow();
 
     const marks = performance.getEntriesByType('mark');
     expect(marks).toStrictEqual([
@@ -207,7 +207,7 @@ describe('Profiler', () => {
         tooltipText: 'Test marker without default color',
         properties: [['key', 'value']],
       });
-    }).not.toThrowError();
+    }).not.toThrow();
 
     const marks = performance.getEntriesByType('mark');
     expect(marks).toStrictEqual([
@@ -233,7 +233,7 @@ describe('Profiler', () => {
         color: 'primary',
         tooltipText: 'This should not create a mark',
       });
-    }).not.toThrowError();
+    }).not.toThrow();
 
     const marks = performance.getEntriesByType('mark');
     expect(marks).toHaveLength(0);
@@ -304,7 +304,7 @@ describe('Profiler', () => {
       throw error;
     });
 
-    expect(() => profiler.measure('test-event', workFn)).toThrowError(error);
+    expect(() => profiler.measure('test-event', workFn)).toThrow(error);
     expect(workFn).toHaveBeenCalled();
   });
 
@@ -314,7 +314,7 @@ describe('Profiler', () => {
       throw error;
     });
 
-    expect(() => profiler.measure('test-event', workFn)).toThrowError(error);
+    expect(() => profiler.measure('test-event', workFn)).toThrow(error);
     expect(workFn).toHaveBeenCalled();
   });
 
@@ -325,9 +325,9 @@ describe('Profiler', () => {
       throw error;
     });
 
-    expect(() =>
-      enabledProfiler.measure('test-event-error', workFn),
-    ).toThrowError(error);
+    expect(() => enabledProfiler.measure('test-event-error', workFn)).toThrow(
+      error,
+    );
     expect(workFn).toHaveBeenCalled();
 
     // Verify that performance marks were created even though error occurred
@@ -424,7 +424,7 @@ describe('Profiler', () => {
 
     await expect(
       profiler.measureAsync('test-async-event', workFn),
-    ).rejects.toThrowError(error);
+    ).rejects.toThrow(error);
     expect(workFn).toHaveBeenCalled();
   });
 
@@ -438,7 +438,7 @@ describe('Profiler', () => {
 
     await expect(
       enabledProfiler.measureAsync('test-async-event-error', workFn),
-    ).rejects.toThrowError(error);
+    ).rejects.toThrow(error);
     expect(workFn).toHaveBeenCalled();
 
     // Verify that performance marks were created even though error occurred

packages/utils/src/lib/wal.unit.test.ts

@@ -34,7 +34,7 @@ describe('createTolerantCodec', () => {
         throw new Error('decoding error');
       },
     });
-    expect(() => c.encode(42)).toThrowError('encoding error');
+    expect(() => c.encode(42)).toThrow('encoding error');
     const result = c.decode('42');
     expect(result).toEqual({ __invalid: true, raw: '42' });
   });
@@ -169,7 +169,7 @@ describe('WriteAheadLogFile', () => {
     w.open();
     expect(() =>
       w.append('{ id: 1, name:...' as unknown as object),
-    ).not.toThrowError();
+    ).not.toThrow();
     w.close();
     expect(w.recover().records).toStrictEqual([
       { id: 1, name: 'test' },
@@ -187,7 +187,7 @@ describe('WriteAheadLogFile', () => {
   it('throws error when appending without opening', () => {
     const w = wal('/test/a.log');
     expect(w.isClosed()).toBeTrue();
-    expect(() => w.append('a')).toThrowError('WAL not opened');
+    expect(() => w.append('a')).toThrow('WAL not opened');
   });
 
   it('opens and closes correctly', () => {
@@ -920,7 +920,7 @@ describe('ShardedWal', () => {
     vol.unlinkSync(
       '/shards/20231114-221320-000/test.20231114-221320-000.10001.2.1.log',
     );
-    expect(() => sw.cleanup()).not.toThrowError();
+    expect(() => sw.cleanup()).not.toThrow();
   });
 
   it('should use custom options in finalizer', () => {