Test auth guards (#225)

* router structure changed

* Fix relative paths

* Login route guards moved to separate components

* Tests for loginById added

* loginByToken tests added

* Remove unneeded stubActions option

src/axios/onResponseRejected.test.ts

@@ -27,7 +27,7 @@ describe('custom axios', () => {
 
   beforeEach(() => {
     const app = createApp({});
-    const pinia = createTestingPinia({ createSpy: vi.fn, stubActions: false });
+    const pinia = createTestingPinia({ createSpy: vi.fn });
     app.use(pinia);
     setActivePinia(pinia);
 

src/router.ts → src/router/index.ts

@@ -6,9 +6,9 @@ import {
 import useAuth from '@/stores/auth';
 import useUser from '@/stores/user';
 import useStudies from '@/stores/studies';
-import useMaterials from './stores/materials';
-import useDiplomas from './stores/diplomas';
-import useLoading from './stores/loading';
+import useMaterials from '@/stores/materials';
+import useDiplomas from '@/stores/diplomas';
+import useLoading from '@/stores/loading';
 const VHomeView = () => import('@/views/VHomeView.vue');
 const VMailSentView = () => import('@/views/VMailSentView.vue');
 const VSettingsView = () => import('@/views/VSettingsView.vue');
@@ -21,6 +21,8 @@ const VHomeworkAnswerView = () => import('@/views/VHomeworkAnswerView.vue');
 const VLoginResetView = () => import('@/views/VLoginResetView.vue');
 const VLoginChangeView = () => import('@/views/VLoginChangeView.vue');
 const VCertificatesView = () => import('@/views/VCertificatesView.vue');
+import loginByToken from '@/router/loginByToken';
+import loginById from '@/router/loginById';
 
 const isAuthorized = () => {
   const auth = useAuth();
@@ -90,11 +92,7 @@ export const routes = [
     path: '/auth/passwordless/:passwordlessToken',
     name: 'token',
     component: VLoadingView,
-    beforeEnter: async (to: RouteLocationNormalized) => {
-      const auth = useAuth();
-      await auth.exchangeTokens(String(to.params.passwordlessToken));
-      return { name: 'home' };
-    },
+    beforeEnter: [loginByToken],
     meta: {
       isPublic: true,
     },
@@ -103,12 +101,7 @@ export const routes = [
     path: '/auth/as/:userId',
     name: 'auth-as',
     component: VLoadingView,
-    beforeEnter: async (to: RouteLocationNormalized) => {
-      const auth = useAuth();
-
-      await auth.loginWithUserId(String(to.params.userId));
-      return { name: 'home' };
-    },
+    beforeEnter: [loginById],
   },
   {
     path: '/materials/:id',

src/router/loginById.test.ts

@@ -0,0 +1,42 @@
+import { faker } from '@faker-js/faker';
+import { describe, test, beforeEach, vi, expect } from 'vitest';
+import loginById from './loginById';
+import useAuth from '@/stores/auth';
+import type { RouteLocationNormalized } from 'vue-router';
+import { createApp } from 'vue';
+import { createTestingPinia } from '@pinia/testing';
+import { setActivePinia } from 'pinia';
+
+const userId = faker.datatype.uuid();
+
+const to: Partial<RouteLocationNormalized> = {
+  params: {
+    userId,
+  },
+};
+
+describe('loginById', () => {
+  let auth: ReturnType<typeof useAuth>;
+
+  beforeEach(() => {
+    const app = createApp({});
+    const pinia = createTestingPinia({ createSpy: vi.fn });
+    app.use(pinia);
+    setActivePinia(pinia);
+
+    auth = useAuth();
+  });
+
+  test('should call loginWithUserId', async () => {
+    await loginById(to as RouteLocationNormalized);
+
+    expect(auth.loginWithUserId).toHaveBeenCalled();
+    expect(auth.loginWithUserId).toHaveBeenCalledWith(userId);
+  });
+
+  test('should return directions to home', async () => {
+    expect(await loginById(to as RouteLocationNormalized)).toStrictEqual({
+      name: 'home',
+    });
+  });
+});

src/router/loginById.ts

@@ -0,0 +1,11 @@
+import type { RouteLocationNormalized } from 'vue-router';
+import useAuth from '@/stores/auth';
+
+const loginByToken = async (to: RouteLocationNormalized) => {
+  const auth = useAuth();
+  await auth.loginWithUserId(String(to.params.userId));
+
+  return { name: 'home' };
+};
+
+export default loginByToken;

src/router/loginByToken.test.ts

@@ -0,0 +1,42 @@
+import { faker } from '@faker-js/faker';
+import { describe, test, beforeEach, vi, expect } from 'vitest';
+import loginByToken from './loginByToken';
+import useAuth from '@/stores/auth';
+import type { RouteLocationNormalized } from 'vue-router';
+import { createApp } from 'vue';
+import { createTestingPinia } from '@pinia/testing';
+import { setActivePinia } from 'pinia';
+
+const passwordlessToken = faker.datatype.uuid();
+
+const to: Partial<RouteLocationNormalized> = {
+  params: {
+    passwordlessToken,
+  },
+};
+
+describe('loginByToken', () => {
+  let auth: ReturnType<typeof useAuth>;
+
+  beforeEach(() => {
+    const app = createApp({});
+    const pinia = createTestingPinia({ createSpy: vi.fn });
+    app.use(pinia);
+    setActivePinia(pinia);
+
+    auth = useAuth();
+  });
+
+  test('should call exchangeTokens', async () => {
+    await loginByToken(to as RouteLocationNormalized);
+
+    expect(auth.exchangeTokens).toHaveBeenCalled();
+    expect(auth.exchangeTokens).toHaveBeenCalledWith(passwordlessToken);
+  });
+
+  test('should return directions to home', async () => {
+    expect(await loginByToken(to as RouteLocationNormalized)).toStrictEqual({
+      name: 'home',
+    });
+  });
+});

src/router/loginByToken.ts

@@ -0,0 +1,10 @@
+import type { RouteLocationNormalized } from 'vue-router';
+import useAuth from '@/stores/auth';
+
+const loginByToken = async (to: RouteLocationNormalized) => {
+  const auth = useAuth();
+  await auth.exchangeTokens(String(to.params.passwordlessToken));
+  return { name: 'home' };
+};
+
+export default loginByToken;