Fix maxInitWaitTime tests by adding a bit of tolerance to timing checks

configcat · Nov 10, 2023 · 477342c · 477342c
1 parent 8cc752e
commit 477342c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/ConfigCatClientTests.ts b/test/ConfigCatClientTests.ts
@@ -568,7 +568,7 @@ describe("ConfigCatClient", () => {
     const actualValue = await client.getValueAsync("debug", false);
     const elapsedMilliseconds: number = new Date().getTime() - startDate;
 
-    assert.isAtLeast(elapsedMilliseconds, maxInitWaitTimeSeconds * 1000);
+    assert.isAtLeast(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) - 10); // 10 ms for tolerance
     assert.isAtMost(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) + 50); // 50 ms for tolerance
     assert.equal(actualValue, false);
   });
@@ -605,7 +605,7 @@ describe("ConfigCatClient", () => {
       const state = await client.waitForReady();
       const elapsedMilliseconds: number = new Date().getTime() - startDate;
 
-      assert.isAtLeast(elapsedMilliseconds, maxInitWaitTimeSeconds * 1000);
+      assert.isAtLeast(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) - 10); // 10 ms for tolerance
       assert.isAtMost(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) + 50); // 50 ms for tolerance
 
       assert.equal(state, ClientReadyState.NoFlagData);
@@ -631,7 +631,7 @@ describe("ConfigCatClient", () => {
       const state = await client.waitForReady();
       const elapsedMilliseconds: number = new Date().getTime() - startDate;
 
-      assert.isAtLeast(elapsedMilliseconds, maxInitWaitTimeSeconds * 1000);
+      assert.isAtLeast(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) - 10); // 10 ms for tolerance
       assert.isAtMost(elapsedMilliseconds, (maxInitWaitTimeSeconds * 1000) + 50); // 50 ms for tolerance
 
       assert.equal(state, ClientReadyState.HasCachedFlagDataOnly);