test linting

Author: walker-sean

src/frontend/src/tests/app/AppPublic.test.tsx

@@ -3,9 +3,8 @@
  * See the LICENSE file in the repository root folder for details.
  */
 
-import { AuthenticatedUser, User } from 'shared';
+import { AuthenticatedUser } from 'shared';
 import { render, screen, routerWrapperBuilder } from '../test-support/test-utils';
-import { exampleAdminUser } from '../test-support/test-data/users.stub';
 import { mockAuth } from '../test-support/test-data/test-utils.stub';
 import { useAuth } from '../../hooks/auth.hooks';
 import { routes } from '../../utils/routes';

src/frontend/src/tests/pages/CreateChangeRequestPage/CreateProposedSolutionsList.test.tsx

@@ -10,6 +10,7 @@ import { mockAuth } from '../../test-support/test-data/test-utils.stub';
 import { exampleAdminUser } from '../../test-support/test-data/users.stub';
 import * as userHooks from '../../../hooks/users.hooks';
 import AppContextUser from '../../../app/AppContextUser';
+import { exampleAuthenticatedAdminUser } from '../../test-support/test-data/authenticated-user.stub';
 
 /**
  * Sets up the component under test with the desired values and renders it.
@@ -33,7 +34,7 @@ describe('Proposed Solutions List Test Suite', () => {
       onboardedTeamTypeIds: [],
       onboardingTeamTypeIds: []
     });
-    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAdminUser));
+    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAuthenticatedAdminUser));
   });
 
   it('Renders correctly when empty', () => {

src/frontend/src/tests/pages/HomePage/Home.test.tsx

@@ -11,6 +11,7 @@ import * as userHooks from '../../../hooks/users.hooks';
 import { exampleAdminUser } from '../../test-support/test-data/users.stub';
 import { mockAuth } from '../../test-support/test-data/test-utils.stub';
 import { mockUseSingleUserSettings } from '../../test-support/mock-hooks';
+import { exampleAuthenticatedAdminUser } from '../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../pages/HomePage/components/UsefulLinks', () => {
   return {
@@ -53,7 +54,7 @@ const renderComponent = () => {
 
 describe('home component', () => {
   beforeEach(() => {
-    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAdminUser));
+    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAuthenticatedAdminUser));
     vi.spyOn(userHooks, 'useCurrentUser').mockReturnValue({
       ...exampleAdminUser,
       organizations: [],

src/frontend/src/tests/pages/SettingsPage/Settings.test.tsx

@@ -7,8 +7,8 @@ import { render, screen, routerWrapperBuilder } from '../../test-support/test-ut
 import { routes } from '../../../utils/routes';
 import Settings from '../../../pages/SettingsPage/SettingsPage';
 import * as authHooks from '../../../hooks/auth.hooks';
-import { exampleAdminUser } from '../../test-support/test-data/users.stub';
 import { mockAuth } from '../../test-support/test-data/test-utils.stub';
+import { exampleAuthenticatedAdminUser } from '../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../pages/SettingsPage/UserSettings/UserSettings', () => {
   return {
@@ -32,7 +32,7 @@ const renderComponent = () => {
 };
 
 describe.skip('settings page component', () => {
-  beforeEach(() => vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAdminUser)));
+  beforeEach(() => vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAuthenticatedAdminUser)));
   it('renders all the information', () => {
     renderComponent();
     expect(screen.getAllByText('Settings').length).toEqual(1);

src/frontend/src/tests/pages/WorkPackageDetailPage/ActivateWorkPackageModalContainer/ActivateWorkPackageModalContainer.test.tsx

@@ -10,14 +10,15 @@ import { exampleWbs1 } from '../../../test-support/test-data/wbs-numbers.stub';
 import ActivateWorkPackageModalContainer from '../../../../pages/WorkPackageDetailPage/ActivateWorkPackageModalContainer/ActivateWorkPackageModalContainer';
 import { mockAuth, mockUseMutationResult } from '../../../test-support/test-data/test-utils.stub';
 import { useCreateActivationChangeRequest } from '../../../../hooks/change-requests.hooks';
-import { exampleAdminUser, exampleAllUsers } from '../../../test-support/test-data/users.stub';
+import { exampleAllUsers } from '../../../test-support/test-data/users.stub';
 import * as authHooks from '../../../../hooks/auth.hooks';
 import * as userHooks from '../../../../hooks/users.hooks';
 import {
   mockLogUserInDevReturnValue,
   mockLogUserInReturnValue,
   mockUseAllUsersReturnValue
 } from '../../../test-support/mock-hooks';
+import { exampleAuthenticatedAdminUser } from '../../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../../hooks/change-requests.hooks');
 vi.mock('../../../../hooks/toasts.hooks');
@@ -37,7 +38,7 @@ const renderComponent = () => {
 
 describe('activate work package modal container test suite', () => {
   beforeEach(() => {
-    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAdminUser));
+    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAuthenticatedAdminUser));
     vi.spyOn(userHooks, 'useLogUserIn').mockReturnValue(mockLogUserInReturnValue);
     vi.spyOn(userHooks, 'useLogUserInDev').mockReturnValue(mockLogUserInDevReturnValue);
     vi.spyOn(userHooks, 'useAllUsers').mockReturnValue(mockUseAllUsersReturnValue(exampleAllUsers));

src/frontend/src/tests/pages/WorkPackageDetailPage/WorkPackagePage.test.tsx

@@ -16,6 +16,10 @@ import { exampleAdminUser, exampleGuestUser } from '../../test-support/test-data
 import WorkPackagePage from '../../../pages/WorkPackageDetailPage/WorkPackagePage';
 import AppContextUser from '../../../app/AppContextUser';
 import { useCurrentUser } from '../../../hooks/users.hooks';
+import {
+  exampleAuthenticatedAdminUser,
+  exampleAuthenticatedGuestUser
+} from '../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../hooks/work-packages.hooks');
 
@@ -35,7 +39,7 @@ vi.mock('../../../hooks/auth.hooks');
 
 const mockedUseAuth = useAuth as jest.Mock<Auth>;
 
-const mockAuthHook = (user = exampleAdminUser) => {
+const mockAuthHook = (user = exampleAuthenticatedAdminUser) => {
   mockedUseAuth.mockReturnValue(mockAuth(false, user));
 };
 
@@ -105,7 +109,7 @@ describe('work package container', () => {
 
   it('enables the edit button for non-guest user', () => {
     mockSingleWPHook(false, false, exampleResearchWorkPackage);
-    mockAuthHook(exampleAdminUser);
+    mockAuthHook(exampleAuthenticatedAdminUser);
     mockCurrentUserHook();
     mockGetBlockingWorkPackagesHook(false, false, [exampleDesignWorkPackage]);
     renderComponent();
@@ -116,7 +120,7 @@ describe('work package container', () => {
 
   it('disables the edit button for guest user', () => {
     mockSingleWPHook(false, false, exampleResearchWorkPackage);
-    mockAuthHook(exampleGuestUser);
+    mockAuthHook(exampleAuthenticatedGuestUser);
     mockCurrentUserHook(exampleGuestUser);
     mockGetBlockingWorkPackagesHook(false, false, [exampleDesignWorkPackage]);
     renderComponent();

src/frontend/src/tests/test-support/test-data/test-utils.stub.ts

@@ -5,7 +5,7 @@
 
 import { AxiosHeaders, AxiosResponse } from 'axios';
 import { UseMutationResult, UseQueryResult } from 'react-query';
-import { AuthenticatedUser, User } from 'shared';
+import { AuthenticatedUser } from 'shared';
 import { exampleAuthenticatedAdminUser } from './authenticated-user.stub';
 import { Auth } from '../../../utils/types';