patch(vest): add cache invalidation for canceled tests

.gitignore

@@ -1,6 +1,7 @@
 packages/*/types/
 packages/vest/classnames/
 packages/vest/promisify/
+packages/vest/parser/
 packages/anyone/all/
 packages/anyone/any/
 packages/anyone/none/

packages/shared/src/__tests__/cache.test.ts

@@ -100,4 +100,14 @@ describe('lib: cache', () => {
       });
     });
   });
+
+  describe('cache.invalidate', () => {
+    it('Should remove cached item from cache storage by its dependcies', () => {
+      const deps = [1, 2, 3];
+      c(deps, Math.random);
+      expect(c.get(deps)).not.toBeNull();
+      c.invalidate(deps);
+      expect(c.get(deps)).toBeNull();
+    });
+  });
 });

packages/shared/src/cache.ts

@@ -6,13 +6,10 @@ import { lengthEquals } from 'lengthEquals';
 export default function createCache(maxSize = 10): {
   <T>(deps: unknown[], cacheAction: (...args: unknown[]) => T): T;
   get(deps: unknown[]): any;
+  invalidate(item: any): void;
 } {
   const cacheStorage: Array<[unknown[], any]> = [];
 
-  /**
-   * @param {any[]} deps  dependency array.
-   * @param {Function}    cache action function.
-   */
   const cache = <T>(
     deps: unknown[],
     cacheAction: (...args: unknown[]) => T
@@ -24,17 +21,22 @@ export default function createCache(maxSize = 10): {
     const result = cacheAction();
     cacheStorage.unshift([deps.concat(), result]);
 
-    if (cacheStorage.length > maxSize) {
-      cacheStorage.length = maxSize;
-    }
+    if (cacheStorage.length > maxSize) cacheStorage.length = maxSize;
 
     return result;
   };
 
-  /**
-   * Retrieves an item from the cache.
-   * @param {deps} deps Dependency array
-   */
+  // invalidate an item in the cache by its dependencies
+  cache.invalidate = (deps: any[]): void => {
+    const index = cacheStorage.findIndex(
+      ([cachedDeps]) =>
+        lengthEquals(deps, cachedDeps.length) &&
+        deps.every((dep, i) => dep === cachedDeps[i])
+    );
+    if (index > -1) cacheStorage.splice(index, 1);
+  };
+
+  // Retrieves an item from the cache.
   cache.get = (deps: unknown[]): [unknown[], any] | null =>
     cacheStorage[
       cacheStorage.findIndex(

packages/vest/parser/package.json

@@ -0,0 +1,7 @@
+{
+  "main": "../dist/cjs/parser.js",
+  "module": "../dist/es/parser.production.js",
+  "name": "parser",
+  "types": "../types/parser.d.ts",
+  "private": true
+}

packages/vest/src/core/test/__tests__/memo.test.ts

@@ -111,6 +111,37 @@ describe('test.memo', () => {
         }
       });
     });
+
+    describe('Test is canceled', () => {
+      it('Should refresh', async () => {
+        let count = 0;
+        const tests = [];
+        const suite = vest.create(() => {
+          count++;
+
+          tests.push(
+            vestTest.memo(
+              'f1',
+              async () => {
+                await wait(10);
+              },
+              [true]
+            )
+          );
+
+          if (count === 1) {
+            tests[0].cancel();
+          }
+        });
+
+        suite();
+        suite();
+        suite();
+
+        expect(tests[0]).not.toBe(tests[1]);
+        expect(tests[1]).toBe(tests[2]);
+      });
+    });
   });
 
   describe('cache miss', () => {

packages/vest/src/core/test/test.memo.ts

@@ -17,9 +17,7 @@ export default function bindTestMemo(test: TTestBase): {
 } {
   const cache = createCache(100); // arbitrary cache size
 
-  /**
-   * Caches, or returns an already cached test call
-   */
+  // Caches, or returns an already cached test call
   function memo(
     fieldName: string,
     ...args: [test: TTestFn, deps: unknown[]]
@@ -28,6 +26,7 @@ export default function bindTestMemo(test: TTestBase): {
     fieldName: string,
     ...args: [message: string, test: TTestFn, deps: unknown[]]
   ): VestTest;
+  // eslint-disable-next-line max-statements
   function memo(
     fieldName: string,
     ...args:
@@ -49,6 +48,12 @@ export default function bindTestMemo(test: TTestBase): {
       return cache(dependencies, () => test(fieldName, msg, testFn));
     }
 
+    if (cached[1].isCanceled()) {
+      // cache hit, but test is canceled
+      cache.invalidate(dependencies);
+      return cache(dependencies, () => test(fieldName, msg, testFn));
+    }
+
     return registerPrevRunTest(cached[1]);
   }
 

packages/vest/src/hooks/group.ts

@@ -7,7 +7,7 @@ import context from 'ctx';
 /**
  * Runs a group callback.
  */
-export default function group(groupName: string, tests: () => any): void {
+export default function group(groupName: string, tests: () => void): void {
   if (!isStringValue(groupName)) {
     throwGroupError('name must be a string');
   }