fix(tests): increase test fuzziness

These tests periodically fail in CI, 10 milliseconds is very much not
enough time to account for fuzziness, and 5 minutes is more than enough
precision given the scales of time involved in each test.

PR-URL: https://github.com/npm/cli/pull/3201
Credit: @wraithgar
Close: #3201
Reviewed-by: @nlf

1 files changed, 4 insertions, 4 deletions

diff --git a/test/lib/utils/update-notifier.js b/test/lib/utils/update-notifier.js
index 1735b3105..ad4d40772 100644
--- a/test/lib/utils/update-notifier.js
+++ b/test/lib/utils/update-notifier.js
@@ -145,15 +145,15 @@ t.test('situations in which we do not notify', t => {
   })
 
   t.test('only check weekly for GA releases', async t => {
-    // the 10 is fuzz factor for test environment
-    STAT_MTIME = Date.now() - (1000 * 60 * 60 * 24 * 7) + 10
+    // One week (plus five minutes to account for test environment fuzziness)
+    STAT_MTIME = Date.now() - (1000 * 60 * 60 * 24 * 7) + (1000 * 60 * 5)
     t.equal(await updateNotifier(npm), null)
     t.strictSame(MANIFEST_REQUEST, [], 'no requests for manifests')
   })
 
   t.test('only check daily for betas', async t => {
-    // the 10 is fuzz factor for test environment
-    STAT_MTIME = Date.now() - (1000 * 60 * 60 * 24) + 10
+    // One day (plus five minutes to account for test environment fuzziness)
+    STAT_MTIME = Date.now() - (1000 * 60 * 60 * 24) + (1000 * 60 * 5)
     t.equal(await updateNotifier({ ...npm, version: HAVE_BETA }), null)
     t.strictSame(MANIFEST_REQUEST, [], 'no requests for manifests')
   })