Revert updateLastUsedIdentity

Author: Llorenç

src/frontend/src/lib/stores/last-used-identities.store.test.ts

@@ -146,100 +146,6 @@ describe("lastUsedIdentitiesStore", () => {
     lastUsedIdentitiesStore.reset();
     expect(get(lastUsedIdentitiesStore).identities).toEqual({});
   });
-
-  describe("updateLastUsedIdentity", () => {
-    const newName = "Updated Name";
-    const newCredId = strToUint8Array("new-cred-id");
-    const newAuthMethod = { passkey: { credentialId: newCredId } };
-    const newCreatedAtMillis = 1695000000000;
-
-    beforeEach(() => {
-      // Add an identity to be updated
-      lastUsedIdentitiesStore.addLastUsedIdentity({
-        identityNumber: identity1,
-        name: name1,
-        authMethod: { passkey: { credentialId: credId1 } },
-      });
-    });
-
-    it("should not update if the identity does not exist", () => {
-      const nonExistentIdentity = BigInt("999");
-      const originalState = get(lastUsedIdentitiesStore).identities;
-
-      lastUsedIdentitiesStore.updateLastUsedIdentity(nonExistentIdentity, {
-        name: "A New Name",
-      });
-
-      expect(get(lastUsedIdentitiesStore).identities).toEqual(originalState);
-    });
-
-    it("should update only the name of an existing identity", () => {
-      lastUsedIdentitiesStore.updateLastUsedIdentity(identity1, {
-        name: newName,
-      });
-
-      const updatedIdentity = get(lastUsedIdentitiesStore).identities[
-        identity1.toString()
-      ];
-      expect(updatedIdentity.name).toBe(newName);
-      expect(updatedIdentity.authMethod).toEqual({
-        passkey: { credentialId: credId1 },
-      });
-      expect(updatedIdentity.lastUsedTimestampMillis).toBe(mockTimestamp1);
-    });
-
-    it("should update only the authMethod of an existing identity", () => {
-      lastUsedIdentitiesStore.updateLastUsedIdentity(identity1, {
-        authMethod: newAuthMethod,
-      });
-
-      const updatedIdentity = get(lastUsedIdentitiesStore).identities[
-        identity1.toString()
-      ];
-      expect(updatedIdentity.name).toBe(name1);
-      expect(updatedIdentity.authMethod).toEqual(newAuthMethod);
-      expect(updatedIdentity.lastUsedTimestampMillis).toBe(mockTimestamp1);
-    });
-
-    it("should update only the createdAtMillis of an existing identity", () => {
-      lastUsedIdentitiesStore.updateLastUsedIdentity(identity1, {
-        createdAtMillis: newCreatedAtMillis,
-      });
-
-      const updatedIdentity = get(lastUsedIdentitiesStore).identities[
-        identity1.toString()
-      ];
-      expect(updatedIdentity.createdAtMillis).toBe(newCreatedAtMillis);
-      expect(updatedIdentity.name).toBe(name1);
-      expect(updatedIdentity.lastUsedTimestampMillis).toBe(mockTimestamp1);
-    });
-
-    it("should update multiple properties at once", () => {
-      lastUsedIdentitiesStore.updateLastUsedIdentity(identity1, {
-        name: newName,
-        authMethod: newAuthMethod,
-      });
-
-      const updatedIdentity = get(lastUsedIdentitiesStore).identities[
-        identity1.toString()
-      ];
-      expect(updatedIdentity.name).toBe(newName);
-      expect(updatedIdentity.authMethod).toEqual(newAuthMethod);
-      expect(updatedIdentity.lastUsedTimestampMillis).toBe(mockTimestamp1);
-    });
-
-    it("should not update the lastUsedTimestampMillis", () => {
-      vi.setSystemTime(mockTimestamp2);
-      lastUsedIdentitiesStore.updateLastUsedIdentity(identity1, {
-        name: newName,
-      });
-
-      const updatedIdentity = get(lastUsedIdentitiesStore).identities[
-        identity1.toString()
-      ];
-      expect(updatedIdentity.lastUsedTimestampMillis).toBe(mockTimestamp1);
-    });
-  });
 });
 
 describe("lastUsedIdentityStore (derived store)", () => {

src/frontend/src/lib/stores/last-used-identities.store.ts

@@ -49,12 +49,6 @@ type LastUsedIdentitiesStore = Readable<{
       "identityNumber" | "name" | "authMethod" | "createdAtMillis"
     >,
   ) => void;
-  updateLastUsedIdentity: (
-    identityNumber: bigint,
-    params: Partial<
-      Pick<LastUsedIdentity, "name" | "authMethod" | "createdAtMillis">
-    >,
-  ) => void;
   addLastUsedAccount: (
     params: Omit<LastUsedAccount, "lastUsedTimestampMillis">,
   ) => void;
@@ -106,19 +100,6 @@ export const initLastUsedIdentitiesStore = (): LastUsedIdentitiesStore => {
         return lastUsedIdentities;
       });
     },
-    updateLastUsedIdentity: (identityNumber, params) => {
-      lastUsedStore.update((lastUsedIdentities) => {
-        const identity = lastUsedIdentities[identityNumber.toString()];
-        if (isNullish(identity)) {
-          return lastUsedIdentities;
-        }
-        lastUsedIdentities[identityNumber.toString()] = {
-          ...identity,
-          ...params,
-        };
-        return lastUsedIdentities;
-      });
-    },
     addLastUsedAccount: (params) => {
       lastUsedStore.update((lastUsedIdentities) => {
         const identity = lastUsedIdentities[params.identityNumber.toString()];