Add check response.ok to default useFetch formatter

Author: ilyalesik

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-fetch-hook",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "React fetch hook",
   "main": "./dist/index.js",
   "scripts": {

src/__tests__/useFetch.test.js

@@ -79,4 +79,23 @@ describe("useFetch", () => {
             expect(fetch.mock.calls[0][1]).toMatchObject({ ...options });
         });
     });
+
+    it("error on throw error", async () => {
+        fetch.mockReject(new Error("fake error message"));
+
+        const Component = () => {
+            const result = useFetch("https://google.com");
+            return (result.error && result.error.message) || "text";
+        };
+
+        const { container, rerender } = render(<Component />);
+
+        await wait(() => {
+            rerender(<Component />);
+
+            expect(fetch.mock.calls.length).toEqual(1);
+            expect(container).toHaveTextContent("fake error message");
+            expect(fetch.mock.calls[0][0]).toEqual("https://google.com");
+        });
+    });
 });

src/useFetch.js

@@ -11,7 +11,12 @@ export function useFetch<T>(
     path: RequestInfo,
     options?: { ...RequestOptions, formatter?: Response => Promise<T> }
 ): TUseFetchResult<T> {
-    const defaultFormatter = response => response.json();
+    const defaultFormatter = response => {
+        if (!response.ok) {
+            throw Error(response.statusText);
+        }
+        return response.json();
+    };
     const fetchInstance = formatter => (path, options) => {
         return fetch(path, options).then((typeof formatter === "function" && formatter) || defaultFormatter);
     };