test faster

Signed-off-by: 🕷️ <[email protected]>

Author: zendive

Makefile

@@ -25,7 +25,10 @@ valid:
 	pnpm exec svelte-check --no-tsconfig # only for *.svelte files
 
 test: valid
-	deno test --no-check --trace-leaks --reporter=dot
+	deno test --parallel --no-check --trace-leaks --reporter=dot
+
+test-dev:
+	deno test --watch --parallel --no-check --trace-leaks
 
 prod: test
 	rm -rf $(BUILD_DIR) $(CHROME_ZIP)

tests/AnimationWrapper_test.ts

@@ -84,4 +84,4 @@ describe('AnimationWrapper', () => {
   });
 });
 
-await wait(1e1);
+await wait(10);

tests/EvalWrapper_test.ts

@@ -91,4 +91,4 @@ describe('EvalWrapper', () => {
   });
 });
 
-await wait(1e1);
+await wait(10);

tests/IdleWrapper_test.ts

@@ -96,4 +96,4 @@ describe('IdleWrapper', () => {
   });
 });
 
-await wait(1e1);
+await wait(10);

tests/TimerWrapper_test.ts

@@ -266,7 +266,7 @@ describe('wrappers', () => {
     expect(Fact.check(rec.facts, ClearTimerFact.BAD_HANDLER)).toBe(false);
   });
 
-  test('clearIntervalHistory - non existent handler', () => {
+  test('clearIntervalHistory - non-existent handler', () => {
     globalThis.clearInterval(1000);
 
     const rec = Array.from(apiTimer.clearIntervalHistory.values())[0];
@@ -287,4 +287,4 @@ describe('wrappers', () => {
   });
 });
 
-await wait(1e1);
+await wait(10);

tests/browserPolyfill.ts

@@ -1,4 +1,3 @@
-console.log('👋 globals');
 import { PropertySymbol, Window } from 'happy-dom';
 
 const window = new Window();
@@ -11,17 +10,19 @@ Object.assign(globalThis, {
   location: window.location,
   Element: browserWindow.Element,
   Document: browserWindow.Document,
+
   requestAnimationFrame: function requestAnimationFrameDumbStub(
     callback: (delay: number) => void,
   ) {
     return setTimeout(() => {
       callback(performance.now());
     }, 1e3 / 60);
-    // We need to call endImmediate() before the callback as the callback might throw an error.
   },
+
   cancelAnimationFrame: function cancelAnimationFrameDumbStub(id: number) {
     clearTimeout(id);
   },
+
   requestIdleCallback: function requestIdleCallbackDumbStub(
     callback: (deadline: {
       didTimeout: boolean;
@@ -33,6 +34,7 @@ Object.assign(globalThis, {
       callback({ didTimeout: false, timeRemaining: performance.now() });
     }, options?.timeout || 0);
   },
+
   cancelIdleCallback: function cancelIdleCallbackDumbStub(id: number) {
     clearTimeout(id);
   },

tests/time_test.ts

@@ -11,6 +11,8 @@ import {
   wait,
 } from '../src/api/time.ts';
 
+const DELAY = 10;
+
 describe('Stopper', () => {
   let stopper: Stopper;
 
@@ -19,7 +21,6 @@ describe('Stopper', () => {
   });
 
   test('start/stop', async () => {
-    const DELAY = 10;
     stopper.start();
     await wait(DELAY);
     stopper.stop();
@@ -34,7 +35,6 @@ describe('Stopper', () => {
   });
 
   test('elapsed continues counting after stop', async () => {
-    const DELAY = 10;
     stopper.start();
     await wait(DELAY);
     stopper.stop();
@@ -58,7 +58,6 @@ describe('Stopper', () => {
 describe('Timer - default options', () => {
   test('start', async () => {
     let counter = 0;
-    const DELAY = 10;
     const timer = new Timer({ delay: DELAY }, () => {
       counter++;
     });
@@ -73,7 +72,6 @@ describe('Timer - default options', () => {
 
   test('stop before expected', async () => {
     let counter = 0;
-    const DELAY = 10;
     const timer = new Timer({ delay: 2 * DELAY }, () => {
       counter++;
     });
@@ -91,7 +89,6 @@ describe('Timer - default options', () => {
 describe('Timer - interval option', () => {
   test('start/stop', async () => {
     let counter = 0;
-    const DELAY = 10;
     const interval = new Timer({ delay: DELAY, repetitive: true }, () => {
       counter++;
     });
@@ -106,23 +103,27 @@ describe('Timer - interval option', () => {
 
 describe('Timer - animation + measurable', () => {
   test('start/stop', async () => {
-    const SECOND = 1e3;
     let count = 0;
-    const timer = new Timer({ animation: true, measurable: true }, () => {
+    const TIMESPAN = 100;
+    const timer = new Timer({
+      animation: true,
+      repetitive: true,
+      measurable: true,
+    }, () => {
       count++;
     });
 
     expect(timer.isPending()).toBe(false);
-
     timer.start();
-
     expect(timer.isPending()).toBe(true);
-    await wait(SECOND);
+
+    await wait(TIMESPAN);
 
     timer.stop();
     expect(timer.isPending()).toBe(false);
-    expect(timer.callbackSelfTime < 0.5).toBe(true);
-    expect(count).toBeLessThan(62);
+    expect(timer.callbackSelfTime).toBeLessThan(0.5);
+    expect(count).toBeGreaterThan(1);
+    expect(count).toBeLessThan(TIMESPAN / (1e3 / 60));
   });
 });
 
@@ -174,4 +175,4 @@ describe('callableOnce', () => {
   });
 });
 
-await wait(1e3);
+await wait(10);