chore(state): add test cases for rxEffects options

michaelbe812 · michaelbe812 · commit e2fcc4b58015 · 2025-01-09T20:03:22.000+01:00
diff --git a/libs/state/effects/src/lib/rx-effects.spec.ts b/libs/state/effects/src/lib/rx-effects.spec.ts
@@ -1,4 +1,4 @@
-import { Component } from '@angular/core';
+import { Component, EnvironmentInjector } from '@angular/core';
 import { TestBed } from '@angular/core/testing';
 import { jestMatcher } from '@test-helpers/rx-angular';
 import { Observable, of, Subject, tap, timer } from 'rxjs';
@@ -104,6 +104,24 @@ describe(rxEffects, () => {
     expect(spyInternalOnCleanup).toHaveBeenCalled();
     expect(spyOnCleanup).toHaveBeenCalled();
   });
+
+  describe('without injection context', () => {
+    it('should throw when invoked without injection context', () => {
+      const { rxEffects } = setUpWithoutInjectionContext();
+      expect(() => rxEffects(() => {}, {})).toThrow(
+        /rxEffects\(\) can only be used within an injection context/,
+      );
+    });
+
+    it('should be able to use a custom Injector', () => {
+      const { rxEffects } = setUpWithoutInjectionContext();
+
+      const envInjector = TestBed.inject(EnvironmentInjector);
+
+      const effects = rxEffects(() => {}, { injector: envInjector });
+      expect(effects).toBeDefined();
+    });
+  });
 });
 
 function setupComponent(setupFn?: RxEffectsSetupFn) {
@@ -123,3 +141,12 @@ function setupComponent(setupFn?: RxEffectsSetupFn) {
     component: fixture.componentInstance,
   };
 }
+
+function setUpWithoutInjectionContext() {
+  return {
+    rxEffects(...args: Parameters<typeof rxEffects>) {
+      const effects = rxEffects(...args);
+      return effects;
+    },
+  };
+}
