throw ValueError instead on InvalidArgumentException

CHANGELOG.md

@@ -11,6 +11,7 @@
   * `stubbles\input\broker\param\ParamBroker::procureParam()`
     * `stubbles\input\broker\param\CustomDatespanParamBroker::procureParam()`
     * `stubbles\input\broker\param\ultipleSourceParamBroker::procureParam()`
+* `stubbles\input\filter\range\StringLength::truncate()` now throws an `\ValueError` instead of an `\InvalidArgumentException`
 
 ## 8.0.2 (2019-12-16)
 

src/main/php/filter/range/StringLength.php

@@ -8,9 +8,9 @@
  */
 namespace stubbles\input\filter\range;
 
-use InvalidArgumentException;
 use LogicException;
-use stubbles\values\Secret;
+use ValueError;
+
 /**
  * String length limitation.
  *
@@ -30,13 +30,13 @@ public function __construct(
      * create instance which treats above max border not as error, but will lead
      * to a truncated value only
      *
-     * @throws  InvalidArgumentException
+     * @throws  ValueError when $maxLength smaller than 1
      * @since   2.3.1
      */
-    public static function truncate(?int $minLength, ?int $maxLength = null)
+    public static function truncate(?int $minLength, int $maxLength)
     {
         if (0 >= $maxLength) {
-            throw new InvalidArgumentException(
+            throw new ValueError(
                 'Max length must be greater than 0, otherwise truncation doesn\'t make sense'
             );
         }

src/test/php/filter/range/StringLengthTest.php

@@ -16,6 +16,7 @@
 use PHPUnit\Framework\Attributes\Test;
 use PHPUnit\Framework\Attributes\TestWith;
 use PHPUnit\Framework\TestCase;
+use ValueError;
 
 use function bovigo\assert\assertThat;
 use function bovigo\assert\assertEmptyArray;
@@ -182,7 +183,7 @@ public function createWithTruncateAndZeroMaxLengthThrowsIllegalArgumentException
         int $maxLength
     ): void {
         expect(fn() => StringLength::truncate(50, $maxLength))
-            ->throws(InvalidArgumentException::class);
+            ->throws(ValueError::class);
     }
 
     /**