Added tests for Service API

.eslintrc.json

@@ -26,7 +26,10 @@
     "react-hooks"
   ],
   "rules": {
-    "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
+    "@typescript-eslint/consistent-type-definitions": [
+      "error",
+      "interface"
+    ],
     "@typescript-eslint/member-delimiter-style": "error",
     "@typescript-eslint/no-explicit-any": "off",
     "@typescript-eslint/no-non-null-assertion": "off",
@@ -58,12 +61,20 @@
       }
     ],
     "react/jsx-props-no-spreading": "off",
+    "react/state-in-constructor": [
+      "error",
+      "never"
+    ],
     "semi": "off"
   },
   "settings": {
     "import/resolver": {
       "node": {
-        "extensions": [".js", ".ts", ".tsx"]
+        "extensions": [
+          ".js",
+          ".ts",
+          ".tsx"
+        ]
       }
     }
   }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "suspense-service",
-  "version": "0.0.5-rc",
+  "version": "0.1.0",
   "description": "Suspense integration library for React",
   "repository": "github:patrickroberts/suspense-service",
   "main": "dst/cjs/suspense-service.js",

src/Service/PromiseState.ts

@@ -1,6 +1,6 @@
 /** @ignore */
-interface PromiseStatePending {
-  promise: Promise<void>;
+interface PromiseStatePending<TResponse> {
+  promise: Promise<TResponse>;
   status: 'pending';
 }
 
@@ -18,7 +18,7 @@ interface PromiseStateRejected {
 
 /** @ignore */
 type PromiseState<TResponse> =
-  PromiseStatePending |
+  PromiseStatePending<TResponse> |
   PromiseStateFulfilled<TResponse> |
   PromiseStateRejected;
 

src/Service/useThenable.ts

@@ -18,13 +18,17 @@ export default function useThenable<TResponse>(
         if (state.status === 'pending' && state.promise === promise) {
           ref.current = { value, status: 'fulfilled' };
         }
+
+        return value;
       },
       (reason) => {
         const state = ref.current!;
 
         if (state.status === 'pending' && state.promise === promise) {
           ref.current = { reason, status: 'rejected' };
         }
+
+        throw reason;
       },
     );
 

src/__fixtures__/ErrorBoundary.ts

@@ -0,0 +1,30 @@
+import { ReactNode, Component } from 'react';
+
+interface ErrorBoundaryProps {
+  fallback: ReactNode;
+  onError: (error: Error) => void;
+}
+
+interface ErrorBoundaryState {
+  hasError: boolean;
+}
+
+export default class ErrorBoundary extends Component<ErrorBoundaryProps> {
+  state: ErrorBoundaryState = { hasError: false };
+
+  static getDerivedStateFromError(): ErrorBoundaryState {
+    return { hasError: true };
+  }
+
+  componentDidCatch(error: Error): void {
+    const { onError } = this.props;
+    onError(error);
+  }
+
+  render(): ReactNode {
+    const { fallback, children } = this.props;
+    const { hasError } = this.state;
+
+    return hasError ? fallback : children;
+  }
+}

src/__fixtures__/mockRender.tsx

@@ -0,0 +1,9 @@
+import React, { ReactNode } from 'react';
+
+const mockRender = jest.fn((value: ReactNode) => (
+  <span>
+    {value}
+  </span>
+));
+
+export default mockRender;

src/__fixtures__/withContext.tsx

@@ -0,0 +1,13 @@
+import React, { ComponentType, ReactNode } from 'react';
+import Id from '../Context/Id';
+import Context, { useContext } from '../Context';
+
+const withContext: (
+  context: Context<ReactNode>
+) => ComponentType<{ id?: Id }> = (context) => ({ id = null }) => (
+  <div>
+    {useContext(context, id)}
+  </div>
+);
+
+export default withContext;

src/__fixtures__/withService.tsx

@@ -0,0 +1,13 @@
+import React, { ComponentType, ReactNode } from 'react';
+import Id from '../Context/Id';
+import Service, { useService } from '../Service';
+
+const withService: (
+  service: Service<any, ReactNode>
+) => ComponentType<{ id?: Id }> = (service) => ({ id = null }) => (
+  <div>
+    {useService(service, id)}
+  </div>
+);
+
+export default withService;