From 8ea7e1278d329cfe3969abc2e0c9a685fed19b75 Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Fri, 10 Jul 2026 00:23:59 +0000 Subject: [PATCH 1/2] [Tests] Add unit tests for hrtime utilities --- .../cli-kit/src/public/node/hrtime.test.ts | 51 +++++++++++++++++++ 1 file changed, 51 insertions(+) create mode 100644 packages/cli-kit/src/public/node/hrtime.test.ts diff --git a/packages/cli-kit/src/public/node/hrtime.test.ts b/packages/cli-kit/src/public/node/hrtime.test.ts new file mode 100644 index 00000000000..d93143e8630 --- /dev/null +++ b/packages/cli-kit/src/public/node/hrtime.test.ts @@ -0,0 +1,51 @@ +import {startHRTime, endHRTimeInMs} from './hrtime.js' +import {describe, test, expect, vi, afterEach} from 'vitest' + +describe('hrtime', () => { + afterEach(() => { + vi.restoreAllMocks() + }) + + test('startHRTime returns the current high-resolution real time', () => { + // Given + const mockTime: [number, number] = [123, 456] + vi.spyOn(process, 'hrtime').mockReturnValue(mockTime) + + // When + const result = startHRTime() + + // Then + expect(process.hrtime).toHaveBeenCalled() + expect(result).toEqual(mockTime) + }) + + test('endHRTimeInMs returns the time in milliseconds with 2 decimal places', () => { + // Given + const startTime: [number, number] = [100, 0] + const diffTime: [number, number] = [1, 500000] + vi.spyOn(process, 'hrtime').mockReturnValue(diffTime) + + // When + const result = endHRTimeInMs(startTime) + + // Then + expect(process.hrtime).toHaveBeenCalledWith(startTime) + // 1 second + 500,000 nanoseconds = 1000ms + 0.5ms = 1000.5ms + expect(result).toBe('1000.50') + }) + + test('endHRTimeInMs handles sub-millisecond precision', () => { + // Given + const startTime: [number, number] = [100, 0] + const diffTime: [number, number] = [0, 1234567] + vi.spyOn(process, 'hrtime').mockReturnValue(diffTime) + + // When + const result = endHRTimeInMs(startTime) + + // Then + expect(process.hrtime).toHaveBeenCalledWith(startTime) + // 0 seconds + 1,234,567 nanoseconds = 1.234567ms + expect(result).toBe('1.23') + }) +}) From 31030f875729b0fc95024c1b46a5c425d440ffc9 Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Fri, 10 Jul 2026 00:44:15 +0000 Subject: [PATCH 2/2] [Tests] Add unit tests for hrtime utilities --- packages/cli-kit/src/public/node/hrtime.test.ts | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/packages/cli-kit/src/public/node/hrtime.test.ts b/packages/cli-kit/src/public/node/hrtime.test.ts index d93143e8630..aebb6b4501b 100644 --- a/packages/cli-kit/src/public/node/hrtime.test.ts +++ b/packages/cli-kit/src/public/node/hrtime.test.ts @@ -1,11 +1,7 @@ import {startHRTime, endHRTimeInMs} from './hrtime.js' -import {describe, test, expect, vi, afterEach} from 'vitest' +import {describe, test, expect, vi} from 'vitest' describe('hrtime', () => { - afterEach(() => { - vi.restoreAllMocks() - }) - test('startHRTime returns the current high-resolution real time', () => { // Given const mockTime: [number, number] = [123, 456]