feature(loans): show all loans for specific user

- filter loans based on userid field
chidimo · May 28, 2019 · 6f72ea3 · 6f72ea3
1 parent a96df7e
commit 6f72ea3
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 5 deletions.
diff --git a/controllers/LoansController.js b/controllers/LoansController.js
@@ -19,19 +19,29 @@ import
 
 const loans_model = new Model('loans');
 const repayments_model = new Model('repayments');
+const rows = `id, userid, createdon, status, repaid, useremail,
+amount, tenor, interest, balance, paymentinstallment`;
 
 const LoansController = {
+
+    get_user_loans: async (req, res) => {
+        const { id } = req.params;
+        const clause = `WHERE userid=${id}`;
+        try {
+            const data = await loans_model.select(rows, clause);
+            return res.status(200).json({ data: data.rows });
+        }
+        catch (e) { return InternalServerError(res, e); }
+    },
     get_all_loans: async (req, res) => {
         const { status, repaid } = req.query;
-        const rows = `id, userid, createdon, status, repaid, useremail,
-            amount, tenor, interest, balance, paymentinstallment`;
+        // const rows = ;
+        const clause = `WHERE status='${status}' AND repaid='${repaid}'`;
         dev_logger(`rep ${repaid}, ${typeof repaid}`);
         try {
             let data;
             if (status && repaid) {
-                data = await loans_model.select(rows,
-                    `WHERE status='${status}' AND repaid='${repaid}'`
-                );
+                data = await loans_model.select(rows, clause);
             }
             else {
                 data = await loans_model.select(rows);

diff --git a/routes/index.js b/routes/index.js
@@ -75,6 +75,12 @@ router.get('/loans',
     AuthenticationMiddleware.verifyToken,
     LoansController.get_all_loans
 );
+
+router.get('/loans/user/:id',
+    AuthenticationMiddleware.verifyToken,
+    LoansController.get_user_loans
+);
+
 router.get('/loans/:id',
     AuthenticationMiddleware.verifyToken,
     LoansController.get_loan

diff --git a/test/loans-spec.js b/test/loans-spec.js
@@ -77,6 +77,20 @@ describe('/loans', () => {
                     done();
                 });
         });
+
+        it('should return all loans for specified user', done => {
+            const id = 1;
+            server
+                .get(`${BASE_URL}/loans/user/${id}`)
+                .expect(200)
+                .end((err, res) => {
+                    res.status.should.equal(200);
+                    for (const each of res.body.data) {
+                        each.should.have.property('userid', id);
+                    }
+                    done();
+                });
+        });
     });
 
     describe('/loans: Get loan', () => {