Skip to content

Commit

Permalink
test: add failing invalid ObservableInput tests
Browse files Browse the repository at this point in the history
  • Loading branch information
cartant committed Mar 28, 2021
1 parent 02219cc commit 50c99a3
Show file tree
Hide file tree
Showing 4 changed files with 52 additions and 0 deletions.
13 changes: 13 additions & 0 deletions spec/operators/concatMap-spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -691,4 +691,17 @@ describe('Observable.prototype.concatMap', () => {
done(new Error('Subscriber complete handler not supposed to be called.'));
});
});

it('should report invalid observable inputs via error notifications', () => {
const e1 = hot('--1|');
const e1subs = '^ !';
const expected = '--#';

const result = e1.pipe(concatMap(() => null as any));

expectObservable(result).toBe(expected, null, new TypeError(
"You provided 'null' where a stream was expected. You can provide an Observable, Promise, Array, or Iterable."
));
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});
});
13 changes: 13 additions & 0 deletions spec/operators/exhaustMap-spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -434,4 +434,17 @@ describe('exhaustMap', () => {
expectSubscriptions(x.subscriptions).toBe(xsubs);
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});

it('should report invalid observable inputs via error notifications', () => {
const e1 = hot('--1|');
const e1subs = '^ !';
const expected = '--#';

const result = e1.pipe(exhaustMap(() => null as any));

expectObservable(result).toBe(expected, null, new TypeError(
"You provided 'null' where a stream was expected. You can provide an Observable, Promise, Array, or Iterable."
));
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});
});
13 changes: 13 additions & 0 deletions spec/operators/mergeMap-spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -852,4 +852,17 @@ describe('mergeMap', () => {
expectObservable(result).toBe(expected, undefined, noXError);
});
});

it('should report invalid observable inputs via error notifications', () => {
const e1 = hot('--1|');
const e1subs = '^ !';
const expected = '--#';

const result = e1.pipe(mergeMap(() => null as any));

expectObservable(result).toBe(expected, null, new TypeError(
"You provided 'null' where a stream was expected. You can provide an Observable, Promise, Array, or Iterable."
));
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});
});
13 changes: 13 additions & 0 deletions spec/operators/switchMap-spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -444,4 +444,17 @@ describe('switchMap', () => {
expectSubscriptions(x.subscriptions).toBe(xsubs);
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});

it('should report invalid observable inputs via error notifications', () => {
const e1 = hot('--1|');
const e1subs = '^ !';
const expected = '--#';

const result = e1.pipe(switchMap(() => null as any));

expectObservable(result).toBe(expected, null, new TypeError(
"You provided 'null' where a stream was expected. You can provide an Observable, Promise, Array, or Iterable."
));
expectSubscriptions(e1.subscriptions).toBe(e1subs);
});
});

0 comments on commit 50c99a3

Please sign in to comment.