fix: accounts update tests

novasamatech · Jan 13, 2025 · af2794b · af2794b
1 parent 2568fac
commit af2794b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/renderer/domains/network/accounts/model.test.ts b/src/renderer/domains/network/accounts/model.test.ts
@@ -55,7 +55,7 @@ describe('accounts model', () => {
   it('should successfully update account', async () => {
     const scope = fork({
       values: [[accountsDomainModel.__test.$list, accounts]],
-      handlers: [[accountsDomainModel.updateAccounts, () => true]],
+      handlers: [[accountsDomainModel.updateAccount, () => true]],
     });
 
     const draft: AnyAccountDraft = {

diff --git a/src/renderer/domains/network/accounts/model.ts b/src/renderer/domains/network/accounts/model.ts
@@ -75,9 +75,10 @@ sample({
   source: $accounts,
   filter: (_, { result: successful }) => successful,
   fn: (accounts, { params: draft }) => {
+    console.log({ accounts, draft });
     const draftId = accountsService.uniqId(draft);
 
-    return accounts.map(a => (a.id === draftId ? ({ ...a, ...draft } as AnyAccount) : a)) as AnyAccount[];
+    return accounts.map(a => (accountsService.uniqId(a) === draftId ? { ...a, ...draft } : a));
   },
   target: $accounts,
 });
@@ -93,7 +94,9 @@ sample({
       return acc;
     }, {});
 
-    return accounts.map(a => (a.id in draftsMap ? ({ ...a, ...draftsMap[a.id] } as AnyAccount) : a)) as AnyAccount[];
+    return accounts.map(a =>
+      accountsService.uniqId(a) in draftsMap ? { ...a, ...draftsMap[accountsService.uniqId(a)] } : a,
+    );
   },
   target: $accounts,
 });