test(Subscriber): add test for no error propagation

[staltz]

Add test to assert that error happening in the next() handler should not propagate to the error()
handler.

For issue #1135.

[luisgabriel]

LGTM.

[kwonoj]

Can we convert test case to synchronous? I've tried way like below snippet.

var subscriber = Subscriber.create(
  function next(value) {
    if (value === 2) {
      throw 'error!';
    }
  },
  errorSpy,
  completeSpy
);

subscriber.next(1);

expect(function () {
  subscriber.next(2);
}).toThrow('error!');

expect(errorSpy).not.toHaveBeenCalled();
expect(completeSpy).not.toHaveBeenCalled();

[staltz]

@kwonoj Done

[kwonoj]

👍 looks good to me.

[kwonoj]

Merged with d6f9668, thanks @staltz

[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.