Merge pull request #4 from FluorescentHallucinogen/fix-false-positive

vsimko · web-flow · commit fb1ecee2c765 · 2019-01-31T13:45:12.000+01:00
How did it even work and pass the tests?
diff --git a/src/index.js b/src/index.js
@@ -75,7 +75,7 @@ const prepareConstraintDirective = (validationCallback, errorMessageCallback) =>
         )
 
         const errors = validate(this.args)
-        if (errors) throw new Error(errors)
+        if (errors && errors.length > 0) throw new Error(errors)
 
         return originalResolver.apply(this, resolveArgs)
       }

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ const prepareConstraintDirective = (validationCallback, errorMessageCallback) =>`
`75`	`75`	`)`
`76`	`76`
`77`	`77`	`const errors = validate(this.args)`
`78`		`- if (errors) throw new Error(errors)`
	`78`	`+ if (errors && errors.length > 0) throw new Error(errors)`
`79`	`79`
`80`	`80`	`return originalResolver.apply(this, resolveArgs)`
`81`	`81`	`}`