fix(memoize): add jest

.github/workflows/test.yml

@@ -0,0 +1,31 @@
+# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created
+# For more information see: https://help.github.com/actions/language-and-framework-guides/publishing-nodejs-packages
+
+name: test
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
+        node: ['12', '14', '16']
+    name: ${{ matrix.os }} Use Node.js ${{ matrix.node }}
+    steps:
+      - name: Check out Git repository
+        uses: actions/checkout@v2
+      - name: Setup node
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node }}
+      - name: Setup yarn cache
+        uses: actions/cache@v2
+        with:
+          path: '**/node_modules'
+          key: ${{ matrix.os }}-${{ matrix.node }}-modules-${{ hashFiles('**/yarn.lock') }}
+      - name: Install dependencies
+        run: yarn
+      - name: Jest
+        run: yarn test

__tests__/promise.memoize.test.ts

@@ -0,0 +1,18 @@
+import { memoize, delay } from '../src/promise';
+
+describe('promise/memoize', () => {
+  test('cache 1000ms', async () => {
+    let i = 1;
+    const fn = memoize(
+      () => Promise.resolve(++i),
+      () => 1,
+      1000,
+    );
+    for (let i = 0; i < 100; i++) {
+      expect(await fn()).toBe(2);
+    }
+
+    await delay(1000);
+    expect(await fn()).toBe(3);
+  });
+});

src/promise/memoize.ts

@@ -27,7 +27,7 @@ function memoize<FnType extends AnyPromiseFN>(
 
     if (timeoutMs) {
       memos.forEach((item, k) => {
-        if (item.expiration > Date.now()) {
+        if (item.expiration < Date.now()) {
           memos.delete(k);
         }
       });
@@ -37,10 +37,6 @@ function memoize<FnType extends AnyPromiseFN>(
       return memos.get(key)!.value;
     }
 
-    if (queues.has(key)) {
-      return await queues.get(key)!;
-    }
-
     const promise = fn(...args);
     queues.set(key, promise);