Improve time-sensitive unit test stability

Author: wojtekmaj

src/every.spec.js

@@ -42,7 +42,7 @@ describe('asyncEvery()', () => {
     const timeElapsed = timer.stop();
 
     expect(timeElapsed).toBeGreaterThanOrEqual(delay);
-    expect(timeElapsed).toBeLessThan(delay + 10);
+    expect(timeElapsed).toBeLessThan(delay * 1.25);
   });
 
   it.skip('assertions below are valid for synchronous .every()', () => {

src/every_strict.spec.js

@@ -50,8 +50,8 @@ describe('asyncEvery()', () => {
 
     const timeElapsed = timer.stop();
 
-    expect(timeElapsed).toBeGreaterThan(delay * 3);
-    expect(timeElapsed).toBeLessThan((delay + 10) * 3);
+    expect(timeElapsed).toBeGreaterThanOrEqual(delay * 3);
+    expect(timeElapsed).toBeLessThan(delay * 1.25 * 3);
   });
 
   it.skip('assertions below are valid for synchronous .every()', () => {

src/filter.spec.js

@@ -30,7 +30,7 @@ describe('asyncFilter()', () => {
     const timeElapsed = timer.stop();
 
     expect(timeElapsed).toBeGreaterThanOrEqual(delay);
-    expect(timeElapsed).toBeLessThan(delay + 10);
+    expect(timeElapsed).toBeLessThan(delay * 1.25);
   });
 
   it.skip('assertions below are valid for synchronous .filter()', () => {

src/filter_strict.spec.js

@@ -38,8 +38,8 @@ describe('asyncFilterStrict()', () => {
 
     const timeElapsed = timer.stop();
 
-    expect(timeElapsed).toBeGreaterThan(delay * 3);
-    expect(timeElapsed).toBeLessThan((delay + 10) * 3);
+    expect(timeElapsed).toBeGreaterThanOrEqual(delay * 3);
+    expect(timeElapsed).toBeLessThan(delay * 1.25 * 3);
   });
 
   it.skip('assertions below are valid for synchronous .filter()', () => {

src/forEach.spec.js

@@ -37,7 +37,7 @@ describe('asyncForEach()', () => {
     const timeElapsed = timer.stop();
 
     expect(timeElapsed).toBeGreaterThanOrEqual(delay);
-    expect(timeElapsed).toBeLessThan(delay + 10);
+    expect(timeElapsed).toBeLessThan(delay * 1.25);
   });
 
   it.skip('assertions below are valid for synchronous .forEach()', () => {

src/forEach_strict.spec.js

@@ -40,8 +40,8 @@ describe('asyncForEachStrict()', () => {
 
     const timeElapsed = timer.stop();
 
-    expect(timeElapsed).toBeGreaterThan(delay * 3);
-    expect(timeElapsed).toBeLessThan((delay + 10) * 3);
+    expect(timeElapsed).toBeGreaterThanOrEqual(delay * 3);
+    expect(timeElapsed).toBeLessThan(delay * 1.25 * 3);
   });
 
   it.skip('assertions below are valid for synchronous .forEach()', () => {

src/map.spec.js

@@ -31,7 +31,7 @@ describe('asyncMap()', () => {
     const timeElapsed = timer.stop();
 
     expect(timeElapsed).toBeGreaterThanOrEqual(delay);
-    expect(timeElapsed).toBeLessThan(delay + 10);
+    expect(timeElapsed).toBeLessThan(delay * 1.25);
   });
 
   it.skip('assertions below are valid for synchronous .map()', () => {

src/map_strict.spec.js

@@ -38,8 +38,8 @@ describe('asyncMapStrict()', () => {
 
     const timeElapsed = timer.stop();
 
-    expect(timeElapsed).toBeGreaterThan(delay * 3);
-    expect(timeElapsed).toBeLessThan((delay + 10) * 3);
+    expect(timeElapsed).toBeGreaterThanOrEqual(delay * 3);
+    expect(timeElapsed).toBeLessThan(delay * 1.25 * 3);
   });
 
   it.skip('assertions below are valid for synchronous .map()', () => {

src/reduce.spec.js

@@ -23,8 +23,8 @@ describe('asyncReduce()', () => {
 
     const timeElapsed = timer.stop();
 
-    expect(timeElapsed).toBeGreaterThan(delay * 3);
-    expect(timeElapsed).toBeLessThan((delay + 10) * 3);
+    expect(timeElapsed).toBeGreaterThanOrEqual(delay * 3);
+    expect(timeElapsed).toBeLessThan(delay * 1.25 * 3);
   });
 
   it.skip('assertions below are valid for synchronous .reduce()', () => {

src/some.spec.js

@@ -34,7 +34,7 @@ describe('asyncSome()', () => {
     const timeElapsed = timer.stop();
 
     expect(timeElapsed).toBeGreaterThanOrEqual(delay);
-    expect(timeElapsed).toBeLessThan(delay + 10);
+    expect(timeElapsed).toBeLessThan(delay * 1.25);
   });
 
   it.skip('assertions below are valid for synchronous .some()', () => {