[Validator] Throw exception on Comparison constraints null options

Author: ogizanagi

Constraints/AbstractComparison.php

@@ -29,6 +29,10 @@ abstract class AbstractComparison extends Constraint
      */
     public function __construct($options = null)
     {
+        if (null === $options) {
+            $options = array();
+        }
+
         if (is_array($options) && !isset($options['value'])) {
             throw new ConstraintDefinitionException(sprintf(
                 'The %s constraint requires the "value" option to be set.',

Tests/Constraints/AbstractComparisonValidatorTestCase.php

@@ -68,14 +68,21 @@ protected static function addPhp5Dot5Comparisons(array $comparisons)
         return $result;
     }
 
+    public function provideInvalidConstraintOptions()
+    {
+        return array(
+            array(null),
+            array(array()),
+        );
+    }
+
     /**
+     * @dataProvider provideInvalidConstraintOptions
      * @expectedException \Symfony\Component\Validator\Exception\ConstraintDefinitionException
      */
-    public function testThrowsConstraintExceptionIfNoValueOrProperty()
+    public function testThrowsConstraintExceptionIfNoValueOrProperty($options)
     {
-        $comparison = $this->createConstraint(array());
-
-        $this->validator->validate('some value', $comparison);
+        $this->createConstraint($options);
     }
 
     /**
@@ -169,9 +176,9 @@ public function provideAllInvalidComparisons()
     abstract public function provideInvalidComparisons();
 
     /**
-     * @param array $options Options for the constraint
+     * @param array|null $options Options for the constraint
      *
      * @return Constraint
      */
-    abstract protected function createConstraint(array $options);
+    abstract protected function createConstraint(array $options = null);
 }

Tests/Constraints/EqualToValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new EqualToValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new EqualTo($options);
     }

Tests/Constraints/GreaterThanOrEqualValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new GreaterThanOrEqualValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new GreaterThanOrEqual($options);
     }

Tests/Constraints/GreaterThanValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new GreaterThanValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new GreaterThan($options);
     }

Tests/Constraints/IdenticalToValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new IdenticalToValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new IdenticalTo($options);
     }

Tests/Constraints/LessThanOrEqualValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new LessThanOrEqualValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new LessThanOrEqual($options);
     }

Tests/Constraints/LessThanValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new LessThanValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new LessThan($options);
     }

Tests/Constraints/NotEqualToValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new NotEqualToValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new NotEqualTo($options);
     }

Tests/Constraints/NotIdenticalToValidatorTest.php

@@ -30,7 +30,7 @@ protected function createValidator()
         return new NotIdenticalToValidator();
     }
 
-    protected function createConstraint(array $options)
+    protected function createConstraint(array $options = null)
     {
         return new NotIdenticalTo($options);
     }