test: fix testcases

opensumi · Jun 21, 2024 · 59675fc · 59675fc
1 parent 4420f08
commit 59675fc
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/__tests__/commander/commander.test.ts b/__tests__/commander/commander.test.ts
@@ -80,7 +80,6 @@ describe('command center', () => {
     });
 
     cc.on('hello', async (ctx) => {
-      console.log(`🚀 ~ file: commander.test.ts:53 ~ cc.on ~ ctx:`, ctx);
       await Promise.race([
         (async () => {
           await sleep(5 * 1000);

diff --git a/__tests__/kv/index.test.ts b/__tests__/kv/index.test.ts
@@ -20,6 +20,7 @@ describe('KV', () => {
       expect(json).toEqual({ key: 'value' });
     });
     it('can cache', async () => {
+      jest.useFakeTimers();
       const manager = KVManager.for<any>('test', 500);
 
       const spy = jest.spyOn(manager.kv, 'get');
@@ -36,12 +37,20 @@ describe('KV', () => {
 
       // should only call once, the second time should get from cache
       expect(spy).toBeCalledTimes(1);
-      await sleep(4000);
+
+      jest.advanceTimersByTime(300);
 
       const c = await manager.getJSONCached('key');
       expect(c).toEqual({ key: 'value' });
+      expect(spy).toBeCalledTimes(1);
+
+      jest.advanceTimersByTime(4000);
+
+      const d = await manager.getJSONCached('key');
+      expect(d).toEqual({ key: 'value' });
 
       expect(spy).toBeCalledTimes(2);
-    }, 10000);
+      jest.useRealTimers();
+    });
   });
 });