Match propTypes to TS-types

Author: Khartir

packages/react-async/src/helpers.spec.js

@@ -2,7 +2,7 @@ import "@testing-library/jest-dom/extend-expect"
 import React from "react"
 import { render, fireEvent, cleanup } from "@testing-library/react"
 import Async, { IfInitial, IfPending, IfFulfilled, IfRejected, IfSettled } from "./index"
-import { resolveIn, resolveTo, rejectTo } from "./specs"
+import { resolveIn, resolveTo, rejectTo, sleep } from "./specs"
 
 afterEach(cleanup)
 
@@ -84,6 +84,12 @@ describe("IfFulfilled", () => {
     await findByText("outer inner")
     expect(queryByText("outer inner")).toBeInTheDocument()
   })
+
+  test("renders without children", async () => {
+    const promiseFn = () => resolveTo("ok")
+    render(<Async promiseFn={promiseFn}>{state => <IfFulfilled state={state} />}</Async>)
+    await sleep(0)
+  })
 })
 
 describe("IfPending", () => {

packages/react-async/src/helpers.tsx

@@ -33,7 +33,7 @@ const renderFn = (children: React.ReactNode | ChildrenFn, ...args: any[]) => {
     const render = children as ChildrenFn
     return render(...args)
   }
-  return children === undefined ? null : children
+  return children
 }
 
 /**

packages/react-async/src/propTypes.ts

@@ -46,27 +46,27 @@ export default PropTypes && {
     debugLabel: PropTypes.string,
   },
   Initial: {
-    children: childrenFn.isRequired,
+    children: childrenFn,
     state: stateObject.isRequired,
     persist: PropTypes.bool,
   },
   Pending: {
-    children: childrenFn.isRequired,
+    children: childrenFn,
     state: stateObject.isRequired,
     initial: PropTypes.bool,
   },
   Fulfilled: {
-    children: childrenFn.isRequired,
+    children: childrenFn,
     state: stateObject.isRequired,
     persist: PropTypes.bool,
   },
   Rejected: {
-    children: childrenFn.isRequired,
+    children: childrenFn,
     state: stateObject.isRequired,
     persist: PropTypes.bool,
   },
   Settled: {
-    children: childrenFn.isRequired,
+    children: childrenFn,
     state: stateObject.isRequired,
     persist: PropTypes.bool,
   },