fix(type): fix curried create return type (#58)

unadlib · Sep 17, 2024 · a38f0dc · a38f0dc
1 parent df531a5
commit a38f0dc
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 11 deletions.
diff --git a/src/makeCreator.ts b/src/makeCreator.ts
@@ -57,7 +57,7 @@ type MakeCreator = <
   <T extends any, O extends PatchesOptions = _O, F extends boolean = _F>(
     base: T,
     options?: ExternalOptions<O, F>
-  ): [T, () => Result<T, O, F>];
+  ): [Draft<T>, () => Result<T, O, F>];
 };
 
 /**

diff --git a/test/apply.test.ts b/test/apply.test.ts
@@ -1305,16 +1305,6 @@ test('merge multiple patches', () => {
       enablePatches: true,
     }
   );
-  console.log(
-    JSON.stringify(
-      {
-        patches,
-        inversePatches,
-      },
-      null,
-      2
-    )
-  );
   const [state1, patches1, inversePatches1] = create(
     state,
     (draft) => {

diff --git a/test/index.test.ts b/test/index.test.ts
@@ -3977,3 +3977,18 @@ it('#51-2 nested drafts work after current', () => {
     k2: { abc: 100, def: 200 },
   });
 });
+
+test('#57 - curried create returns object of wrong type', () => {
+  const state: {
+    readonly a: {
+      readonly x: string;
+    };
+  } = { a: { x: 'x' } };
+
+  const [draft, finalize] = create(state);
+
+  draft.a.x = 'test';
+
+  const nextState = finalize();
+  expect(nextState).toEqual({ a: { x: 'test' } });
+});