Fix formatting

NUS-CRISP · Oct 18, 2024 · 6d2f12e · 6d2f12e
1 parent a8796c1
commit 6d2f12e
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 16 deletions.
diff --git a/backend/test/controllers/courseController.test.ts b/backend/test/controllers/courseController.test.ts
@@ -1064,10 +1064,15 @@ describe('courseController', () => {
 
   describe('addRepositories', () => {
     it('should add repositories to a course', async () => {
-      const req = mockRequest({ gitHubRepoLink: ['repo1', 'repo2'] }, { id: 'courseId' });
+      const req = mockRequest(
+        { gitHubRepoLink: ['repo1', 'repo2'] },
+        { id: 'courseId' }
+      );
       const res = mockResponse();
 
-      jest.spyOn(courseService, 'addRepositoriesToCourse').mockResolvedValue(undefined);
+      jest
+        .spyOn(courseService, 'addRepositoriesToCourse')
+        .mockResolvedValue(undefined);
 
       await addRepositories(req, res);
 
@@ -1078,7 +1083,10 @@ describe('courseController', () => {
     });
 
     it('should handle NotFoundError and send a 404 status', async () => {
-      const req = mockRequest({ gitHubRepoLink: ['repo1', 'repo2'] }, { id: 'courseId' });
+      const req = mockRequest(
+        { gitHubRepoLink: ['repo1', 'repo2'] },
+        { id: 'courseId' }
+      );
       const res = mockResponse();
 
       jest
@@ -1092,7 +1100,10 @@ describe('courseController', () => {
     });
 
     it('should handle errors when adding repositories', async () => {
-      const req = mockRequest({ gitHubRepoLink: ['repo1', 'repo2'] }, { id: 'courseId' });
+      const req = mockRequest(
+        { gitHubRepoLink: ['repo1', 'repo2'] },
+        { id: 'courseId' }
+      );
       const res = mockResponse();
 
       jest
@@ -1110,12 +1121,13 @@ describe('courseController', () => {
 
   describe('updateRepository', () => {
     it('should update repository in a course', async () => {
-      const req = mockRequest({ repoLink: 'repo3' }, { id: 'courseId', repositoryIndex: 1 });
+      const req = mockRequest(
+        { repoLink: 'repo3' },
+        { id: 'courseId', repositoryIndex: 1 }
+      );
       const res = mockResponse();
 
-      jest
-        .spyOn(courseService, 'editRepository')
-        .mockResolvedValue(undefined);
+      jest.spyOn(courseService, 'editRepository').mockResolvedValue(undefined);
 
       await updateRepository(req, res);
 
@@ -1126,7 +1138,10 @@ describe('courseController', () => {
     });
 
     it('should handle NotFoundError and send a 404 status', async () => {
-      const req = mockRequest({ repoLink: 'repo3' }, { id: 'courseId', repositoryIndex: 1 });
+      const req = mockRequest(
+        { repoLink: 'repo3' },
+        { id: 'courseId', repositoryIndex: 1 }
+      );
       const res = mockResponse();
 
       jest
@@ -1140,7 +1155,10 @@ describe('courseController', () => {
     });
 
     it('should handle errors when updating repository', async () => {
-      const req = mockRequest({ repoLink: 'repo3' }, { id: 'courseId', repositoryIndex: 1 });
+      const req = mockRequest(
+        { repoLink: 'repo3' },
+        { id: 'courseId', repositoryIndex: 1 }
+      );
       const res = mockResponse();
 
       jest

diff --git a/backend/test/services/courseService.test.ts b/backend/test/services/courseService.test.ts
@@ -928,24 +928,26 @@ describe('courseService', () => {
 
   describe('getRepositoriesFromCourse', () => {
     it('should get repositories from a course', async () => {
-      // Mock course data
       const course = await CourseModel.findOne({ _id: courseId });
       if (!course) {
         throw new Error('Course not found');
       }
 
+      const repo1 = 'https://github.com/org/repo1';
+      const repo2 = 'https://github.com/org/repo2';
+
       // Add repository links to the course
-      course.gitHubRepoLinks.push('https://github.com/org/repo1');
-      course.gitHubRepoLinks.push('https://github.com/org/repo2');
+      course.gitHubRepoLinks.push(repo1);
+      course.gitHubRepoLinks.push(repo2);
       await course.save();
 
       const repositories = await getRepositoriesFromCourse(courseId);
 
       // Ensure repositories are returned correctly
       expect(repositories).toBeDefined();
       expect(repositories.repositories.length).toBe(2);
-      expect(repositories.repositories).toContain('https://github.com/org/repo1');
-      expect(repositories.repositories).toContain('https://github.com/org/repo2');
+      expect(repositories.repositories).toContain(repo1);
+      expect(repositories.repositories).toContain(repo2);
     });
 
     it('should throw NotFoundError for invalid courseId', async () => {
@@ -1012,7 +1014,9 @@ describe('courseService', () => {
       await editRepository(courseId, repositoryIndex, updateData);
 
       const updatedCourse = await CourseModel.findById(courseId);
-      expect(updatedCourse?.gitHubRepoLinks[repositoryIndex]).toBe(updatedRepoLink);
+      expect(updatedCourse?.gitHubRepoLinks[repositoryIndex]).toBe(
+        updatedRepoLink
+      );
     });
 
     it('should throw NotFoundError for invalid courseId', async () => {