fix individual useManagedStorageState tests, no including test intefe…

…rence
pixiebrix · Oct 10, 2024 · 2dfd26d · 2dfd26d
1 parent 083ee39
commit 2dfd26d
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/src/store/enterprise/useManagedStorageState.test.ts b/src/store/enterprise/useManagedStorageState.test.ts
@@ -44,30 +44,28 @@ describe("useManagedStorageState", () => {
   });
 
   it("handles already initialized state", async () => {
-    await browser.storage.managed.set({ partnerId: "foo" });
+    const expectedPolicy = { partnerId: "foo" };
+    await browser.storage.managed.set(expectedPolicy);
 
     const { result, waitForNextUpdate } = renderHook(() =>
       useManagedStorageState(),
     );
 
     await waitForNextUpdate();
 
-    expect(result.current).toStrictEqual(
-      valueToAsyncState({ partnerId: "foo" }),
-    );
+    expect(result.current).toStrictEqual(valueToAsyncState(expectedPolicy));
   });
 
-  // Can't test because mock doesn't fire change events: https://github.com/clarkbw/jest-webextension-mock/issues/170
-  it.skip("listens for changes", async () => {
+  it("listens for changes", async () => {
     const { result, waitForNextUpdate } = renderHook(() =>
       useManagedStorageState(),
     );
     expect(result.current.data).toBeUndefined();
-    await browser.storage.managed.set({ partnerId: "taco-bell" });
+
+    const expectedPolicy = { partnerId: "foo" };
+    await browser.storage.managed.set(expectedPolicy);
+
     await waitForNextUpdate();
-    expect(result.current).toStrictEqual({
-      data: { partnerId: "taco-bell" },
-      isLoading: false,
-    });
+    expect(result.current).toStrictEqual(valueToAsyncState(expectedPolicy));
   });
 });