✨ Show student list complete

api/controllers/coursesControllers/__tests__/renderStudentList.controller.test.js

@@ -0,0 +1,63 @@
+jest.mock('../../helpers');
+jest.mock('../../../../database/services/modelServices/courseServices');
+
+const {
+  renderStudentListController,
+} = require('../renderStudentList.controller');
+const {
+  render500ErrorHelper,
+  redirectNonexistentCourseHelper,
+} = require('../../helpers');
+const {
+  findOneCourseService,
+} = require('../../../../database/services/modelServices/courseServices');
+const {
+  validateMockValueToHaveBeenCalled,
+} = require('../../../../utils/test-utils/validators.utils');
+const {
+  mockRequest,
+  mockResponse,
+} = require('../../../../utils/test-utils/interceptors.utils');
+
+let req;
+let res;
+
+describe('renderStudentList Controller Test Suite', () => {
+  beforeEach(() => {
+    req = mockRequest();
+    res = mockResponse();
+
+    req.user = {
+      local: expect.anything(),
+    };
+  });
+
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  test('should validate render500ErrorHelper is called', async () => {
+    findOneCourseService.mockReturnValueOnce(new Error());
+
+    await renderStudentListController(req, res);
+
+    validateMockValueToHaveBeenCalled(render500ErrorHelper);
+  });
+
+  test('should validate redirectNonexistentCourseHelper is called', async () => {
+    findOneCourseService.mockReturnValueOnce(null);
+
+    await renderStudentListController(req, res);
+
+    validateMockValueToHaveBeenCalled(redirectNonexistentCourseHelper);
+  });
+
+  test('should validate res.status & res.render is called', async () => {
+    await renderStudentListController(req, res);
+
+    const { status, render } = res;
+
+    validateMockValueToHaveBeenCalled(status);
+    validateMockValueToHaveBeenCalled(render);
+  });
+});

api/controllers/coursesControllers/__tests__/renderTaughtCourse.controller.test.js

@@ -4,7 +4,10 @@ jest.mock('../../../../database/services/modelServices/courseServices');
 const {
   renderTaughtCourseController,
 } = require('../renderTaughtCourse.controller');
-const { render500ErrorHelper } = require('../../helpers');
+const {
+  render500ErrorHelper,
+  redirectNonexistentCourseHelper,
+} = require('../../helpers');
 const {
   findOneCourseService,
 } = require('../../../../database/services/modelServices/courseServices');
@@ -41,16 +44,12 @@ describe('renderTaughtCourse Controller Test Suite', () => {
     validateMockValueToHaveBeenCalled(render500ErrorHelper);
   });
 
-  test('should validate req.flash & res.redirect is called', async () => {
+  test('should validate redirectNonexistentCourseHelper is called', async () => {
     findOneCourseService.mockReturnValueOnce(null);
 
     await renderTaughtCourseController(req, res);
 
-    const { flash } = req;
-    const { redirect } = res;
-
-    validateMockValueToHaveBeenCalled(flash);
-    validateMockValueToHaveBeenCalled(redirect);
+    validateMockValueToHaveBeenCalled(redirectNonexistentCourseHelper);
   });
 
   test('should validate res.status & res.render is called', async () => {

api/controllers/coursesControllers/index.js

@@ -2,5 +2,12 @@ const { renderMyCoursesController } = require('./renderMyCourses.controller');
 const {
   renderTaughtCourseController,
 } = require('./renderTaughtCourse.controller');
+const {
+  renderStudentListController,
+} = require('./renderStudentList.controller');
 
-module.exports = { renderMyCoursesController, renderTaughtCourseController };
+module.exports = {
+  renderMyCoursesController,
+  renderStudentListController,
+  renderTaughtCourseController,
+};

api/controllers/coursesControllers/renderStudentList.controller.js

@@ -0,0 +1,27 @@
+const {
+  findOneCourseService,
+} = require('../../../database/services/modelServices/courseServices');
+
+const {
+  render500ErrorHelper,
+  redirectNonexistentCourseHelper,
+} = require('../helpers');
+
+exports.renderStudentListController = async (req, res) => {
+  const { local } = req.user;
+  const { id } = req.params;
+
+  const isCourse = await findOneCourseService(id);
+
+  if (isCourse instanceof Error) {
+    render500ErrorHelper(res);
+    return;
+  }
+
+  if (isCourse === null) {
+    redirectNonexistentCourseHelper(req, res);
+    return;
+  }
+
+  res.status(200).render('users/instructors/student-list', { local });
+};

api/controllers/coursesControllers/renderTaughtCourse.controller.js

@@ -1,8 +1,10 @@
 const {
   findOneCourseService,
 } = require('../../../database/services/modelServices/courseServices');
-
-const { render500ErrorHelper } = require('../helpers');
+const {
+  render500ErrorHelper,
+  redirectNonexistentCourseHelper,
+} = require('../helpers');
 
 exports.renderTaughtCourseController = async (req, res) => {
   const { local } = req.user;
@@ -16,8 +18,7 @@ exports.renderTaughtCourseController = async (req, res) => {
   }
 
   if (isCourse === null) {
-    req.flash('error_msg', "Course doesn't exist!");
-    res.redirect(302, '/');
+    redirectNonexistentCourseHelper(req, res);
     return;
   }
 

api/controllers/helpers/__tests__/redirectNonexistentCourse.helper.test.js

@@ -0,0 +1,28 @@
+const { redirectNonexistentCourseHelper } = require('../index');
+const {
+  mockRequest,
+  mockResponse,
+} = require('../../../../utils/test-utils/interceptors.utils');
+
+const {
+  validateMockValueToHaveBeenCalled,
+} = require('../../../../utils/test-utils/validators.utils');
+
+describe('redirectNonexistentCourse Helper Test Suite', () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  test('should validate res.status & res.render being called', () => {
+    const req = mockRequest();
+    const res = mockResponse();
+
+    redirectNonexistentCourseHelper(req, res);
+
+    const { flash } = req;
+    const { redirect } = res;
+
+    validateMockValueToHaveBeenCalled(flash);
+    validateMockValueToHaveBeenCalled(redirect);
+  });
+});

api/controllers/helpers/index.js

@@ -12,6 +12,9 @@ const {
 const {
   renderRegistrationFormWithErrorsHelper,
 } = require('./renderRegistrationFormWithErrors.helper');
+const {
+  redirectNonexistentCourseHelper,
+} = require('./redirectNonexistentCourse.helper');
 
 module.exports = {
   filterCoursesHelper,
@@ -20,6 +23,7 @@ module.exports = {
   authenticateUserHelper,
   checkIfValidObjectIdHelper,
   handleUpdatedPasswordHelper,
+  redirectNonexistentCourseHelper,
   renderLoginFormWithErrorsHelper,
   renderRegistrationFormWithErrorsHelper,
 };

api/controllers/helpers/redirectNonexistentCourse.helper.js

@@ -0,0 +1,4 @@
+exports.redirectNonexistentCourseHelper = (request, response) => {
+  request.flash('error_msg', "Course doesn't exist!");
+  response.redirect(302, '/');
+};

api/controllers/index.js

@@ -21,6 +21,7 @@ const {
 } = require('./instructorsControllers');
 const {
   renderMyCoursesController,
+  renderStudentListController,
   renderTaughtCourseController,
 } = require('./coursesControllers');
 
@@ -34,6 +35,7 @@ module.exports = {
   registerNewUserController,
   createNewCourseController,
   renderMyCoursesController,
+  renderStudentListController,
   updateUserProfileController,
   renderUserProfileController,
   renderTaughtCourseController,

api/routes/courses.routes.js

@@ -1,8 +1,13 @@
 const { Router } = require('express');
 const { isInstructor, isLoggedInUser } = require('../middleware');
-const { myCoursesPrefix, myCoursesTeachPrefix } = require('../../configs');
+const {
+  myCoursesPrefix,
+  myCoursesTeachPrefix,
+  myCoursesTeachStudentListPrefix,
+} = require('../../configs');
 const {
   renderMyCoursesController,
+  renderStudentListController,
   renderTaughtCourseController,
 } = require('../controllers');
 
@@ -15,5 +20,11 @@ router.get(
   isInstructor,
   renderTaughtCourseController
 );
+router.get(
+  myCoursesTeachStudentListPrefix,
+  isLoggedInUser,
+  isInstructor,
+  renderStudentListController
+);
 
 module.exports = router;

configs/index.js

@@ -27,6 +27,7 @@ module.exports = {
   newCoursePrefix: '/course/new',
   myCoursesPrefix: '/my-courses',
   myCoursesTeachPrefix: '/my-courses/teach/:id',
+  myCoursesTeachStudentListPrefix: '/my-courses/teach/:id/studentList',
   myCoursesLearnPrefix: '/my-courses/learn/:id',
 
   /* Database  */

database/services/modelServices/courseServices/__tests__/findOneCourse.service.test.js

@@ -1,5 +1,10 @@
 const { findOneCourseService, createNewCourseService } = require('../index');
-const { fakeCourseData, fakeIdFormatData } = require('../../../../fixtures');
+const { createNewUserService } = require('../../userServices');
+const {
+  fakeUserData,
+  fakeCourseData,
+  fakeIdFormatData,
+} = require('../../../../fixtures');
 const {
   dbConnect,
   dbDisconnect,
@@ -23,10 +28,17 @@ describe('findOneCourse Service Test Suite', () => {
   });
 
   test('should validate successfully finding one saved course', async () => {
+    const newUser = await createNewUserService(fakeUserData);
+
     const { title, description } = fakeCourseData;
 
     const newCourse = await createNewCourseService(title, description);
 
+    const { _id: userId } = newUser;
+
+    newCourse.instructors.push(userId);
+    newCourse.save();
+
     const { _id } = newCourse;
 
     const results = await findOneCourseService(_id);

database/services/modelServices/courseServices/findOneCourse.service.js

@@ -5,7 +5,7 @@ const {
 
 exports.findOneCourseService = async (courseId) => {
   const [foundResults, foundError] = await handleAsyncFunction(
-    Course.findById(courseId)
+    Course.findById(courseId).populate('instructors students')
   );
 
   return foundResults || foundError;

views/users/instructors/courses-taught.hbs

@@ -10,7 +10,7 @@
             <div>
                 <div>
                     <p>Registered Students: <span>{{isCourse.students.length}}</span></p>
-                    <a href="/my-courses/{{isCourse._id}}/studentList">See Student List</a>
+                    <a href="/my-courses/teach/{{isCourse._id}}/studentList">See Student List</a>
                 </div>
                 <div>
                     <p>Edit Course</p>

views/users/instructors/student-list.hbs

@@ -0,0 +1,20 @@
+<section>
+    <h2>Students List</h2>
+
+    {{!-- <table>
+        <thead>
+            <tr>
+                <th>Firstname</th>
+                <th>Lastname</th>
+            </tr>
+        </thead>
+        <tbody>
+            {{#each nameList}}
+            <tr>
+                <td>{{this.firstName}}</td>
+                <td>{{this.lastName}}</td>
+            </tr>
+            {{/each}}
+        </tbody>
+    </table> --}}
+</section>