X-Git-Url: https://git.arvados.org/arvados-workbench2.git/blobdiff_plain/b6a5b173cd4e9f325f371d26204dfe156d911c20..cba00eaa2d205b13cfa696f866a31452c6511cf3:/src/store/auth/auth-reducer.test.ts

diff --git a/src/store/auth/auth-reducer.test.ts b/src/store/auth/auth-reducer.test.ts
index ea08e589..0e05263d 100644
--- a/src/store/auth/auth-reducer.test.ts
+++ b/src/store/auth/auth-reducer.test.ts
@@ -2,67 +2,44 @@
 //
 // SPDX-License-Identifier: AGPL-3.0
 
-import { authReducer } from "./auth-reducer";
-import { authActions } from "./auth-action";
-import {
-    API_TOKEN_KEY,
-    USER_EMAIL_KEY,
-    USER_FIRST_NAME_KEY,
-    USER_LAST_NAME_KEY,
-    USER_OWNER_UUID_KEY,
-    USER_UUID_KEY
-} from "../../services/auth-service/auth-service";
-import { API_HOST } from "../../common/api/server-api";
+import { authReducer, AuthState } from "./auth-reducer";
+import { AuthAction, authActions } from "./auth-action";
 
 import 'jest-localstorage-mock';
+import { createServices } from "../../services/services";
 
 describe('auth-reducer', () => {
+    let reducer: (state: AuthState | undefined, action: AuthAction) => any;
+
     beforeAll(() => {
         localStorage.clear();
+        reducer = authReducer(createServices("/arvados/v1"));
     });
 
-    it('should return default state on initialisation', () => {
-        const initialState = undefined;
-        const state = authReducer(initialState, authActions.INIT());
-        expect(state).toEqual({
-            apiToken: undefined,
-            user: undefined
-        });
-    });
-
-    it('should read user and api token from local storage on init if they are there', () => {
+    it('should correctly initialise state', () => {
         const initialState = undefined;
-
-        localStorage.setItem(API_TOKEN_KEY, "token");
-        localStorage.setItem(USER_EMAIL_KEY, "test@test.com");
-        localStorage.setItem(USER_FIRST_NAME_KEY, "John");
-        localStorage.setItem(USER_LAST_NAME_KEY, "Doe");
-        localStorage.setItem(USER_UUID_KEY, "uuid");
-        localStorage.setItem(USER_OWNER_UUID_KEY, "ownerUuid");
-
-        const state = authReducer(initialState, authActions.INIT());
+        const user = {
+            email: "test@test.com",
+            firstName: "John",
+            lastName: "Doe",
+            uuid: "uuid",
+            ownerUuid: "ownerUuid"
+        };
+        const state = reducer(initialState, authActions.INIT({user, token: "token"}));
         expect(state).toEqual({
             apiToken: "token",
-            user: {
-                email: "test@test.com",
-                firstName: "John",
-                lastName: "Doe",
-                uuid: "uuid",
-                ownerUuid: "ownerUuid"
-            }
+            user
         });
     });
 
-    it('should store token in local storage', () => {
+    it('should save api token', () => {
         const initialState = undefined;
 
-        const state = authReducer(initialState, authActions.SAVE_API_TOKEN("token"));
+        const state = reducer(initialState, authActions.SAVE_API_TOKEN("token"));
         expect(state).toEqual({
             apiToken: "token",
             user: undefined
         });
-
-        expect(localStorage.getItem(API_TOKEN_KEY)).toBe("token");
     });
 
     it('should set user details on success fetch', () => {
@@ -76,7 +53,7 @@ describe('auth-reducer', () => {
             ownerUuid: "ownerUuid"
         };
 
-        const state = authReducer(initialState, authActions.USER_DETAILS_SUCCESS(user));
+        const state = reducer(initialState, authActions.USER_DETAILS_SUCCESS(user));
         expect(state).toEqual({
             apiToken: undefined,
             user: {
@@ -87,25 +64,5 @@ describe('auth-reducer', () => {
                 ownerUuid: "ownerUuid",
             }
         });
-
-        expect(localStorage.getItem(API_TOKEN_KEY)).toBe("token");
-    });
-
-    it('should fire external url to login', () => {
-        const initialState = undefined;
-        window.location.assign = jest.fn();
-        authReducer(initialState, authActions.LOGIN());
-        expect(window.location.assign).toBeCalledWith(
-            `${API_HOST}/login?return_to=${window.location.protocol}//${window.location.host}/token`
-        );
-    });
-
-    it('should fire external url to logout', () => {
-        const initialState = undefined;
-        window.location.assign = jest.fn();
-        authReducer(initialState, authActions.LOGOUT());
-        expect(window.location.assign).toBeCalledWith(
-            `${API_HOST}/logout?return_to=${location.protocol}//${location.host}`
-        );
     });
 });