remove async/await from client code

boardgameio · Jun 19, 2018 · 6194986 · 6194986
1 parent 10abff0
commit 6194986
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 11 deletions.
diff --git a/src/client/debug/debug.js b/src/client/debug/debug.js
@@ -351,16 +351,16 @@ export class Debug extends React.Component {
     );
   }
 
-  simulate = async (iterations = 10000, sleepTimeout = 100) => {
-    function sleep() {
-      return new Promise(resolve => setTimeout(resolve, sleepTimeout));
-    }
-
-    for (let i = 0; i < iterations; i++) {
+  simulate = (iterations = 10000, sleepTimeout = 100) => {
+    const step = () => {
       const action = this.props.step();
-      if (!action) break;
-      await sleep(100);
-    }
+      if (action && iterations > 1) {
+        iterations--;
+        setTimeout(step, sleepTimeout);
+      }
+    };
+
+    step();
   };
 
   renderControls() {

diff --git a/src/client/debug/debug.test.js b/src/client/debug/debug.test.js
@@ -307,6 +307,8 @@ test('toggle AIDebug', () => {
 });
 
 describe('simulate', () => {
+  jest.useFakeTimers();
+
   test('basic', () => {
     const step = jest.fn(() => true);
     Enzyme.mount(
@@ -319,7 +321,8 @@ describe('simulate', () => {
     );
     expect(step).not.toHaveBeenCalled();
     Mousetrap.simulate('5');
-    expect(step).toHaveBeenCalled();
+    jest.runAllTimers();
+    expect(step).toHaveBeenCalledTimes(10000);
   });
 
   test('break out if no action is returned', () => {
@@ -335,6 +338,7 @@ describe('simulate', () => {
 
     expect(step).not.toHaveBeenCalled();
     Mousetrap.simulate('5');
-    expect(step).toHaveBeenCalled();
+    jest.runAllTimers();
+    expect(step).toHaveBeenCalledTimes(1);
   });
 });