fix: support empty subscription marbles (#5502)

* test: add failing subscription marble tests * fix: support empty subscription marbles Closes #5499
ReactiveX · Jun 22, 2020 · e65696e · e65696e
1 parent edde63b
commit e65696e
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/spec/schedulers/TestScheduler-spec.ts b/spec/schedulers/TestScheduler-spec.ts
@@ -282,6 +282,18 @@ describe('TestScheduler', () => {
         expectSubscriptions(source.subscriptions).toBe(subs);
         source.subscribe();
       });
+
+      it('should support empty subscription marbles', () => {
+        const source = cold('---a---b-|');
+        const subs =        '----------';
+        expectSubscriptions(source.subscriptions).toBe(subs);
+      });
+
+      it('should support empty subscription marbles within arrays', () => {
+        const source = cold('---a---b-|');
+        const subs =       ['----------'];
+        expectSubscriptions(source.subscriptions).toBe(subs);
+      });
     });
 
     describe('end-to-end helper tests', () => {

diff --git a/src/internal/testing/TestScheduler.ts b/src/internal/testing/TestScheduler.ts
@@ -170,7 +170,7 @@ export class TestScheduler extends VirtualTimeScheduler {
         flushTest.ready = true;
         flushTest.expected = marblesArray.map(marbles =>
           TestScheduler.parseMarblesAsSubscriptions(marbles, runMode)
-        );
+        ).filter(marbles => marbles.subscribedFrame !== Infinity);
       }
     };
   }