test(state): add more rx effects tests

hoebbelsB · hoebbelsB · commit 90fdaf3de474 · 2023-09-25T13:07:58.000+02:00
diff --git a/libs/state/effects/src/lib/rx-effects.spec.ts b/libs/state/effects/src/lib/rx-effects.spec.ts
@@ -1,6 +1,6 @@
 import { Component } from '@angular/core';
 import { TestBed } from '@angular/core/testing';
-import { of, Subject, tap } from 'rxjs';
+import { Observable, of, Subject, tap } from 'rxjs';
 import { rxEffects, RxEffectsSetupFn } from './rx-effects';
 
 describe(rxEffects, () => {
@@ -16,6 +16,20 @@ describe(rxEffects, () => {
     expect(spy).toHaveBeenCalledWith('src');
   });
 
+  it('should continuously run sideEffect', () => {
+    const spyNext = jest.fn();
+    const trigger = new Subject();
+    setupComponent(({ register }) => {
+      register(trigger, spyNext);
+    });
+
+    trigger.next(1);
+    expect(spyNext).toHaveBeenCalledWith(1);
+    trigger.next(2);
+    expect(spyNext).toHaveBeenCalledWith(2);
+    expect(spyNext).toHaveBeenCalledTimes(2);
+  });
+
   it('should register multiple observables', () => {
     const spy = jest.fn();
     setupComponent(({ register }) => {
@@ -43,6 +57,20 @@ describe(rxEffects, () => {
     expect(spyError).toHaveBeenCalledTimes(1);
   });
 
+  it('should unsubscribe onDestroy', () => {
+    const spyInternalOnCleanup = jest.fn();
+
+    const { fixture } = setupComponent(({ register }) => {
+      register(
+        new Observable(() => {
+          return spyInternalOnCleanup;
+        })
+      );
+    });
+    fixture.destroy();
+    expect(spyInternalOnCleanup).toHaveBeenCalled();
+  });
+
   it('should call onDestroy', () => {
     const spySideEffect = jest.fn();
     const spyInternalOnCleanup = jest.fn();
