Fix: schedule tests (#2547)

* refactor: update schedule tests * refactor: remove comments * fix: update nest tests
rolling-scopes · Nov 5, 2024 · 99dddca · 99dddca
1 parent eea004c
commit 99dddca
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 3 deletions.
diff --git a/client/src/modules/Schedule/components/StatusTabs/StatusTabs.test.tsx b/client/src/modules/Schedule/components/StatusTabs/StatusTabs.test.tsx
@@ -46,13 +46,16 @@ describe('StatusTabs', () => {
 
     render(<StatusTabs statuses={statuses} onTabChange={onTabChangeMock} />);
 
-    const [all, available, review, future, missed, done, archived] = screen.getAllByRole('tab');
+    const [all, available, review, future, missed, done, registered, unAvailable, archived] =
+      screen.getAllByRole('tab');
     expect(all).toHaveTextContent(new RegExp(ALL_TAB_KEY, 'i'));
     expect(available).toHaveTextContent(new RegExp(StatusEnum.Available, 'i'));
     expect(review).toHaveTextContent(new RegExp(StatusEnum.Review, 'i'));
     expect(future).toHaveTextContent(new RegExp(StatusEnum.Future, 'i'));
     expect(missed).toHaveTextContent(new RegExp(StatusEnum.Missed, 'i'));
     expect(done).toHaveTextContent(new RegExp(StatusEnum.Done, 'i'));
+    expect(registered).toHaveTextContent(new RegExp(StatusEnum.Registered, 'i'));
+    expect(unAvailable).toHaveTextContent(new RegExp(StatusEnum.Unavailable, 'i'));
     expect(archived).toHaveTextContent(new RegExp(StatusEnum.Archived, 'i'));
   });
 
@@ -111,11 +114,13 @@ describe('StatusTabs', () => {
       ${StatusEnum.Archived}
       ${StatusEnum.Future}
       ${StatusEnum.Review}
+      ${StatusEnum.Registered}
+      ${StatusEnum.Unavailable}
     `('should call onTabChange with tab name "$tabName"', ({ tabName }: { tabName: string }) => {
       const statuses = generateStatuses(undefined, { [tabName]: 2 });
       render(<StatusTabs statuses={statuses} onTabChange={onTabChangeMock} />);
 
-      const selectedTab = screen.getByText(new RegExp(tabName, 'i'));
+      const selectedTab = screen.getByText(new RegExp(`^${tabName}$`, 'i'));
       fireEvent.click(selectedTab);
 
       expect(onTabChangeMock).toHaveBeenCalledWith(tabName);

diff --git a/client/src/modules/Schedule/constants.ts b/client/src/modules/Schedule/constants.ts
@@ -53,6 +53,7 @@ export const TAG_NAME_MAP: Record<CourseScheduleItemDto['tag'], string> = {
   'cross-check-review': 'Cross-Check: Review',
   'self-study': 'Self-study',
   lecture: 'Lecture',
+  'team-distribution': 'Team Distribution',
 };
 
 export const SCHEDULE_STATUSES = Object.keys(CourseScheduleItemDtoStatusEnum).map(key => ({
@@ -85,6 +86,7 @@ export const DEFAULT_TAG_COLOR_MAP: Record<CourseScheduleItemDto['tag'], string>
   'cross-check-review': '#36A836',
   'self-study': '#595959',
   lecture: '#eb2f96',
+  'team-distribution': '#308e00',
 };
 
 export const SPECIAL_TASK_TYPES = {

diff --git a/nestjs/src/config/config.service.ts b/nestjs/src/config/config.service.ts
@@ -81,7 +81,7 @@ export class ConfigService {
     };
 
     this.awsClient = {
-      region: conf.get('RSSHCOOL_AWS_REGION') ?? '',
+      region: conf.get('RSSHCOOL_AWS_REGION') ?? 'eu-central-1',
       credentials: {
         accessKeyId: conf.get('RSSHCOOL_AWS_ACCESS_KEY_ID') ?? '',
         secretAccessKey: conf.get('RSSHCOOL_AWS_SECRET_ACCESS_KEY') || '',

diff --git a/nestjs/src/courses/course-schedule/course-schedule.service.spec.ts b/nestjs/src/courses/course-schedule/course-schedule.service.spec.ts
@@ -15,6 +15,8 @@ import {
   TaskInterviewResult,
   TaskResult,
   TaskSolution,
+  TeamDistribution,
+  TeamDistributionStudent,
 } from '@entities/index';
 
 const MOCK_CURRENT_TIME = new Date('2022-03-22T00:00:00.000Z');
@@ -111,6 +113,14 @@ describe('CourseScheduleService', () => {
           provide: getRepositoryToken(TaskChecker),
           useValue: {},
         },
+        {
+          provide: getRepositoryToken(TeamDistributionStudent),
+          useValue: {},
+        },
+        {
+          provide: getRepositoryToken(TeamDistribution),
+          useValue: {},
+        },
       ],
     }).compile();