[Validator] Fix minRatio and maxRatio when getting rounded

Author: alexander-schranz

Constraints/ImageValidator.php

@@ -169,10 +169,10 @@ public function validate($value, Constraint $constraint)
                 throw new ConstraintDefinitionException(sprintf('"%s" is not a valid minimum ratio.', $constraint->minRatio));
             }
 
-            if ($ratio < $constraint->minRatio) {
+            if ($ratio < round($constraint->minRatio, 2)) {
                 $this->context->buildViolation($constraint->minRatioMessage)
                     ->setParameter('{{ ratio }}', $ratio)
-                    ->setParameter('{{ min_ratio }}', $constraint->minRatio)
+                    ->setParameter('{{ min_ratio }}', round($constraint->minRatio, 2))
                     ->setCode(Image::RATIO_TOO_SMALL_ERROR)
                     ->addViolation();
             }
@@ -183,10 +183,10 @@ public function validate($value, Constraint $constraint)
                 throw new ConstraintDefinitionException(sprintf('"%s" is not a valid maximum ratio.', $constraint->maxRatio));
             }
 
-            if ($ratio > $constraint->maxRatio) {
+            if ($ratio > round($constraint->maxRatio, 2)) {
                 $this->context->buildViolation($constraint->maxRatioMessage)
                     ->setParameter('{{ ratio }}', $ratio)
-                    ->setParameter('{{ max_ratio }}', $constraint->maxRatio)
+                    ->setParameter('{{ max_ratio }}', round($constraint->maxRatio, 2))
                     ->setCode(Image::RATIO_TOO_BIG_ERROR)
                     ->addViolation();
             }

Tests/Constraints/Fixtures/test_16by9.gif

@@ -48,6 +48,7 @@ protected function setUp(): void
         $this->imageLandscape = __DIR__.'/Fixtures/test_landscape.gif';
         $this->imagePortrait = __DIR__.'/Fixtures/test_portrait.gif';
         $this->image4By3 = __DIR__.'/Fixtures/test_4by3.gif';
+        $this->image16By9 = __DIR__.'/Fixtures/test_16by9.gif';
         $this->imageCorrupted = __DIR__.'/Fixtures/test_corrupted.gif';
     }
 
@@ -304,6 +305,28 @@ public function testMaxRatioUsesTwoDecimalsOnly()
         $this->assertNoViolation();
     }
 
+    public function testMinRatioUsesInputMoreDecimals()
+    {
+        $constraint = new Image([
+            'minRatio' => 4 / 3,
+        ]);
+
+        $this->validator->validate($this->image4By3, $constraint);
+
+        $this->assertNoViolation();
+    }
+
+    public function testMaxRatioUsesInputMoreDecimals()
+    {
+        $constraint = new Image([
+            'maxRatio' => 16 / 9,
+        ]);
+
+        $this->validator->validate($this->image16By9, $constraint);
+
+        $this->assertNoViolation();
+    }
+
     public function testInvalidMinRatio()
     {
         $this->expectException(ConstraintDefinitionException::class);