Merge pull request #187 from weierophinney/hotfix/183

Ensure Between validator has both min and max values during construction

Author: weierophinney

src/Between.php

@@ -90,15 +90,17 @@ public function __construct($options = null)
             $options = $temp;
         }
 
-        if (count($options) !== 2
-            && (! array_key_exists('min', $options) || ! array_key_exists('max', $options))
-        ) {
+        if (! array_key_exists('min', $options) || ! array_key_exists('max', $options)) {
             throw new Exception\InvalidArgumentException("Missing option: 'min' and 'max' have to be given");
         }
 
-        if (is_numeric($options['min']) && is_numeric($options['max'])) {
+        if ((isset($options['min']) && is_numeric($options['min']))
+            && (isset($options['max']) && is_numeric($options['max']))
+        ) {
             $this->numeric = true;
-        } elseif (is_string($options['min']) && is_string($options['max'])) {
+        } elseif ((isset($options['min']) && is_string($options['min']))
+            && (isset($options['max']) && is_string($options['max']))
+        ) {
             $this->numeric = false;
         } else {
             throw new Exception\InvalidArgumentException(

test/BetweenTest.php

@@ -265,12 +265,13 @@ public function testMissingMinOrMax(array $args)
     public function constructBetweenValidatorInvalidDataProvider()
     {
         return [
-            [
-                ['min' => 1],
-            ],
-            [
-                ['max' => 5],
-            ],
+            'only-min'      => [['min' => 1]],
+            'only-max'      => [['max' => 5]],
+            'min-inclusive' => [['min' => 0, 'inclusive' => true]],
+            'max-inclusive' => [['max' => 5, 'inclusive' => true]],
+            'min-undefined' => [['min' => 0, 'foo' => 'bar']],
+            'max-undefined' => [['max' => 5, 'foo' => 'bar']],
+            'no-min-or-max' => [['bar' => 'foo', 'foo' => 'bar']],
         ];
     }