fix: Make arrayOf return null when there are no errors

cicerono · Nov 22, 2016 · 031a27d · 031a27d
1 parent 13aef96
commit 031a27d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/rules/__tests__/arrayOf_tests.js b/src/rules/__tests__/arrayOf_tests.js
@@ -36,4 +36,8 @@ describe('rules.arrayOf', () => {
 
     expect(output).toEqual(expectedOutput);
   });
+
+  it('should return null if everything is valid', () => {
+    expect(arrayOf('elements', [{}], { name: { numeric: { max: 2 } } }, validator)).toBe(null);
+  });
 });
diff --git a/src/rules/arrayOf.js b/src/rules/arrayOf.js
@@ -1,5 +1,5 @@
 // @flow
-import { keys, map, omit } from 'lodash/fp';
+import { isEmpty, flow, keys, map, omit, reject } from 'lodash/fp';
 import type { ValidatorErrors, Validator, FieldConfig } from '../types';
 
 export default function arrayOf(
@@ -11,5 +11,7 @@ export default function arrayOf(
   const ruleOptions = omit('values')(options);
   const validate = validator(ruleOptions);
   const fields = keys(ruleOptions);
-  return map(item => validate(fields, item))(value);
+  const errors = map(item => validate(fields, item))(value);
+
+  return flow(reject(isEmpty), isEmpty)(errors) ? null : errors;
 }