fix(TestScheduler): flush expectations correctly

Closes ReactiveX#3898
cartant · Jul 9, 2018 · 66eb866 · 66eb866
1 parent ce52394
commit 66eb866
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/internal/testing/TestScheduler.ts b/src/internal/testing/TestScheduler.ts
@@ -141,17 +141,14 @@ export class TestScheduler extends VirtualTimeScheduler {
     }
 
     super.flush();
-    const { flushTests } = this;
-    const flushTestsCopy = flushTests.slice();
 
-    for (let i = 0, l = flushTests.length; i < l; i++) {
-      const test = flushTestsCopy[i];
+    this.flushTests = this.flushTests.filter(test => {
       if (test.ready) {
-        // remove it from the original array, not our copy
-        flushTests.splice(i, 1);
         this.assertDeepEqual(test.actual, test.expected);
+        return false;
       }
-    }
+      return true;
+    });
   }
 
   /** @nocollapse */