From c2c2b88cd4b82790a9013b141725a2088a900c6c Mon Sep 17 00:00:00 2001 From: Chris Smith Date: Thu, 16 Jan 2020 08:45:51 +0000 Subject: [PATCH] Change error message for consistency --- .../Functions/RandomIntParametersRule.php | 6 +----- .../Functions/RandomIntParametersRuleTest.php | 20 +++++++++---------- 2 files changed, 11 insertions(+), 15 deletions(-) diff --git a/src/Rules/Functions/RandomIntParametersRule.php b/src/Rules/Functions/RandomIntParametersRule.php index bf395f95ee..1db860e10b 100644 --- a/src/Rules/Functions/RandomIntParametersRule.php +++ b/src/Rules/Functions/RandomIntParametersRule.php @@ -57,11 +57,7 @@ public function processNode(Node $node, Scope $scope): array } if (!$maxPermittedType->isSuperTypeOf($maxType)->yes()) { - $message = 'Cannot call random_int() when $min parameter (%s) can be greater than $max parameter (%s).'; - - if ($maxType->isSuperTypeOf($minType)->no()) { - $message = 'Cannot call random_int() when $min parameter (%s) is greater than $max parameter (%s).'; - } + $message = 'Parameter #1 $min (%s) of function random_int expects lower number than parameter #2 $max (%s).'; return [ RuleErrorBuilder::message(sprintf( diff --git a/tests/PHPStan/Rules/Functions/RandomIntParametersRuleTest.php b/tests/PHPStan/Rules/Functions/RandomIntParametersRuleTest.php index 144ec84877..06d0f8a03a 100644 --- a/tests/PHPStan/Rules/Functions/RandomIntParametersRuleTest.php +++ b/tests/PHPStan/Rules/Functions/RandomIntParametersRuleTest.php @@ -10,46 +10,46 @@ class RandomIntParametersRuleTest extends \PHPStan\Testing\RuleTestCase protected function getRule(): \PHPStan\Rules\Rule { - return new RandomIntParametersRule($this->createReflectionProvider()); + return new RandomIntParametersRule($this->createReflectionProvider(), true); } public function testFile(): void { $this->analyse([__DIR__ . '/data/random-int.php'], [ [ - 'Cannot call random_int() when $min parameter (1) is greater than $max parameter (0).', + 'Parameter #1 $min (1) of function random_int expects lower number than parameter #2 $max (0).', 8, ], [ - 'Cannot call random_int() when $min parameter (0) is greater than $max parameter (-1).', + 'Parameter #1 $min (0) of function random_int expects lower number than parameter #2 $max (-1).', 9, ], [ - 'Cannot call random_int() when $min parameter (0) is greater than $max parameter (int<-10, -1>).', + 'Parameter #1 $min (0) of function random_int expects lower number than parameter #2 $max (int<-10, -1>).', 11, ], [ - 'Cannot call random_int() when $min parameter (0) can be greater than $max parameter (int<-10, 10>).', + 'Parameter #1 $min (0) of function random_int expects lower number than parameter #2 $max (int<-10, 10>).', 12, ], [ - 'Cannot call random_int() when $min parameter (int<1, 10>) is greater than $max parameter (0).', + 'Parameter #1 $min (int<1, 10>) of function random_int expects lower number than parameter #2 $max (0).', 15, ], [ - 'Cannot call random_int() when $min parameter (int<-10, 10>) can be greater than $max parameter (0).', + 'Parameter #1 $min (int<-10, 10>) of function random_int expects lower number than parameter #2 $max (0).', 16, ], [ - 'Cannot call random_int() when $min parameter (int<-5, 1>) can be greater than $max parameter (int<0, 5>).', + 'Parameter #1 $min (int<-5, 1>) of function random_int expects lower number than parameter #2 $max (int<0, 5>).', 19, ], [ - 'Cannot call random_int() when $min parameter (int<-5, 0>) can be greater than $max parameter (int<-1, 5>).', + 'Parameter #1 $min (int<-5, 0>) of function random_int expects lower number than parameter #2 $max (int<-1, 5>).', 20, ], [ - 'Cannot call random_int() when $min parameter (int<0, 10>) can be greater than $max parameter (int<0, 10>).', + 'Parameter #1 $min (int<0, 10>) of function random_int expects lower number than parameter #2 $max (int<0, 10>).', 31, ], ]);