Edit route name

server/controllers/index.js

@@ -3,6 +3,6 @@ const {
   cohort: { getSpecificCohort },
 } = require('./routes/user');
 
-router.get('/cohort/:cohortid', getSpecificCohort);
+router.get('/cohorts/:cohortid', getSpecificCohort);
 
 module.exports = router;

server/controllers/routes/user/cohort/getSpecificCohort.js

@@ -1,21 +1,19 @@
-const {
-  cohort: { getSpecificCohort },
-} = require('../../../../database/queries');
+const getCohortQuery = require('../../../../database/queries');
 
-exports.getSpecificCohort = async (req, res, next) => {
+exports.getSpecificCohort = async (req, res) => {
   try {
     const { cohortid } = req.params;
-    const { rows } = await getSpecificCohort(cohortid);
+    const { rows } = await getCohortQuery(cohortid);
     const data = { ...rows[0] };
     if (data.id) {
       res.json({ statusCode: 200, data });
     } else {
-      res.status(404).json({
-        statusCode: 404,
-        message: "Sorry There's no cohort for this id",
-      });
+      throw new Error();
     }
   } catch (err) {
-    next(err);
+    res.status(404).json({
+      statusCode: 404,
+      message: "Sorry There's no cohort for this id",
+    });
   }
 };

server/database/queries/cohort/getSpecificCohort.js

@@ -1,4 +1,4 @@
 const connection = require('../../config/connection');
 
-exports.getSpecificCohort = (id) =>
+exports.getCohortQuery = (id) =>
   connection.query('select * from cohort where id = $1', [id]);

server/database/queries/cohort/index.js

@@ -1,3 +1,3 @@
-const { getSpecificCohort } = require('./getSpecificCohort');
+const { getCohortQuery } = require('./getSpecificCohort');
 
-module.exports = { getSpecificCohort };
+module.exports = getCohortQuery;

server/database/queries/index.js

@@ -1,3 +1,3 @@
-const cohort = require('./cohort');
+const getCohortQuery = require('./cohort');
 
-module.exports = { cohort };
+module.exports = getCohortQuery;

test/index.test.js

@@ -13,9 +13,9 @@ afterAll(() => {
 });
 
 describe('Cohort', () => {
-  test('Route /cohort/1 status 200, json header, data.name =G8 ', (done) => {
+  test('Route /cohorts/1 status 200, json header, data.name =G8 ', (done) => {
     return request(app)
-      .get('/api/v1/cohort/1')
+      .get('/api/v1/cohorts/1')
       .expect(200)
       .expect('Content-Type', /json/)
       .end((err, res) => {
@@ -26,9 +26,9 @@ describe('Cohort', () => {
       });
   });
 
-  test('Route /cohort/10 status 404, json header, data.message = "Sorry There\'s no cohort for this id" ', (done) => {
+  test('Route /cohorts/10 status 404, json header, data.message = "Sorry There\'s no cohort for this id" ', (done) => {
     return request(app)
-      .get('/api/v1/cohort/10')
+      .get('/api/v1/cohorts/10')
       .expect(404)
       .expect('Content-Type', /json/)
       .end((err, res) => {