fix up earwurm test

beefchimi · Dec 22, 2023 · 89e70fa · 89e70fa
1 parent bebec8c
commit 89e70fa
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/src/tests/Earwurm.test.ts b/src/tests/Earwurm.test.ts
@@ -261,6 +261,7 @@ describe('Earwurm component', () => {
       expect(spyKeysChange).toBeCalledWith(mockInitialKeys, []);
       expect(spyKeysChange).toBeCalledTimes(1);
 
+      // Does not add/remove when both `id + path` are identical.
       mockManager.add(mockEntries[0]);
       expect(spyKeysChange).not.toBeCalledTimes(2);
 
@@ -279,6 +280,19 @@ describe('Earwurm component', () => {
         [...mockInitialKeys, mockUniqueEntry.id],
         mockInitialKeys,
       );
+
+      const keysSnapshot = mockManager.keys;
+
+      // Emits twice as an existing key is removed then re-added
+      // as a result of the `path` value changing.
+      mockManager.add({...mockUniqueEntry, path: 'changed'});
+      expect(spyKeysChange).toBeCalledTimes(4);
+
+      expect(spyKeysChange).toBeCalledWith(mockInitialKeys, keysSnapshot);
+      expect(spyKeysChange).toHaveBeenLastCalledWith(
+        keysSnapshot,
+        mockInitialKeys,
+      );
     });
 
     // TODO: Figure out how best to read `fadeMs` and `request` from Stack.