[Expressions] Fix expressions execution unit test to use fake timers …

…for reliability (elastic#99952) (elastic#100016)
kibanamachine · May 13, 2021 · 6554b9c · 6554b9c
1 parent 5f5401c
commit 6554b9c
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/plugins/expressions/common/execution/execution.test.ts b/src/plugins/expressions/common/execution/execution.test.ts
@@ -362,14 +362,21 @@ describe('Execution', () => {
     });
 
     test('result is undefined until execution completes', async () => {
+      jest.useFakeTimers();
       const execution = createExecution('sleep 10');
       expect(execution.state.get().result).toBe(undefined);
       execution.start(null).subscribe(jest.fn());
       expect(execution.state.get().result).toBe(undefined);
-      await new Promise((r) => setTimeout(r, 1));
+
+      jest.advanceTimersByTime(1);
+      await new Promise(process.nextTick);
       expect(execution.state.get().result).toBe(undefined);
-      await new Promise((r) => setTimeout(r, 11));
+
+      jest.advanceTimersByTime(10);
+      await new Promise(process.nextTick);
       expect(execution.state.get().result).toBe(null);
+
+      jest.useRealTimers();
     });
 
     test('handles functions returning observables', () => {