fix(combineLatest): emit unique array instances with the default proj…

…ection
ReactiveX · Aug 1, 2016 · 2e30fd1 · 2e30fd1
1 parent 897fe3b
commit 2e30fd1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/spec/operators/combineLatest-spec.ts b/spec/operators/combineLatest-spec.ts
@@ -461,4 +461,14 @@ describe('Observable.prototype.combineLatest', () => {
  expectSubscriptions(e1.subscriptions).toBe(e1subs);
  expectSubscriptions(e2.subscriptions).toBe(e2subs);
  });
+
+ it('should emit unique array instances with the default projection', () => {
+ const e1 = hot('-a--b--|');
+ const e2 = hot('--1--2-|');
+ const expected = '-------(c|)';
+
+ const result = e1.combineLatest(e2).distinct().count();
+
+ expectObservable(result).toBe(expected, { c: 3 });
+ });
 });
diff --git a/src/operator/combineLatest.ts b/src/operator/combineLatest.ts
@@ -150,7 +150,7 @@ export class CombineLatestSubscriber<T, R> extends OuterSubscriber<T, R> {
  if (this.project) {
  this._tryProject(values);
  } else {
- this.destination.next(values);
+ this.destination.next(values.slice());
  }
  }
  }