Rename courseService

Author: carolynzhang18

backend/rest/courseRoutes.ts

@@ -1,5 +1,5 @@
 import { Router } from "express";
-import CourseService from "../services/implementations/courseService";
+import CourseUnitService from "../services/implementations/courseUnitService";
 import { getErrorMessage } from "../utilities/errorUtils";
 import {
   createCourseUnitDtoValidator,
@@ -8,14 +8,14 @@ import {
 import { isAuthorizedByRole } from "../middlewares/auth";
 
 const courseRouter: Router = Router();
-const courseService: CourseService = new CourseService();
+const courseUnitService: CourseUnitService = new CourseUnitService();
 
 courseRouter.get(
   "/",
-  isAuthorizedByRole(new Set(["Administrator"])),
+  isAuthorizedByRole(new Set(["Administrator", "Facilitator", "Learner"])),
   async (req, res) => {
     try {
-      const courses = await courseService.getCourseUnits();
+      const courses = await courseUnitService.getCourseUnits();
       res.status(200).json(courses);
     } catch (e: unknown) {
       res.status(500).send(getErrorMessage(e));
@@ -29,7 +29,7 @@ courseRouter.post(
   createCourseUnitDtoValidator,
   async (req, res) => {
     try {
-      const newCourse = await courseService.createCourseUnit({
+      const newCourse = await courseUnitService.createCourseUnit({
         title: req.body.title,
       });
       res.status(201).json(newCourse);
@@ -40,13 +40,13 @@ courseRouter.post(
 );
 
 courseRouter.put(
-  "/:id",
+  "/:unitId",
   isAuthorizedByRole(new Set(["Administrator"])),
   updateCourseUnitDtoValidator,
   async (req, res) => {
-    const { id } = req.params;
+    const { unitId } = req.params;
     try {
-      const course = await courseService.updateCourseUnit(id, {
+      const course = await courseUnitService.updateCourseUnit(unitId, {
         title: req.body.title,
       });
       res.status(200).json(course);
@@ -57,12 +57,14 @@ courseRouter.put(
 );
 
 courseRouter.delete(
-  "/:id",
+  "/:unitId",
   isAuthorizedByRole(new Set(["Administrator"])),
   async (req, res) => {
-    const { id } = req.params;
+    const { unitId } = req.params;
     try {
-      const deletedCourseUnitId = await courseService.deleteCourseUnit(id);
+      const deletedCourseUnitId = await courseUnitService.deleteCourseUnit(
+        unitId,
+      );
       res.status(200).json({ id: deletedCourseUnitId });
     } catch (e: unknown) {
       res.status(500).send(getErrorMessage(e));

backend/server.ts

@@ -77,7 +77,7 @@ app.use("/entities", entityRouter);
 app.use("/users", userRouter);
 app.use("/help-request", helpRequestRouter);
 app.use("/notifications", notificationRouter);
-app.use("/courses", courseRouter);
+app.use("/course", courseRouter);
 app.use("/api-docs", swaggerUi.serve, swaggerUi.setup(swaggerDocument));
 app.set("io", io);
 

backend/services/implementations/courseService.ts renamed to backend/services/implementations/courseUnitService.ts

@@ -5,13 +5,13 @@ import {
   UpdateCourseUnitDTO,
 } from "../../types/courseTypes";
 import logger from "../../utilities/logger";
-import ICourseService from "../interfaces/courseService";
+import ICourseUnitService from "../interfaces/courseUnitService";
 import MgCourseUnit, { CourseUnit } from "../../models/courseunit.mgmodel";
 import { getErrorMessage } from "../../utilities/errorUtils";
 
 const Logger = logger(__filename);
 
-class CourseService implements ICourseService {
+class CourseUnitService implements ICourseUnitService {
   async getCourseUnits(): Promise<Array<CourseUnitDTO>> {
     try {
       const courseUnits: Array<CourseUnit> = await MgCourseUnit.find().sort(
@@ -105,4 +105,4 @@ class CourseService implements ICourseService {
   }
 }
 
-export default CourseService;
+export default CourseUnitService;

backend/services/implementations/userService.ts

@@ -254,7 +254,7 @@ class UserService implements IUserService {
         await firebaseAdmin.auth().deleteUser(deletedUser.authId);
       } catch (error) {
         // rollback user deletion in MongoDB
-        const { id, ...rest } = deletedUser.toJSON();
+        const { id, ...rest } = deletedUser.toObject();
         try {
           await MgUser.create(rest);
         } catch (mongoDbError: unknown) {
@@ -292,7 +292,7 @@ class UserService implements IUserService {
         await firebaseAdmin.auth().deleteUser(firebaseUser.uid);
       } catch (error) {
         // rollback user deletion in MongoDB
-        const { id, ...rest } = deletedUser.toJSON();
+        const { id, ...rest } = deletedUser.toObject();
         try {
           await MgUser.create(rest);
         } catch (mongoDbError: unknown) {

backend/services/interfaces/courseService.ts renamed to backend/services/interfaces/courseUnitService.ts

@@ -4,7 +4,7 @@ import {
   UpdateCourseUnitDTO,
 } from "../../types/courseTypes";
 
-interface ICourseService {
+interface ICourseUnitService {
   /**
    * Returns all courseunits
    * @throwsError if courses were not successfully fetched
@@ -38,4 +38,4 @@ interface ICourseService {
   deleteCourseUnit(id: string): Promise<string>;
 }
 
-export default ICourseService;
+export default ICourseUnitService;