Fix linter issues

Author: DarkoKukovec

.eslintignore

@@ -0,0 +1,3 @@
+*.js
+*.d.ts
+dist

.eslintrc.json renamed to .eslintrc.js

@@ -1,12 +1,7 @@
-{
+module.exports = {
 	"extends": ["plugin:@infinum/core", "plugin:@infinum/typescript"],
-	"overrides": [
-		{
-			"files": ["*.ts"]
-		}
-	],
 	"parserOptions": {
 		"project": ["./tsconfig.json"]
 	},
 	"parser": "@typescript-eslint/parser"
-}
+};

example/src/App.tsx

@@ -10,20 +10,17 @@ const providerUrls: Record<string, string> = {
 	auth0: 'https://dev-u8csbbr8zashh2k8.us.auth0.com/userinfo',
 };
 
-const useSW = localStorage.getItem('useSW') === 'true';
-
 function App() {
-	const [result, setResult] = useState<null | { data: { name: string; picture: string } }>(null);
+	const [result, setResult] = useState<null | { data: { name: string; picture: string }, provider: string }>(null);
 
 	useEffect(() => {
 		if (!result) {
-			getUserData().then(setResult as any, () => null);
+			getUserData().then(setResult as (data: unknown) => void, () => null);
 		}
 	}, []);
 
 	const getUserInfo = useCallback(async () => {
-		// @ts-ignore
-		const userInfoUrl: string | undefined = providerUrls[result?.provider];
+		const userInfoUrl: string | null = result && providerUrls[result.provider];
 
 		if (userInfoUrl) {
 			await fetch('/test');

package.json

@@ -6,7 +6,7 @@
 	"module": "index.mjs",
 	"scripts": {
 		"build": "tsup --minify && node ./prepare.js",
-		"lint": "eslint -c ./.eslintrc.json .",
+		"lint": "eslint .",
 		"dev": "nodemon -i example -i dist -i node_modules -i test -i 'src/**.test.ts' -e ts,tsx --exec \"npm run build:example\"",
 		"build:example": "npm run build && cd example && rm -rf node_modules/auth-worker && npm install && npm run build:sw -- --clean=false",
 		"test": "jest"

src/utils/register.test.ts

@@ -23,6 +23,7 @@ describe('utils/register', () => {
 				{ workerPath: './test-service-worker.js', scope: '/test', debug: true }
 			);
 			expect(window.navigator.serviceWorker.register).toHaveBeenCalledWith(
+				// eslint-disable-next-line max-len
 				'./test-service-worker.js?config=%7B%22google%22%3A%7B%22clientId%22%3A%22example-client-id%22%2C%22scopes%22%3A%22https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fuserinfo.profile%22%7D%7D&v=1&debug=1',
 				{
 					scope: '/test',
@@ -39,6 +40,7 @@ describe('utils/register', () => {
 				},
 			});
 			expect(window.navigator.serviceWorker.register).toHaveBeenCalledWith(
+				// eslint-disable-next-line max-len
 				'./service-worker.js?config=%7B%22google%22%3A%7B%22clientId%22%3A%22example-client-id%22%2C%22scopes%22%3A%22https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fuserinfo.profile%22%7D%7D&v=1&debug=0',
 				{
 					scope: '/',

src/worker/fetch.test.ts

@@ -63,6 +63,7 @@ describe('worker/fetch', () => {
 
 			(fetch as jest.Mock).mockResolvedValueOnce(
 				new Response(
+					// eslint-disable-next-line max-len
 					'{"access_token": "mockAccessToken", "token_type": "mockTokenType", "refresh_token": "newMockRefreshToken", "expires_in": 1234567890}',
 					{ status: 200 }
 				)
@@ -200,6 +201,7 @@ describe('worker/fetch', () => {
 
 			(fetch as jest.Mock).mockResolvedValueOnce(
 				new Response(
+					// eslint-disable-next-line max-len
 					'{"access_token": "mockAccessToken", "token_type": "mockTokenType", "refresh_token": "newMockRefreshToken", "expires_in": 1234567890}',
 					{ status: 200 }
 				)
@@ -231,6 +233,7 @@ describe('worker/fetch', () => {
 
 			(fetch as jest.Mock).mockResolvedValueOnce(
 				new Response(
+					// eslint-disable-next-line max-len
 					'{"access_token": "mockAccessToken", "token_type": "mockTokenType", "refresh_token": "newMockRefreshToken", "expires_in": 1234567890}',
 					{ status: 200 }
 				)

src/worker/interceptor.test.ts

@@ -62,6 +62,7 @@ describe('worker/interceptor', () => {
 
 			(fetch as jest.Mock).mockResolvedValueOnce(
 				new Response(
+					// eslint-disable-next-line max-len
 					'{"access_token": "mockAccessToken", "token_type": "mockTokenType", "refresh_token": "newMockRefreshToken", "expires_in": 1234567890}',
 					{ status: 200 }
 				)
@@ -112,10 +113,10 @@ describe('worker/interceptor', () => {
 
 			const location = response.headers.get('location');
 
-			expect(location.startsWith('https://example.com/login?client_id=fooClientId&response_type=token&state=')).toBe(
-				true
-			);
-			expect(location.endsWith('&scope=&redirect_uri=https%3A%2F%2Fexample.com%2Ffoobar%2Fcallback%2Ffoo')).toBe(true);
+			expect(location.startsWith('https://example.com/login?client_id=fooClientId&response_type=token&state='))
+				.toBe(true);
+			expect(location.endsWith('&scope=&redirect_uri=https%3A%2F%2Fexample.com%2Ffoobar%2Fcallback%2Ffoo'))
+				.toBe(true);
 			expect(response.status).toBe(302);
 		});
 

src/worker/operations.test.ts

@@ -26,7 +26,8 @@ describe('worker/operations', () => {
 
 			(getAuthState as jest.Mock).mockReturnValue(state);
 
-			await expect(createSession('', 'mockProvider', '', 'http://example.com')).rejects.toThrow('No config found');
+			await expect(createSession('', 'mockProvider', '', 'http://example.com'))
+				.rejects.toThrow('No config found');
 		});
 
 		it('should fail if there is no valid providers', async () => {
@@ -54,7 +55,8 @@ describe('worker/operations', () => {
 
 			(getAuthState as jest.Mock).mockReturnValue(state);
 
-			await expect(createSession('', 'mockProvider', '123', 'http://example.com')).rejects.toThrow('Invalid state');
+			await expect(createSession('', 'mockProvider', '123', 'http://example.com'))
+				.rejects.toThrow('Invalid state');
 		});
 
 		it('should work for token flow', async () => {
@@ -316,6 +318,7 @@ describe('worker/operations', () => {
 
 			(fetch as jest.Mock).mockResolvedValueOnce(
 				new Response(
+					// eslint-disable-next-line max-len
 					'{"access_token": "mockAccess", "user": "mockUserInfo", "expiresIn": 321, "tokenType": "Foo", "refreshToken": "mockRefresh"}',
 					{ status: 200 }
 				)

src/worker/utils.test.ts

@@ -56,7 +56,8 @@ describe('worker/utils', () => {
 
 			await mockState;
 			expect(getAuthState).toHaveBeenCalled();
-			expect(console.log).toHaveBeenCalledWith(expect.stringMatching(/%cWW\/\w{4}/), expect.any(String), 'Test Log');
+			expect(console.log)
+				.toHaveBeenCalledWith(expect.stringMatching(/%cWW\/\w{4}/), expect.any(String), 'Test Log');
 		});
 
 		it('should not log the arguments if debug is disabled in the state', async () => {