Skip to content

Commit 73f78c5

Browse files
authored
Merge pull request #12 from gsmithun4/development
improve test case coverage
2 parents 68a0fd4 + 174117c commit 73f78c5

File tree

2 files changed

+368
-4
lines changed

2 files changed

+368
-4
lines changed

lib/validator/validator.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ module.exports = (location = 'body', validation = [], response = {}) => {
2121
case 'headers':
2222
return req.headers[param];
2323
default:
24-
return;
24+
return req.body[param];
2525
}
2626
};
2727
const checkEmpty = (value) => {
@@ -107,8 +107,8 @@ module.exports = (location = 'body', validation = [], response = {}) => {
107107
if (!dateString.match(regEx)) { // Invalid format
108108
return false;
109109
}
110-
var d = new Date(`${YYYY}-${MM}-${DD}`);
111-
var dNum = d.getTime();
110+
let d = new Date(`${YYYY}-${MM}-${DD}`);
111+
let dNum = d.getTime();
112112
if(!dNum && dNum !== 0) { // NaN value, Invalid date
113113
return false;
114114
}
@@ -126,7 +126,7 @@ module.exports = (location = 'body', validation = [], response = {}) => {
126126
if (isArray && !Array.isArray(value)) {
127127
throw new Error('Must Be An Array');
128128
}
129-
if (isObject && !(value instanceof Object) && Array.isArray(value)) {
129+
if (isObject && (Array.isArray(value) || !(value instanceof Object))) {
130130
throw new Error('Must Be An Object');
131131
}
132132
if (!isArray && !isObject) {

0 commit comments

Comments
 (0)