[tools] throttle.test.ts: Introduce 0.2ms tolerance

logtail · Jul 31, 2024 · 5ccae41 · 5ccae41
1 parent e3d1caf
commit 5ccae41
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/tools/src/batch.test.ts b/packages/tools/src/batch.test.ts
@@ -151,8 +151,9 @@ describe("batch tests", () => {
 
     // Expect time to have taken at least this long...
     const expectedTime = ((numberOfLogs / batchSize) * throttleResolveAfter) / maxThrottle;
+    const toleranceMilliseconds = 0.2;
 
-    expect(end).toBeGreaterThanOrEqual(expectedTime);
+    expect(end).toBeGreaterThanOrEqual(expectedTime - toleranceMilliseconds);
   });
 
   it("should send after flush (with long timeout)", async () => {

diff --git a/packages/tools/src/throttle.test.ts b/packages/tools/src/throttle.test.ts
@@ -46,8 +46,9 @@ describe("Throttle tests", () => {
 
     // Expect time to have taken (numberOfPromises / max) * throttleTime
     const expectedTime = (numberOfPromises / max) * throttleTime;
+    const toleranceMilliseconds = 0.2;
 
-    expect(end).toBeGreaterThanOrEqual(expectedTime);
+    expect(end).toBeGreaterThanOrEqual(expectedTime - toleranceMilliseconds);
   });
 
   it("should handle rejections", async () => {