Skip to content

Commit ce1d4d6

Browse files
committed
📦 Rename enum to be clear about semantics and short
As suggested in the review comment #482 (comment)
1 parent 0498984 commit ce1d4d6

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

rules/CodeQuality/Enum/NonStaticNonAssertPrefixedAssertionMethods.php renamed to rules/CodeQuality/Enum/NonAssertNonStaticMethods.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
namespace Rector\PHPUnit\CodeQuality\Enum;
66

7-
final class NonStaticNonAssertPrefixedAssertionMethods
7+
final class NonAssertNonStaticMethods
88
{
99
/**
1010
* @var string[]

rules/CodeQuality/NodeAnalyser/AssertMethodAnalyzer.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
use PHPStan\Type\ObjectType;
1212
use Rector\NodeNameResolver\NodeNameResolver;
1313
use Rector\NodeTypeResolver\NodeTypeResolver;
14-
use Rector\PHPUnit\CodeQuality\Enum\NonStaticNonAssertPrefixedAssertionMethods;
14+
use Rector\PHPUnit\CodeQuality\Enum\NonAssertNonStaticMethods;
1515
use Rector\PHPUnit\Enum\PHPUnitClassName;
1616
use Rector\Reflection\ReflectionResolver;
1717

@@ -37,7 +37,7 @@ public function detectTestCaseCall(MethodCall|StaticCall $call): bool
3737
$methodName = $this->nodeNameResolver->getName($call->name);
3838
if (! str_starts_with((string) $methodName, 'assert') && ! ($call instanceof StaticCall && in_array(
3939
$methodName,
40-
NonStaticNonAssertPrefixedAssertionMethods::ALL,
40+
NonAssertNonStaticMethods::ALL,
4141
true
4242
))) {
4343
return false;

0 commit comments

Comments
 (0)