fix: skip and limit cast to number in user.model + add test case (#306)

Author: osahner

server/models/user.model.js

@@ -65,8 +65,8 @@ UserSchema.statics = {
   list({ skip = 0, limit = 50 } = {}) {
     return this.find()
       .sort({ createdAt: -1 })
-      .skip(skip)
-      .limit(limit)
+      .skip(+skip)
+      .limit(+limit)
       .exec();
   }
 };

server/tests/user.test.js

@@ -91,6 +91,18 @@ describe('## User APIs', () => {
         })
         .catch(done);
     });
+
+    it('should get all users (with limit and skip)', (done) => {
+      request(app)
+        .get('/api/users')
+        .query({ limit: 10, skip: 1 })
+        .expect(httpStatus.OK)
+        .then((res) => {
+          expect(res.body).to.be.an('array');
+          done();
+        })
+        .catch(done);
+    });
   });
 
   describe('# DELETE /api/users/', () => {