test: refine tests

also remove some patching around behavior

Signed-off-by: Yuval Datner <[email protected]>

Author: datner

test/hooks/useResult.ts

@@ -17,23 +17,28 @@ const { useResult } = RuntimeProvider.makeFromLayer(Layer.succeed(foo, { value:
 describe("useResult", () => {
   it("should run effects", async () => {
     const testEffect = Effect.succeed(1)
-    const { result } = await waitFor(async () => renderHook(() => useResult(() => testEffect, [])))
-    expect(Result.isSuccess(result.current.result)).toBe(true)
+    const { result } = renderHook(() => useResult(() => testEffect, []))
+    expect(Result.isInitial(result.current.result)).toBe(true)
+    await waitFor(() => expect(Result.isSuccess(result.current.result)).toBe(true))
   })
 
   it("should provide context", async () => {
     const testEffect = Effect.map(foo, (_) => _.value)
-    const { result } = await waitFor(async () => renderHook(() => useResult(() => testEffect, [])))
-    await waitFor(() => expect(Result.isSuccess(result.current.result)).toBe(true))
-    assert(Result.isSuccess(result.current.result))
-    expect(result.current.result.value).toBe(1)
+    const { result } = renderHook(() => useResult(() => testEffect, []))
+    expect(Result.isInitial(result.current.result)).toBe(true)
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current.result))
+      return expect(result.current.result.value).toBe(1)
+    })
   })
 
   it("should run streams", async () => {
     const testStream = Stream.succeed(1)
-    const { result } = await waitFor(async () => renderHook(() => useResult(() => testStream, [])))
-    await waitFor(() => expect(Result.isSuccess(result.current.result)).toBe(true))
-    assert(Result.isSuccess(result.current.result))
-    expect(result.current.result.value).toBe(1)
+    const { result } = renderHook(() => useResult(() => testStream, []))
+    expect(Result.isInitial(result.current.result)).toBe(true)
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current.result))
+      return expect(result.current.result.value).toBe(1)
+    })
   })
 })

test/hooks/useResultCallback.ts

@@ -17,20 +17,48 @@ describe("useResultCallback", () => {
   it("should do good", async () => {
     const testEffect = (value: number) => Effect.succeed(value)
 
-    const { result } = await waitFor(async () => renderHook(() => useResultCallback(testEffect)))
+    const { result } = renderHook(() => useResultCallback(testEffect))
 
-    expect(Result.isInitial(result.current[0].result)).toBe(true)
+    assert.isTrue(Result.isInitial(result.current[0].result))
     act(() => {
       result.current[1](1)
     })
-    await waitFor(() => expect(Result.isSuccess(result.current[0].result)).toBe(true))
-    assert(Result.isSuccess(result.current[0].result))
-    expect(result.current[0].result.value).toBe(1)
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current[0].result))
+      return expect(result.current[0].result.value).toBe(1)
+    })
     act(() => {
       result.current[1](2)
     })
-    await waitFor(() => expect(Result.isSuccess(result.current[0].result)).toBe(true))
-    assert(Result.isSuccess(result.current[0].result))
-    expect(result.current[0].result.value).toBe(2)
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current[0].result))
+      return expect(result.current[0].result.value).toBe(2)
+    })
+  })
+
+  it("should do good async", async () => {
+    const testEffect = (value: number) =>
+      Effect.async<never, never, number>((cb) => {
+        setTimeout(() => cb(Effect.succeed(value)), 100)
+      })
+    const { result } = renderHook(() => useResultCallback(testEffect))
+
+    assert.isTrue(Result.isInitial(result.current[0].result))
+    act(() => {
+      result.current[1](1)
+    })
+    await waitFor(() => assert.isTrue(result.current[0].isLoading))
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current[0].result))
+      return expect(result.current[0].result.value).toBe(1)
+    })
+    act(() => {
+      result.current[1](2)
+    })
+    await waitFor(() => assert.isTrue(result.current[0].isRefreshing))
+    await waitFor(() => {
+      assert(Result.isSuccess(result.current[0].result))
+      return expect(result.current[0].result.value).toBe(2)
+    })
   })
 })