🦄 refactor: Simplify type definitions

Author: Snowflyt

src/README.example.spec.ts

@@ -681,7 +681,6 @@ test("Handling return values", () => {
         ],
       ]
     `);
-    logSpy.mockRestore();
   }
 });
 
@@ -779,23 +778,29 @@ test("Handling multiple effects in one handler", () => {
     const logs: unknown[][] = [];
     expect(
       range2(1, 0)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "range", message: "Start must be less than stop" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range2(1.5, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "type", message: "Start and stop must be integers" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range2(1, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(ok([1, 2, 3, 4]));
     expect(logs).toEqual([["Generating range from 1 to 5"]]);
@@ -805,23 +810,29 @@ test("Handling multiple effects in one handler", () => {
 
     expect(
       range3(1, 0)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "range", message: "Start must be less than stop" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range3(1.5, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "type", message: "Start and stop must be integers" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range3(1, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(ok([1, 2, 3, 4]));
     expect(logs).toEqual([["Generating range from 1 to 5"]]);
@@ -834,23 +845,29 @@ test("Handling multiple effects in one handler", () => {
 
     expect(
       range4(1, 0)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "range", message: "Start must be less than stop" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range4(1.5, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(err({ error: "type", message: "Start and stop must be integers" }));
     expect(logs).toEqual([]);
     logs.length = 0;
 
     expect(
       range4(1, 5)
-        .resume("println", (...args) => logs.push(args))
+        .resume("println", (...args) => {
+          logs.push(args);
+        })
         .runSync(),
     ).toEqual(ok([1, 2, 3, 4]));
     expect(logs).toEqual([["Generating range from 1 to 5"]]);

src/backend-app.spec.ts

@@ -128,7 +128,9 @@ test("app", () => {
     }
   })
     .provideBy("currentUser", () => currentUser)
-    .resume("setCurrentUser", (user) => (currentUser = user))
+    .resume("setCurrentUser", (user) => {
+      currentUser = user;
+    })
     .catch("authentication", console.error)
     .catch("unauthorized", console.error)
     .catch("userNotFound", console.error)

src/effected.spec.ts

@@ -628,7 +628,7 @@ describe("Effected#tap", () => {
       Effected.of(42)
         .tap((value) => {
           logs.push(["tap", value]);
-          return value + 1;
+          return (value + 1) as unknown as void;
         })
         .runSync(),
     ).toBe(42);
@@ -643,7 +643,9 @@ describe("Effected#tap", () => {
           yield* log("tap", value);
           return (value + 1) as unknown as void;
         })
-        .resume("log", (...args) => Array.prototype.push.apply(logs, args))
+        .resume("log", (...args) => {
+          Array.prototype.push.apply(logs, args);
+        })
         .runSync(),
     ).toBe(42);
     expect(logs).toEqual(["tap", 42]);
@@ -657,7 +659,9 @@ describe("Effected#tap", () => {
             return (value + 1) as unknown as void;
           }),
         )
-        .resume("log", (...args) => Array.prototype.push.apply(logs, args))
+        .resume("log", (...args) => {
+          Array.prototype.push.apply(logs, args);
+        })
         .runSync(),
     ).toBe(42);
     expect(logs).toEqual(["tap", 42]);