Merge pull request #43 from GSG-G8/29-DELETE-cohort

handle Delete cohort route
Close #29

.eslintrc.json

@@ -34,6 +34,8 @@
         "jest/no-focused-tests": "error",
         "jest/no-identical-title": "error",
         "jest/prefer-to-have-length": "warn",
-        "jest/valid-expect": "error"
+        "jest/valid-expect": "error",
+        "consistent-return":"off",
+        "jest/no-test-callback":"off"
     }
 }

server/controllers/index.js

@@ -1,8 +1,11 @@
 const router = require('express').Router();
+const admin = require('./routes/admin');
 
 const { clientError, serverError } = require('./middlewares/errorHandle');
 
 router.use(clientError);
 router.use(serverError);
 
+router.use(admin);
+
 module.exports = router;

server/controllers/routes/admin/cohort/deleteCohort.js

@@ -0,0 +1,23 @@
+const { deleteCohortQuery } = require('../../../../database/queries');
+
+const deleteCohort = async (req, res, next) => {
+  try {
+    const { cohortId } = req.params;
+    const check = await deleteCohortQuery(cohortId);
+    if (check.rowCount !== 0) {
+      res.json({
+        StatusCode: 200,
+        data: { message: 'Cohort deleted successfully' },
+      });
+    } else {
+      res.status(404).json({
+        StatusCode: 404,
+        data: { message: 'Cohort does not exist' },
+      });
+    }
+  } catch (err) {
+    next(err);
+  }
+};
+
+module.exports = deleteCohort;

server/controllers/routes/admin/cohort/index.js

@@ -0,0 +1,5 @@
+const deleteCohort = require('./deleteCohort');
+
+module.exports = {
+  deleteCohort,
+};

server/controllers/routes/admin/index.js

@@ -0,0 +1,21 @@
+const router = require('express').Router();
+const { deleteCohort } = require('./cohort');
+
+router
+  .route('/cohorts/:cohortId')
+  .all((req, res, next) => {
+    // ToDo: make middleware to check authentication
+    next();
+  })
+  .get((req, res, next) => {
+    next(new Error('not implemented'));
+  })
+  .put((req, res, next) => {
+    next(new Error('not implemented'));
+  })
+  .post((req, res, next) => {
+    next(new Error('not implemented'));
+  })
+  .delete(deleteCohort);
+
+module.exports = router;

server/database/queries/cohort/deleteCohort.js

@@ -0,0 +1,6 @@
+const connection = require('../../config/connection');
+
+const deleteCohortQuery = (cohortId) =>
+  connection.query('DELETE FROM cohort WHERE id = $1', [cohortId]);
+
+module.exports = deleteCohortQuery;

server/database/queries/cohort/index.js

@@ -0,0 +1,5 @@
+const deleteCohortQuery = require('./deleteCohort');
+
+module.exports = {
+  deleteCohortQuery,
+};

server/database/queries/index.js

@@ -0,0 +1,5 @@
+const { deleteCohortQuery } = require('./cohort');
+
+module.exports = {
+  deleteCohortQuery,
+};

test/index.test.js

@@ -1,3 +1,28 @@
-test('Initial test', () => {
-  expect(1).toBe(1);
+const request = require('supertest');
+const connection = require('../server/database/config/connection');
+const dbBuild = require('../server/database/config/build');
+
+const app = require('../server/app');
+
+beforeAll(() => dbBuild());
+
+afterAll(() => connection.end());
+
+describe('Admin, (/cohorts/:cohortId)', () => {
+  test('Route /cohorts/1 status 200, data.message = Cohort deleted successfully ', (done) => {
+    return request(app)
+      .delete('/api/v1/cohorts/1')
+      .expect(200)
+      .expect('Content-Type', /json/)
+      .end(async (err, res) => {
+        const { message } = res.body.data;
+        if (err) return done(err);
+        const { rows } = await connection.query(
+          'SELECT * from cohort WHERE id = 1',
+        );
+        expect(rows).toHaveLength(0);
+        expect(message).toBe('Cohort deleted successfully');
+        done();
+      });
+  });
 });