Derived stores can't point to single values

onsetsoftware · Feb 7, 2024 · 29f3370 · 29f3370
1 parent 12719e9
commit 29f3370
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 26 deletions.
diff --git a/src/automerge-derived-store.ts b/src/automerge-derived-store.ts
@@ -9,7 +9,7 @@ import {
 } from "svelte/store";
 import type { AutomergeSvelteStoreInterface } from "./automerge-svelte-store.type";
 
-export class AutomergeDerivedStore<T, U, V extends Stores = []>
+export class AutomergeDerivedStore<T extends object, U, V extends Stores = []>
   implements Readable<T>, AutomergeSvelteStoreInterface<T>
 {
   #rootStore: AutomergeSvelteStoreInterface<U>;

diff --git a/tests/derived-store.test.ts b/tests/derived-store.test.ts
@@ -100,31 +100,6 @@ describe("root store", () => {
     });
   });
 
-  test("single value derived stores can be changed", () => {
-    new Promise<void>((done) => {
-      let initialRun = true;
-
-      const singleValueStore = new AutomergeDerivedStore(
-        rootStore,
-        (doc) => doc.object?.hello,
-      );
-      singleValueStore.subscribe((hello) => {
-        if (initialRun) {
-          expect(hello).toEqual("world");
-          initialRun = false;
-          return;
-        }
-
-        expect(hello).toEqual("there world");
-        done();
-      });
-
-      singleValueStore.localChange(() => {
-        return "there world";
-      });
-    });
-  });
-
   test("a derived store can switch based on a single other stores", () => {
     const otherStore = writable("id-1");
     const derivedStore = new AutomergeDerivedStore(