chore: Fixed failing tests

Author: hoshomoh

src/Helpers/FormatsMessages.php

@@ -60,9 +60,7 @@ protected function ruleToLower(string $rule): ?string
 
         $lowerRule = strtolower($lowerRule);
 
-        $lowerRule = ltrim($lowerRule, '_');
-
-        return $lowerRule;
+        return ltrim($lowerRule, '_');
     }
 
     /**
@@ -90,9 +88,7 @@ protected function makeReplacements(
 
         $message = $this->replaceValuePlaceholder($message, $value);
 
-        $message = $this->replaceErrorLinePlaceholder($message, $lineNumber);
-
-        return $message;
+        return $this->replaceErrorLinePlaceholder($message, $lineNumber);
     }
 
     /**

src/Rules/AsciiOnly.php

@@ -15,7 +15,7 @@ class AsciiOnly implements ValidationRuleInterface
      */
     public function passes($value, array $parameters): bool
     {
-        return (mb_detect_encoding($value, 'ASCII', true)) ? true : false;
+        return (bool)mb_detect_encoding($value, 'ASCII', true);
     }
 
     /**

src/Rules/Between.php

@@ -21,11 +21,19 @@ public function allowedParameters(): array
      */
     public function passes($value, array $parameters): bool
     {
-        $size = (int) $value;
-
         list($min, $max) = $parameters;
 
-        return $size >= (int) $min && $size <= (int) $max;
+        if (is_numeric($value)) {
+            $convertedValue = +$value;
+            return $convertedValue >= +$min && $convertedValue <= +$max;
+        }
+
+        if (is_string($value)) {
+            $valueLength = strlen($value);
+            return $valueLength >= +$min && $valueLength <= +$max;
+        }
+
+        return false;
     }
 
     /**

src/Rules/ClosureValidationRule.php

@@ -44,8 +44,6 @@ public function __construct(\Closure $callback)
      */
     public function passes($value, array $parameters): bool
     {
-        $this->failed = false;
-
         $this->callback->__invoke($value, function ($message) {
             $this->failed = true;
 

src/Validator/ValidationRuleParser.php

@@ -5,6 +5,7 @@
 namespace Oshomo\CsvUtils\Validator;
 
 use Closure;
+use Oshomo\CsvUtils\Contracts\ValidationRuleInterface;
 use Oshomo\CsvUtils\Contracts\ValidationRuleInterface as ValidationRule;
 use Oshomo\CsvUtils\Rules\ClosureValidationRule;
 
@@ -13,7 +14,7 @@ class ValidationRuleParser
     /**
      * Extract the rule name and parameters from a rule.
      *
-     * @param string $rule|ValidationRuleInterface
+     * @param string|ValidationRuleInterface $rule
      */
     public static function parse($rule): array
     {

tests/src/CsvValidatorParserTest.php

@@ -1,8 +1,8 @@
 <?php
 
-namespace Oshomo\CsvUtils\Validator;
+namespace Oshomo\CsvUtils\Tests\src;
 
-use Oshomo\CsvUtils\Tests\src\UppercaseRule;
+use Oshomo\CsvUtils\Validator\ValidationRuleParser;
 use PHPUnit\Framework\TestCase;
 
 class CsvValidatorParserTest extends TestCase

tests/src/CsvValidatorTest.php

@@ -1,10 +1,10 @@
 <?php
 
-namespace Oshomo\CsvUtils\Validator;
+namespace Oshomo\CsvUtils\Tests\src;
 
 use Oshomo\CsvUtils\Converter\JsonConverter;
 use Oshomo\CsvUtils\Converter\XmlConverter;
-use Oshomo\CsvUtils\Tests\src\UppercaseRule;
+use Oshomo\CsvUtils\Validator\Validator;
 use PHPUnit\Framework\TestCase;
 
 class CsvValidatorTest extends TestCase