feat: return this in expectErrors()

fix #46
hirezio · Feb 18, 2022 · be89804 · be89804
1 parent 7ffe97f
commit be89804
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/observer-spy.spec.ts b/src/observer-spy.spec.ts
@@ -147,6 +147,15 @@ describe('ObserverSpy', () => {
       expect(observerSpy.receivedError()).toBe(true);
     });
 
+    it('should know whether it got an "error" notification if "expectErrors" was called', () => {
+      const observerSpy: ObserverSpy<string> = new ObserverSpy<string>().expectErrors();
+      const { throwingObservable } = getThrowingObservable();
+
+      throwingObservable.subscribe(observerSpy).unsubscribe();
+
+      expect(observerSpy.receivedError()).toBe(true);
+    });
+
     it('should return the error object it received if "expectErrors" is configured', () => {
       const observerSpy: ObserverSpy<string> = new ObserverSpy({ expectErrors: true });
       const { throwingObservable } = getThrowingObservable();

diff --git a/src/observer-spy.ts b/src/observer-spy.ts
@@ -82,8 +82,9 @@ export class ObserverSpy<T> implements Observer<T> {
     });
   }
 
-  expectErrors() {
+  expectErrors(): this {
     this.state.errorIsExpected = true;
+    return this;
   }
 
   getValuesLength(): number {