feat: add excluding urls and cleanup

Author: calebdeji

tests/index.test.ts renamed to __tests__/index.test.ts

@@ -3,11 +3,12 @@ import useNetworkStatusCode from 'use-network-status-code';
 import axios from 'axios';
 
 const App = () => {
-	const { networkStatusCode, clearStatus } = useNetworkStatusCode({
-		urls: [
+	const { networkStatusCode } = useNetworkStatusCode({
+		baseURLs: [
 			'https://ghibliapi.herokuapp.com/film',
 			'https://ghibliapi.herokuapp.com/filmsd'
-		]
+		],
+		excludingURLs: ['https://ghibliapi.herokuapp.com/login']
 	});
 
 	useEffect(() => {
@@ -21,11 +22,15 @@ const App = () => {
 
 		const id = setTimeout(async () => {
 			try {
-				await axios.get('https://ghibliapi.herokuapp.com/filmsd');
+				await axios.get('https://ghibliapi.herokuapp.com/login');
 			} catch (error) {
 				console.log('error ');
 			}
 		}, 4000);
+
+		return () => {
+			clearTimeout(id);
+		};
 	}, []);
 
 	return <div>See this {JSON.stringify(networkStatusCode)} </div>;

tests/useNetworkTest.ts renamed to __tests__/useNetworkTest.ts

@@ -1,4 +1,5 @@
 module.exports = {
-	collectCoverage: true,
-	collectCoverageFrom: ['**/*.{js,jsx}', '!**/node_modules/**', '!**/vendor/**']
+	moduleFileExtensions: ['ts', 'js'],
+	testRegex: '/__tests__/.*\\.(ts|js)$',
+	transformIgnorePatterns: ['<rootDir>/node_modules/']
 };

example/src/App.js

@@ -17,7 +17,7 @@
 		"npm": ">=5"
 	},
 	"scripts": {
-		"test": "react-scripts test",
+		"test": "cross-env CI=1 react-scripts test --env=jsdom",
 		"test:watch": "react-scripts test --env=jsdom",
 		"build": "rollup -c && tsc -d --emitDeclarationOnly --noEmit false --declarationDir dist",
 		"start": "rollup -c -w",

jest.config.js

@@ -4,7 +4,8 @@ import interceptNetwork from './interceptNetwork';
 
 declare namespace useNetworkStatusCodeNamespace {
 	export type Props = {
-		urls: Array<string>;
+		baseURLs: Array<string>;
+		excludingURLs?: Array<string>;
 	};
 
 	export type response = {
@@ -21,13 +22,16 @@ const convertArrayOfURLsToObject = (urls: Array<string>) => {
 
 const useNetworkStatusCode = (props: useNetworkStatusCodeNamespace.Props) => {
 	const [networkStatusCode, setNetworkStatusCode] = React.useState(() => {
-		return convertArrayOfURLsToObject(props.urls);
+		return convertArrayOfURLsToObject(props.baseURLs);
 	});
 
 	const handleStatusAndUrl = (url: string, statusCode: number) => {
-		const isValidURL = props.urls.some((validURL) => {
+		const isValidURL = props.baseURLs.some((validURL) => {
 			const validURLLength = validURL.length;
-			return url.substr(0, validURLLength) === validURL;
+			return (
+				url.substr(0, validURLLength) === validURL &&
+				!props?.excludingURLs?.includes(url)
+			);
 		});
 
 		if (isValidURL) {
@@ -49,7 +53,7 @@ const useNetworkStatusCode = (props: useNetworkStatusCodeNamespace.Props) => {
 			});
 		} else {
 			setNetworkStatusCode(() => {
-				return convertArrayOfURLsToObject(props.urls);
+				return convertArrayOfURLsToObject(props.baseURLs);
 			});
 		}
 	};

package.json

@@ -1,28 +1,41 @@
 {
-	"compilerOptions": {
-		"module": "esnext",
-		"target": "es5",
-		"lib": ["es6", "dom", "es2016", "es2017"],
-		"sourceMap": true,
-		"allowJs": false,
-		"jsx": "preserve",
-		"moduleResolution": "node",
-		"forceConsistentCasingInFileNames": true,
-		"noImplicitReturns": true,
-		"noImplicitThis": true,
-		"noImplicitAny": true,
-		"strictNullChecks": true,
-		"suppressImplicitAnyIndexErrors": true,
-		"noUnusedLocals": true,
-		"noUnusedParameters": true,
-		"skipLibCheck": true,
-		"esModuleInterop": true,
-		"allowSyntheticDefaultImports": true,
-		"strict": true,
-		"resolveJsonModule": true,
-		"isolatedModules": true,
-		"noEmit": true
-	},
-	"include": ["src", "tests"],
-	"exclude": ["node_modules", "build", "dist", "example", "rollup.config.js"]
+  "compilerOptions": {
+    "module": "esnext",
+    "target": "es5",
+    "lib": [
+      "es6",
+      "dom",
+      "es2016",
+      "es2017"
+    ],
+    "sourceMap": true,
+    "allowJs": false,
+    "jsx": "preserve",
+    "moduleResolution": "node",
+    "forceConsistentCasingInFileNames": true,
+    "noImplicitReturns": true,
+    "noImplicitThis": true,
+    "noImplicitAny": true,
+    "strictNullChecks": true,
+    "suppressImplicitAnyIndexErrors": true,
+    "noUnusedLocals": true,
+    "noUnusedParameters": true,
+    "skipLibCheck": true,
+    "esModuleInterop": true,
+    "allowSyntheticDefaultImports": true,
+    "strict": true,
+    "resolveJsonModule": true,
+    "isolatedModules": true,
+    "noEmit": true
+  },
+  "exclude": [
+    "node_modules",
+    "build",
+    "dist",
+    "example",
+    "rollup.config.js"
+  ],
+  "include": [
+    "src"
+  ]
 }