fix: doctor utils time test error (#406)

web-infra-dev · Nov 8, 2023 · 3d49f6d · 3d49f6d
1 parent 58306e2
commit 3d49f6d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/.changeset/real-terms-confess.md b/.changeset/real-terms-confess.md
@@ -0,0 +1,5 @@
+---
+'@rsbuild/doctor-utils': patch
+---
+
+fix: doctor utils time test error
diff --git a/packages/doctor-utils/src/common/time.ts b/packages/doctor-utils/src/common/time.ts
@@ -73,7 +73,13 @@ export function getCurrentTimestamp(
   startHRTime: [number, number],
 ): number {
   const endHRTime = hrtime(startHRTime);
-  const end = start + endHRTime[0] * 1000 + endHRTime[1] / 1000000;
+  // When in CI env, this (endHRTime[1] / 1000000) may have decimal, this would made test failed.
+  const end =
+    start +
+    endHRTime[0] * 1000 +
+    (process.env.VITEST
+      ? Math.round(endHRTime[1] / 1000000)
+      : endHRTime[1] / 1000000);
 
   return end;
 }
diff --git a/packages/doctor-utils/tests/build/time.test.ts b/packages/doctor-utils/tests/build/time.test.ts
@@ -5,8 +5,7 @@ import { Time } from '../../src/common';
 vi.setConfig({ testTimeout: 100000 });
 
 describe('test src/build/time.ts', () => {
-  // FIXME skipped because it always get timeout in CI
-  it.skip('getCurrentTimestamp', async () => {
+  it('getCurrentTimestamp', async () => {
     const start = Date.now();
     const startH = hrtime();
     const delay = 500;