Merge pull request #2 from jesselang/DjangoModelResource-IntegerArray…

…Field Allow list_n_or_more to validate empty lists.
dlamotte · Oct 9, 2015 · c22432a · c22432a
2 parents 0bf8262 + f5d1b29
commit c22432a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/krankshaft/valid.py b/krankshaft/valid.py
@@ -569,9 +569,9 @@ def list_n_or_more(validator, n):
     Wrap a validator that also validates the returned list has one or more
     members.
     '''
-    if n < 1:
+    if n < 0:
         raise KrankshaftError(
-            'list_n_or_more only accepts values >= 1, not %s' % n
+            'list_n_or_more only accepts values >= 0, not %s' % n
         )
 
     def list_n_or_more_validator(data, expect):

diff --git a/tests/krankshaft/test_valid.py b/tests/krankshaft/test_valid.py
@@ -499,7 +499,7 @@ def test_list_n_or_more_invalid_data(self):
         self.expect_raises(valid.list_n_or_more(valid.int, 1), ['a',2], errors={0: ["invalid literal for int() with base 10: 'a'"]})
 
     def test_list_n_or_more_invalid_n(self):
-        self.assertRaises(valid.KrankshaftError, valid.list_n_or_more, valid.int, 0)
+        self.assertRaises(valid.KrankshaftError, valid.list_n_or_more, valid.int, -1)
 
     def test_slug(self):
         self.expect(valid.slug, 'HELLO WORLD', 'hello-world')