Merge pull request #48 from AtkinsSJ/fix-sleep-test-flake

Make sleep builtin test more tolerant of minor variations
HeyPuter · Mar 6, 2024 · a4b3bf1 · a4b3bf1
2 parents 2e2c598 + 4d7feec
commit a4b3bf1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/coreutils/sleep.js b/test/coreutils/sleep.js
@@ -85,8 +85,9 @@ export const runSleepTests = () => {
                     assert.fail(e);
                 }
                 const endTimeMs = performance.now();
-                const actualDurationMs = endTimeMs- startTimeMs;
-                assert.ok(actualDurationMs >= (durationS * 1000), `takes at least ${durationS} seconds`);
+                const actualDurationMs = endTimeMs - startTimeMs;
+                assert.ok(Math.abs(actualDurationMs - (durationS * 1000)) <= 10,
+                    `should pause for ${durationS} seconds, instead was ${actualDurationMs / 1000}`);
 
                 assert.equal(ctx.externs.out.output, '', 'sleep should not write to stdout');
                 assert.equal(ctx.externs.err.output, '', 'sleep should not write to stderr');