fix(zipAll): complete when the source is empty

[martinsik]

This fixes the situation where the zipAll() operator doesn't send complete notification when the source Observable is empty:

Observable
  .empty()
  .zipAll()
  .subscribe(null, null, () => console.log('complete 1'));

This doesn't print anything. Demo: https://jsbin.com/fucijuz/2/edit?js,console

Description:

This problem occurs only when using zipAll() and not zip(). The zip() operator internally wraps zipAll() with ArrayObservable. So the following works fine:

Observable
  .zip(Observable.empty())
  .subscribe(null, null, () => console.log('complete 2'));

This is internally called as:

Observable
  .from([Observable.empty()])
  .zipAll()
  .subscribe(null, null, () => console.log('complete 3'));

which is all right.

Related issue (if exists):
#2426

[coveralls]

Coverage increased (+0.001%) to 97.689% when pulling 712fece on martinsik:zip-send-complete-on-empty into 69d051b on ReactiveX:master.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.