frontend tests

Author: walker-sean

src/frontend/src/pages/HomePage/components/OverdueWorkPackageCard.tsx

@@ -39,7 +39,6 @@ export const CircularProgressWithLabel = (props: CircularProgressProps & { value
 
 const OverdueWorkPackageCard = ({ wp }: { wp: WorkPackage }) => {
   const theme = useTheme();
-  console.log('wp', wp);
   const { data: blockedByWps, isLoading } = useGetManyWorkPackages(wp.blockedBy);
   const numDaysOverdue = daysOverdue(new Date(wp.endDate));
   if (isLoading || !blockedByWps) return <LoadingIndicator />;

src/frontend/src/pages/HomePage/components/OverdueWorkPackages.tsx

@@ -12,8 +12,6 @@ const OverdueWorkPackages: React.FC<{}> = () => {
   if (isLoading || !overdueWPs) return <LoadingIndicator />;
   if (isError) return <ErrorPage error={error} message={error.message} />;
 
-  console.log('Overdue WPs:', overdueWPs);
-
   return <OverdueWorkPackagesView theme={theme} workPackages={overdueWPs} />;
 };
 

src/frontend/src/pages/RetrospectivePage/Retrospective.tsx

@@ -29,8 +29,6 @@ const RetrospectivePage = () => {
   const history = useHistory();
   const { filters, setFilters } = useGanttFilters('retro-gantt');
 
-  console.log(filters.startDate);
-
   const { data: teams, isLoading: teamsIsLoading, isError: teamsIsError, error: teamsError } = useAllTeams();
 
   const {

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

@@ -13,6 +13,7 @@ import * as userHooks from '../../hooks/users.hooks';
 import { mockUseAllWorkPackagesReturnValue } from '../test-support/mock-hooks';
 import { mockAuth } from '../test-support/test-data/test-utils.stub';
 import { exampleAdminUser } from '../test-support/test-data/users.stub';
+import { exampleAuthenticatedAdminUser } from '../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../pages/ProjectsPage/Projects', () => {
   return {
@@ -39,7 +40,7 @@ describe.skip('App Authenticated', () => {
   beforeEach(() => {
     vi.spyOn(workPackageHooks, 'useAllWorkPackages').mockReturnValue(mockUseAllWorkPackagesReturnValue([]));
     vi.spyOn(miscHooks, 'useGetVersionNumber').mockReturnValue(mockGetVersionNumberReturnValue({ tag_name: 'v3.5.4' }));
-    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/app/AppPublic.test.tsx

@@ -3,14 +3,15 @@
  * See the LICENSE file in the repository root folder for details.
  */
 
-import { User } from 'shared';
+import { AuthenticatedUser, User } 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';
 import { Auth } from '../../utils/types';
 import AppPublic from '../../app/AppPublic';
+import { exampleAuthenticatedAdminUser } from '../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../app/AppAuthenticated', () => {
   return {
@@ -25,7 +26,7 @@ vi.mock('../../hooks/auth.hooks');
 
 const mockedUseAuth = useAuth as jest.Mock<Auth>;
 
-const mockHook = (isLoading: boolean, user?: User) => {
+const mockHook = (isLoading: boolean, user?: AuthenticatedUser) => {
   mockedUseAuth.mockReturnValue(mockAuth(isLoading, user));
 };
 
@@ -41,14 +42,14 @@ const renderComponent = (path?: string, route?: string) => {
 
 describe('app public section', () => {
   it('renders loading spinner', () => {
-    mockHook(true, exampleAdminUser);
+    mockHook(true, exampleAuthenticatedAdminUser);
     renderComponent(routes.LOGIN, routes.LOGIN);
 
     expect(screen.getByTestId('loader')).toBeInTheDocument();
   });
 
   it('renders app authenticated', () => {
-    mockHook(false, exampleAdminUser);
+    mockHook(false, exampleAuthenticatedAdminUser);
     renderComponent(routes.PROJECTS, routes.PROJECTS);
 
     expect(screen.getByText('app-authenticated')).toBeInTheDocument();

src/frontend/src/tests/pages/ProjectDetailPage/ProjectViewContainer.test.tsx

@@ -6,21 +6,26 @@
 import { render, screen, routerWrapperBuilder, fireEvent, waitFor } from '../../test-support/test-utils';
 import { exampleProject1 } from '../../test-support/test-data/projects.stub';
 import { mockAuth } from '../../test-support/test-data/test-utils.stub';
-import { exampleAdminUser, exampleGuestUser } from '../../test-support/test-data/users.stub';
 import ProjectViewContainer from '../../../pages/ProjectDetailPage/ProjectViewContainer/ProjectViewContainer';
 import { WorkPackageStage } from 'shared/src/types/work-package-types';
 import * as userHooks from '../../../hooks/users.hooks';
 import * as financeHooks from '../../../hooks/finance.hooks';
 import * as authHooks from '../../../hooks/auth.hooks';
 import * as wpHooks from '../../../hooks/work-packages.hooks';
 import * as bomHooks from '../../../hooks/bom.hooks';
+import * as teamHooks from '../../../hooks/teams.hooks';
 import {
   mockManyMaterials,
   mockManyWorkPackages,
-  mockUseGetReimbursementRequestProjectData
+  mockUseGetReimbursementRequestProjectData,
+  mockUseMyTeamAsHead
 } from '../../test-support/mock-hooks';
 import { exampleAllWorkPackages } from '../../test-support/test-data/work-packages.stub';
 import { exampleRRData } from '../../test-support/test-data/finance.stubs';
+import {
+  exampleAuthenticatedAdminUser,
+  exampleAuthenticatedGuestUser
+} from '../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../utils/axios');
 vi.mock('../../../hooks/toasts.hooks');
@@ -54,18 +59,16 @@ describe('Rendering Project View Container', () => {
       unobserve: vi.fn(),
       disconnect: vi.fn()
     }));
-    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAdminUser));
+    vi.spyOn(authHooks, 'useAuth').mockReturnValue(mockAuth(false, exampleAuthenticatedAdminUser));
     vi.spyOn(userHooks, 'useCurrentUser').mockReturnValue({
-      ...exampleAdminUser,
-      organizations: [],
-      onboardedTeamTypeIds: [],
-      onboardingTeamTypeIds: []
+      ...exampleAuthenticatedAdminUser
     });
     vi.spyOn(wpHooks, 'useGetManyWorkPackages').mockReturnValue(mockManyWorkPackages(exampleAllWorkPackages));
     vi.spyOn(bomHooks, 'useGetMaterialsForWbsElement').mockReturnValue(mockManyMaterials([]));
     vi.spyOn(financeHooks, 'useGetReimbursementRequestProjectData').mockReturnValue(
       mockUseGetReimbursementRequestProjectData(exampleRRData)
     );
+    vi.spyOn(teamHooks, 'useMyTeamAsHead').mockReturnValue(mockUseMyTeamAsHead());
     vi.spyOn(userHooks, 'useUsersFavoriteProjects').mockReturnValue({
       data: [
         {
@@ -92,10 +95,7 @@ describe('Rendering Project View Container', () => {
   it('disables the buttons for guest users', () => {
     renderComponent();
     vi.spyOn(userHooks, 'useCurrentUser').mockReturnValue({
-      ...exampleGuestUser,
-      organizations: [],
-      onboardedTeamTypeIds: [],
-      onboardingTeamTypeIds: []
+      ...exampleAuthenticatedGuestUser
     });
 
     fireEvent.click(screen.getByText('Actions'));

src/frontend/src/tests/pages/WorkPackageDetailPage/WorkPackageViewContainer/WorkPackageDetails.test.tsx

@@ -22,6 +22,7 @@ import { mockAuth, mockUseQueryResult } from '../../../test-support/test-data/te
 import { UseQueryResult } from 'react-query';
 import { exampleAdminUser, exampleAppAdminUser, exampleLeadershipUser } from '../../../test-support/test-data/users.stub';
 import { Auth } from '../../../../utils/types';
+import { exampleAuthenticatedAdminUser } from '../../../test-support/test-data/authenticated-user.stub';
 
 vi.mock('../../../../hooks/users.hooks');
 vi.mock('../../../../hooks/auth.hooks');
@@ -31,7 +32,7 @@ const mockedUseAuth = useAuth as jest.Mock<Auth>;
 
 const mockHook = (isLoading: boolean, isError: boolean, data?: User[], error?: Error) => {
   mockedUseAllUsers.mockReturnValue(mockUseQueryResult<User[]>(isLoading, isError, data, error));
-  mockedUseAuth.mockReturnValue(mockAuth(isLoading, exampleAppAdminUser));
+  mockedUseAuth.mockReturnValue(mockAuth(isLoading, exampleAuthenticatedAdminUser));
 };
 
 const users = [exampleAdminUser, exampleAppAdminUser, exampleLeadershipUser];

src/frontend/src/tests/test-support/mock-hooks.ts

@@ -144,3 +144,6 @@ export const mockManyMaterials = (materials: Material[]) =>
 
 export const mockUseGetReimbursementRequestProjectData = (rrData: ReimbursementRequestData) =>
   mockUseQueryResult<ReimbursementRequestData>(false, false, rrData, new Error());
+
+export const mockUseMyTeamAsHead = (teamId?: string) =>
+  mockUseQueryResult<string | undefined>(false, false, teamId, new Error());

src/frontend/src/tests/test-support/test-data/authenticated-user.stub.ts

@@ -11,7 +11,29 @@ export const exampleAuthenticatedAdminUser: AuthenticatedUser = {
   lastName: 'Shmoe',
   email: 'shmoe.j@husky.neu.edu',
   role: RoleEnum.ADMIN,
-  organizations: [],
+  organizations: ['yello'],
+  onboardingTeamTypeIds: [],
+  onboardedTeamTypeIds: []
+};
+
+export const exampleAuthenticatedGuestUser: AuthenticatedUser = {
+  userId: '5',
+  firstName: 'Guest',
+  lastName: 'User',
+  email: 'guest@ner.edu',
+  role: RoleEnum.GUEST,
+  organizations: ['foo'],
+  onboardingTeamTypeIds: [],
+  onboardedTeamTypeIds: []
+};
+
+export const exampleAuthenticatedMemberUser: AuthenticatedUser = {
+  userId: '6',
+  firstName: 'Member',
+  lastName: 'User',
+  email: 'member@ner.edu',
+  role: RoleEnum.MEMBER,
+  organizations: ['bar'],
   onboardingTeamTypeIds: [],
   onboardedTeamTypeIds: []
 };

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 { User } from 'shared';
+import { AuthenticatedUser, User } from 'shared';
 import { exampleAuthenticatedAdminUser } from './authenticated-user.stub';
 import { Auth } from '../../../utils/types';
 
@@ -88,7 +88,7 @@ export const mockUseMutationResult = <Input>(isLoading: boolean, isError: boolea
   } as UseMutationResult<Input, Error>;
 };
 
-export const mockAuth = (isLoading: boolean, user?: User) => {
+export const mockAuth = (isLoading: boolean, user?: AuthenticatedUser) => {
   return {
     user,
     devSignin: () => new Promise((res) => res(exampleAuthenticatedAdminUser)),