fix(redux): don't override methods of innerStore (#132)

As discussed in #128

Author: rainerhahnekamp

libs/ngrx-toolkit/src/lib/with-redux.spec.ts

@@ -1,4 +1,4 @@
-import { patchState, signalStore, withState } from '@ngrx/signals';
+import { patchState, signalStore, withMethods, withState } from '@ngrx/signals';
 import { inject } from '@angular/core';
 import {
   HttpClient,
@@ -228,4 +228,31 @@ describe('with redux', () => {
     expect(flightStore.effect1()).toBe(true);
     expect(flightStore.effect2()).toBe(true);
   });
+
+  it('should not override methods defined before', () => {
+    const FlightsStore = signalStore(
+      withMethods(() => ({
+        sayHi() {
+          return 'hi';
+        },
+      })),
+      withRedux({
+        actions: {
+          init: noPayload,
+        },
+        reducer() {
+          return {};
+        },
+        effects() {
+          return {};
+        },
+      })
+    );
+
+    const flightStore = TestBed.configureTestingModule({
+      providers: [FlightsStore],
+    }).inject(FlightsStore);
+
+    expect(flightStore.sayHi()).toBe('hi');
+  });
 });

libs/ngrx-toolkit/src/lib/with-redux.ts

@@ -379,7 +379,7 @@ export function withRedux<
     );
     return {
       ...store,
-      methods,
+      methods: { ...store.methods, ...methods },
     };
   };
 }