refactor: wip

Author: John Doe

packages/utils/src/lib/clock-epoch.unit.test.ts

@@ -34,7 +34,7 @@ describe('epochClock', () => {
   it('should support performance clock by default for epochNowUs', () => {
     const c = epochClock();
     expect(c.timeOriginMs).toBe(500_000);
-    expect(c.epochNowUs()).toBe(500_000_000); // timeOrigin + performance.now() = timeOrigin + 0
+    expect(c.epochNowUs()).toBe(500_000_000);
   });
 
   it.each([
@@ -86,7 +86,7 @@ describe('epochClock', () => {
     );
     expect(defaultClock.fromEntry(markEntry, true)).toBe(
       defaultClock.fromPerfMs(1000),
-    ); // useEndTime doesn't matter for marks
+    );
   });
 
   it('should convert performance measure to microseconds', () => {
@@ -99,13 +99,13 @@ describe('epochClock', () => {
 
     expect(defaultClock.fromEntry(measureEntry)).toBe(
       defaultClock.fromPerfMs(1000),
-    ); // useEndTime = false (default)
+    );
     expect(defaultClock.fromEntry(measureEntry, false)).toBe(
       defaultClock.fromPerfMs(1000),
-    ); // explicit false
+    );
     expect(defaultClock.fromEntry(measureEntry, true)).toBe(
       defaultClock.fromPerfMs(1500),
-    ); // useEndTime = true, adds duration
+    );
   });
 
   it('should convert Date.now() milliseconds to microseconds', () => {

packages/utils/src/lib/trace-file-utils.unit.test.ts

@@ -1,7 +1,6 @@
 import type { PerformanceMark, PerformanceMeasure } from 'node:perf_hooks';
 import { describe, expect, it } from 'vitest';
 import {
-  entryToTraceTimestamp,
   frameName,
   frameTreeNodeId,
   getCompleteEvent,
@@ -363,52 +362,10 @@ describe('measureToSpanEvents', () => {
         tid: 666,
         args: { data: { detail: { measurement: 'data' } } },
       }),
-      expect.objectContaining({
-        name: 'custom-measure',
-        pid: 777,
-        tid: 666,
-        args: { data: { detail: { measurement: 'data' } } },
-      }),
     ]);
   });
 });
 
-describe('entryToTraceTimestamp', () => {
-  it('should convert entry timestamp for start time', () => {
-    expect(
-      typeof entryToTraceTimestamp({
-        startTime: 1000,
-        duration: 500,
-        entryType: 'measure',
-      } as PerformanceMeasure),
-    ).toBe('number');
-  });
-
-  it('should convert entry timestamp for end time', () => {
-    const mockEntry = {
-      startTime: 1000,
-      duration: 500,
-      entryType: 'measure',
-    } as PerformanceMeasure;
-
-    expect(entryToTraceTimestamp(mockEntry, true)).toBeGreaterThan(
-      entryToTraceTimestamp(mockEntry, false),
-    );
-  });
-
-  it('should handle non-measure entries', () => {
-    expect(
-      typeof entryToTraceTimestamp(
-        {
-          startTime: 1000,
-          entryType: 'mark',
-        } as PerformanceMark,
-        true,
-      ),
-    ).toBe('number');
-  });
-});
-
 describe('getSpanEvent', () => {
   it('should create begin event with args detail', () => {
     expect(