Updated error message

vladvildanov · vladvildanov · commit 4d10b5303b09 · 2024-07-10T11:28:01.000+03:00
diff --git a/src/Command/Argument/TimeSeries/CommonArguments.php b/src/Command/Argument/TimeSeries/CommonArguments.php
@@ -55,7 +55,7 @@ public function retentionMsecs(int $retentionPeriod): self
     public function ignore(int $maxTimeDiff, float $maxValDiff): self
     {
         if ($maxTimeDiff < 0 || $maxValDiff < 0) {
-            throw new UnexpectedValueException('Ignore does not accept non-positive values');
+            throw new UnexpectedValueException('Ignore does not accept negative values');
         }
 
         array_push($this->arguments, 'IGNORE', $maxTimeDiff, $maxValDiff);
diff --git a/tests/Predis/Command/Argument/TimeSeries/CommonArgumentsTest.php b/tests/Predis/Command/Argument/TimeSeries/CommonArgumentsTest.php
@@ -123,7 +123,7 @@ public function testCreatesArgumentsWithIgnoreModifier(): void
     public function testIgnoreModifierThrowsExceptionOnNonPositiveValues(): void
     {
         $this->expectException(UnexpectedValueException::class);
-        $this->expectExceptionMessage('Ignore does not accept non-positive values');
+        $this->expectExceptionMessage('Ignore does not accept negative values');
 
         $this->arguments->ignore(-1, -1);
     }
diff --git a/tests/Predis/Command/Redis/Search/FTSEARCH_Test.php b/tests/Predis/Command/Redis/Search/FTSEARCH_Test.php
@@ -19,7 +19,6 @@
 use Predis\Command\Argument\Search\SchemaFields\TextField;
 use Predis\Command\Argument\Search\SearchArguments;
 use Predis\Command\Redis\PredisCommandTestCase;
-use Predis\Response\ServerException;
 
 /**
  * @group commands
diff --git a/tests/Predis/Command/Redis/TimeSeries/TSADD_Test.php b/tests/Predis/Command/Redis/TimeSeries/TSADD_Test.php
@@ -61,7 +61,7 @@ public function testFilterArgumentsThrowsExceptionOnNonPositiveValues(): void
         $command = $this->getCommand();
 
         $this->expectException(UnexpectedValueException::class);
-        $this->expectExceptionMessage('Ignore does not accept non-positive values');
+        $this->expectExceptionMessage('Ignore does not accept negative values');
 
         $command->setArguments(['key', 123123121321, 1.0, (new AddArguments())->ignore(-2, -1)]);
     }
diff --git a/tests/Predis/Command/Redis/TimeSeries/TSALTER_Test.php b/tests/Predis/Command/Redis/TimeSeries/TSALTER_Test.php
@@ -62,7 +62,7 @@ public function testFilterArgumentsThrowsExceptionOnNonPositiveValues(): void
         $command = $this->getCommand();
 
         $this->expectException(UnexpectedValueException::class);
-        $this->expectExceptionMessage('Ignore does not accept non-positive values');
+        $this->expectExceptionMessage('Ignore does not accept negative values');
 
         $command->setArguments(['key', 123123121321, 1.0, (new AlterArguments())->ignore(-2, -1)]);
     }
diff --git a/tests/Predis/Command/Redis/TimeSeries/TSCREATE_Test.php b/tests/Predis/Command/Redis/TimeSeries/TSCREATE_Test.php
@@ -61,7 +61,7 @@ public function testFilterArgumentsThrowsExceptionOnNonPositiveValues(): void
         $command = $this->getCommand();
 
         $this->expectException(UnexpectedValueException::class);
-        $this->expectExceptionMessage('Ignore does not accept non-positive values');
+        $this->expectExceptionMessage('Ignore does not accept negative values');
 
         $command->setArguments(['key', 123123121321, 1.0, (new CreateArguments())->ignore(-2, -1)]);
     }

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public function retentionMsecs(int $retentionPeriod): self`
`55`	`55`	`public function ignore(int $maxTimeDiff, float $maxValDiff): self`
`56`	`56`	`{`
`57`	`57`	`if ($maxTimeDiff < 0 \|\| $maxValDiff < 0) {`
`58`		`- throw new UnexpectedValueException('Ignore does not accept non-positive values');`
	`58`	`+ throw new UnexpectedValueException('Ignore does not accept negative values');`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`array_push($this->arguments, 'IGNORE', $maxTimeDiff, $maxValDiff);`
Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ public function testCreatesArgumentsWithIgnoreModifier(): void`
`123`	`123`	`public function testIgnoreModifierThrowsExceptionOnNonPositiveValues(): void`
`124`	`124`	`{`
`125`	`125`	`$this->expectException(UnexpectedValueException::class);`
`126`		`- $this->expectExceptionMessage('Ignore does not accept non-positive values');`
	`126`	`+ $this->expectExceptionMessage('Ignore does not accept negative values');`
`127`	`127`
`128`	`128`	`$this->arguments->ignore(-1, -1);`
`129`	`129`	`}`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ public function testFilterArgumentsThrowsExceptionOnNonPositiveValues(): void`
`61`	`61`	`$command = $this->getCommand();`
`62`	`62`
`63`	`63`	`$this->expectException(UnexpectedValueException::class);`
`64`		`- $this->expectExceptionMessage('Ignore does not accept non-positive values');`
	`64`	`+ $this->expectExceptionMessage('Ignore does not accept negative values');`
`65`	`65`
`66`	`66`	`$command->setArguments(['key', 123123121321, 1.0, (new AddArguments())->ignore(-2, -1)]);`
`67`	`67`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ public function testFilterArgumentsThrowsExceptionOnNonPositiveValues(): void`
`62`	`62`	`$command = $this->getCommand();`
`63`	`63`
`64`	`64`	`$this->expectException(UnexpectedValueException::class);`
`65`		`- $this->expectExceptionMessage('Ignore does not accept non-positive values');`
	`65`	`+ $this->expectExceptionMessage('Ignore does not accept negative values');`
`66`	`66`
`67`	`67`	`$command->setArguments(['key', 123123121321, 1.0, (new AlterArguments())->ignore(-2, -1)]);`
`68`	`68`	`}`