phpstan
diff --git a/‎.github/workflows/build.yml
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎.github/workflows/build.yml
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎Makefile
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎Makefile
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/PhpDoc/PHPUnit/MockObjectTypeNodeResolverExtension.php
Copy file name to clipboard
+1-2Lines changed: 1 addition & 2 deletions b/‎src/PhpDoc/PHPUnit/MockObjectTypeNodeResolverExtension.php
Copy file name to clipboard
+1-2Lines changed: 1 addition & 2 deletions
diff --git a/‎src/Rules/PHPUnit/AnnotationHelper.php
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Rules/PHPUnit/AnnotationHelper.php
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Rules/PHPUnit/AssertRuleHelper.php
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Rules/PHPUnit/AssertRuleHelper.php
Copy file name to clipboard
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Rules/PHPUnit/ClassCoversExistsRule.php
Copy file name to clipboard
+5-7Lines changed: 5 additions & 7 deletions b/‎src/Rules/PHPUnit/ClassCoversExistsRule.php
Copy file name to clipboard
+5-7Lines changed: 5 additions & 7 deletions
diff --git a/‎src/Rules/PHPUnit/ClassMethodCoversExistsRule.php
Copy file name to clipboard
+7-11Lines changed: 7 additions & 11 deletions b/‎src/Rules/PHPUnit/ClassMethodCoversExistsRule.php
Copy file name to clipboard
+7-11Lines changed: 7 additions & 11 deletions
diff --git a/‎src/Rules/PHPUnit/CoversHelper.php
Copy file name to clipboard
+6-7Lines changed: 6 additions & 7 deletions b/‎src/Rules/PHPUnit/CoversHelper.php
Copy file name to clipboard
+6-7Lines changed: 6 additions & 7 deletions
diff --git a/‎src/Rules/PHPUnit/DataProviderDeclarationRule.php
Copy file name to clipboard
+5-8Lines changed: 5 additions & 8 deletions b/‎src/Rules/PHPUnit/DataProviderDeclarationRule.php
Copy file name to clipboard
+5-8Lines changed: 5 additions & 8 deletions
diff --git a/‎src/Rules/PHPUnit/DataProviderHelper.php
Copy file name to clipboard
+10-13Lines changed: 10 additions & 13 deletions b/‎src/Rules/PHPUnit/DataProviderHelper.php
Copy file name to clipboard
+10-13Lines changed: 10 additions & 13 deletions
@@ -56,7 +56,7 @@ jobs:
         with:
           repository: "phpstan/build-cs"
           path: "build-cs"
-          ref: "1.x"
+          ref: "2.x"
 
       - name: "Install PHP"
         uses: "shivammathur/setup-php@v2"
 
@@ -13,7 +13,7 @@ lint:
 .PHONY: cs-install
 cs-install:
 	git clone https://github.com/phpstan/build-cs.git || true
-	git -C build-cs fetch origin && git -C build-cs reset --hard origin/1.x
+	git -C build-cs fetch origin && git -C build-cs reset --hard origin/2.x
 	composer install --working-dir build-cs
 
 .PHONY: cs
 
@@ -17,8 +17,7 @@
 class MockObjectTypeNodeResolverExtension implements TypeNodeResolverExtension, TypeNodeResolverAwareExtension
 {
 
-	/** @var TypeNodeResolver */
-	private $typeNodeResolver;
+	private TypeNodeResolver $typeNodeResolver;
 
 	public function setTypeNodeResolver(TypeNodeResolver $typeNodeResolver): void
 	{
 
@@ -56,7 +56,7 @@ public function processDocComment(Doc $docComment): array
 			}
 
 			$errors[] = RuleErrorBuilder::message(
-				'Annotation "' . $matches['annotation'] . '" is invalid, "@' . $matches['property'] . '" should be followed by a space and a value.'
+				'Annotation "' . $matches['annotation'] . '" is invalid, "@' . $matches['property'] . '" should be followed by a space and a value.',
 			)->identifier('phpunit.invalidPhpDoc')->build();
 		}
 
 
@@ -30,7 +30,7 @@ public static function isMethodOrStaticCallOnAssert(Node $node, Scope $scope): b
 							'static',
 							'parent',
 						],
-						true
+						true,
 					)
 				) {
 					$calledOnType = new ObjectType($scope->getClassReflection()->getName());
 
@@ -23,16 +23,14 @@ class ClassCoversExistsRule implements Rule
 	/**
 	 * Covers helper.
 	 *
-	 * @var CoversHelper
 	 */
-	private $coversHelper;
+	private CoversHelper $coversHelper;
 
 	/**
 	 * Reflection provider.
 	 *
-	 * @var ReflectionProvider
 	 */
-	private $reflectionProvider;
+	private ReflectionProvider $reflectionProvider;
 
 	public function __construct(
 		CoversHelper $coversHelper,
@@ -62,7 +60,7 @@ public function processNode(Node $node, Scope $scope): array
 		if (count($classCoversDefaultClasses) >= 2) {
 			return [
 				RuleErrorBuilder::message(sprintf(
-					'@coversDefaultClass is defined multiple times.'
+					'@coversDefaultClass is defined multiple times.',
 				))->identifier('phpunit.coversDuplicate')->build(),
 			];
 		}
@@ -75,15 +73,15 @@ public function processNode(Node $node, Scope $scope): array
 			if (!$this->reflectionProvider->hasClass($className)) {
 				$errors[] = RuleErrorBuilder::message(sprintf(
 					'@coversDefaultClass references an invalid class %s.',
-					$className
+					$className,
 				))->identifier('phpunit.coversClass')->build();
 			}
 		}
 
 		foreach ($classCovers as $covers) {
 			$errors = array_merge(
 				$errors,
-				$this->coversHelper->processCovers($node, $covers, null)
+				$this->coversHelper->processCovers($node, $covers, null),
 			);
 		}
 
 
@@ -25,16 +25,14 @@ class ClassMethodCoversExistsRule implements Rule
 	/**
 	 * Covers helper.
 	 *
-	 * @var CoversHelper
 	 */
-	private $coversHelper;
+	private CoversHelper $coversHelper;
 
 	/**
 	 * The file type mapper.
 	 *
-	 * @var FileTypeMapper
 	 */
-	private $fileTypeMapper;
+	private FileTypeMapper $fileTypeMapper;
 
 	public function __construct(
 		CoversHelper $coversHelper,
@@ -65,9 +63,7 @@ public function processNode(Node $node, Scope $scope): array
 		$classPhpDoc = $classReflection->getResolvedPhpDoc();
 		[$classCovers, $classCoversDefaultClasses] = $this->coversHelper->getCoverAnnotations($classPhpDoc);
 
-		$classCoversStrings = array_map(static function (PhpDocTagNode $covers): string {
-			return (string) $covers->value;
-		}, $classCovers);
+		$classCoversStrings = array_map(static fn (PhpDocTagNode $covers): string => (string) $covers->value, $classCovers);
 
 		$docComment = $node->getDocComment();
 		if ($docComment === null) {
@@ -83,7 +79,7 @@ public function processNode(Node $node, Scope $scope): array
 			$classReflection->getName(),
 			$scope->isInTrait() ? $scope->getTraitReflection()->getName() : null,
 			$node->name->toString(),
-			$docComment->getText()
+			$docComment->getText(),
 		);
 
 		[$methodCovers, $methodCoversDefaultClasses] = $this->coversHelper->getCoverAnnotations($methodPhpDoc);
@@ -93,21 +89,21 @@ public function processNode(Node $node, Scope $scope): array
 		if (count($methodCoversDefaultClasses) > 0) {
 			$errors[] = RuleErrorBuilder::message(sprintf(
 				'@coversDefaultClass defined on class method %s.',
-				$node->name
+				$node->name,
 			))->identifier('phpunit.covers')->build();
 		}
 
 		foreach ($methodCovers as $covers) {
 			if (in_array((string) $covers->value, $classCoversStrings, true)) {
 				$errors[] = RuleErrorBuilder::message(sprintf(
 					'Class already @covers %s so the method @covers is redundant.',
-					$covers->value
+					$covers->value,
 				))->identifier('phpunit.coversDuplicate')->build();
 			}
 
 			$errors = array_merge(
 				$errors,
-				$this->coversHelper->processCovers($node, $covers, $coversDefaultClass)
+				$this->coversHelper->processCovers($node, $covers, $coversDefaultClass),
 			);
 		}
 
 
@@ -20,9 +20,8 @@ class CoversHelper
 	/**
 	 * Reflection provider.
 	 *
-	 * @var ReflectionProvider
 	 */
-	private $reflectionProvider;
+	private ReflectionProvider $reflectionProvider;
 
 	public function __construct(ReflectionProvider $reflectionProvider)
 	{
@@ -48,12 +47,12 @@ public function getCoverAnnotations(?ResolvedPhpDocBlock $phpDoc): array
 		foreach ($phpDocNodes as $docNode) {
 			$covers = array_merge(
 				$covers,
-				$docNode->getTagsByName('@covers')
+				$docNode->getTagsByName('@covers'),
 			);
 
 			$coversDefaultClasses = array_merge(
 				$coversDefaultClasses,
-				$docNode->getTagsByName('@coversDefaultClass')
+				$docNode->getTagsByName('@coversDefaultClass'),
 			);
 		}
 
@@ -100,14 +99,14 @@ public function processCovers(
 			if ($class->isInterface()) {
 				$errors[] = RuleErrorBuilder::message(sprintf(
 					'@covers value %s references an interface.',
-					$fullName
+					$fullName,
 				))->identifier('phpunit.coversInterface')->build();
 			}
 
 			if (isset($method) && $method !== '' && !$class->hasMethod($method)) {
 				$errors[] = RuleErrorBuilder::message(sprintf(
 					'@covers value %s references an invalid method.',
-					$fullName
+					$fullName,
 				))->identifier('phpunit.coversMethod')->build();
 			}
 		} elseif (isset($method) && $this->reflectionProvider->hasFunction(new Name($method, []), null)) {
@@ -118,7 +117,7 @@ public function processCovers(
 			$error = RuleErrorBuilder::message(sprintf(
 				'@covers value %s references an invalid %s.',
 				$fullName,
-				$isMethod ? 'method' : 'class or function'
+				$isMethod ? 'method' : 'class or function',
 			))->identifier(sprintf('phpunit.covers%s', $isMethod ? 'Method' : ''));
 
 			if (strpos($className, '\\') === false) {
 
@@ -17,23 +17,20 @@ class DataProviderDeclarationRule implements Rule
 	/**
 	 * Data provider helper.
 	 *
-	 * @var DataProviderHelper
 	 */
-	private $dataProviderHelper;
+	private DataProviderHelper $dataProviderHelper;
 
 	/**
 	 * When set to true, it reports data provider method with incorrect name case.
 	 *
-	 * @var bool
 	 */
-	private $checkFunctionNameCase;
+	private bool $checkFunctionNameCase;
 
 	/**
 	 * When phpstan-deprecation-rules is installed, it reports deprecated usages.
 	 *
-	 * @var bool
 	 */
-	private $deprecationRulesInstalled;
+	private bool $deprecationRulesInstalled;
 
 	public function __construct(
 		DataProviderHelper $dataProviderHelper,
@@ -70,8 +67,8 @@ public function processNode(Node $node, Scope $scope): array
 					$dataProviderMethodName,
 					$lineNumber,
 					$this->checkFunctionNameCase,
-					$this->deprecationRulesInstalled
-				)
+					$this->deprecationRulesInstalled,
+				),
 			);
 		}
 
 
@@ -28,19 +28,16 @@ class DataProviderHelper
 	/**
 	 * Reflection provider.
 	 *
-	 * @var ReflectionProvider
 	 */
-	private $reflectionProvider;
+	private ReflectionProvider $reflectionProvider;
 
 	/**
 	 * The file type mapper.
 	 *
-	 * @var FileTypeMapper
 	 */
-	private $fileTypeMapper;
+	private FileTypeMapper $fileTypeMapper;
 
-	/** @var bool */
-	private $phpunit10OrNewer;
+	private bool $phpunit10OrNewer;
 
 	public function __construct(
 		ReflectionProvider $reflectionProvider,
@@ -69,7 +66,7 @@ public function getDataProviderMethods(
 				$classReflection->getName(),
 				$scope->isInTrait() ? $scope->getTraitReflection()->getName() : null,
 				$node->name->toString(),
-				$docComment->getText()
+				$docComment->getText(),
 			);
 			foreach ($this->getDataProviderAnnotations($methodPhpDoc) as $annotation) {
 				$dataProviderValue = $this->getDataProviderAnnotationValue($annotation);
@@ -122,7 +119,7 @@ private function getDataProviderAnnotations(?ResolvedPhpDocBlock $phpDoc): array
 		foreach ($phpDocNodes as $docNode) {
 			$annotations = array_merge(
 				$annotations,
-				$docNode->getTagsByName('@dataProvider')
+				$docNode->getTagsByName('@dataProvider'),
 			);
 		}
 
@@ -145,7 +142,7 @@ public function processDataProvider(
 			return [
 				RuleErrorBuilder::message(sprintf(
 					'@dataProvider %s related class not found.',
-					$dataProviderValue
+					$dataProviderValue,
 				))
 					->line($lineNumber)
 					->identifier('phpunit.dataProviderClass')
@@ -159,7 +156,7 @@ public function processDataProvider(
 			return [
 				RuleErrorBuilder::message(sprintf(
 					'@dataProvider %s related method not found.',
-					$dataProviderValue
+					$dataProviderValue,
 				))
 					->line($lineNumber)
 					->identifier('phpunit.dataProviderMethod')
@@ -173,7 +170,7 @@ public function processDataProvider(
 			$errors[] = RuleErrorBuilder::message(sprintf(
 				'@dataProvider %s related method is used with incorrect case: %s.',
 				$dataProviderValue,
-				$dataProviderMethodReflection->getName()
+				$dataProviderMethodReflection->getName(),
 			))
 				->line($lineNumber)
 				->identifier('method.nameCase')
@@ -183,7 +180,7 @@ public function processDataProvider(
 		if (!$dataProviderMethodReflection->isPublic()) {
 			$errors[] = RuleErrorBuilder::message(sprintf(
 				'@dataProvider %s related method must be public.',
-				$dataProviderValue
+				$dataProviderValue,
 			))
 				->line($lineNumber)
 				->identifier('phpunit.dataProviderPublic')
@@ -193,7 +190,7 @@ public function processDataProvider(
 		if ($deprecationRulesInstalled && $this->phpunit10OrNewer && !$dataProviderMethodReflection->isStatic()) {
 			$errors[] = RuleErrorBuilder::message(sprintf(
 				'@dataProvider %s related method must be static in PHPUnit 10 and newer.',
-				$dataProviderValue
+				$dataProviderValue,
 			))
 				->line($lineNumber)
 				->identifier('phpunit.dataProviderStatic')
Original file line number	Diff line number	Diff line change
`@@ -17,8 +17,7 @@`
`17`	`17`	`class MockObjectTypeNodeResolverExtension implements TypeNodeResolverExtension, TypeNodeResolverAwareExtension`
`18`	`18`	`{`
`19`	`19`
`20`		`- /** @var TypeNodeResolver */`
`21`		`- private $typeNodeResolver;`
	`20`	`+ private TypeNodeResolver $typeNodeResolver;`
`22`	`21`
`23`	`22`	`public function setTypeNodeResolver(TypeNodeResolver $typeNodeResolver): void`
`24`	`23`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public function processDocComment(Doc $docComment): array`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`$errors[] = RuleErrorBuilder::message(`
`59`		`- 'Annotation "' . $matches['annotation'] . '" is invalid, "@' . $matches['property'] . '" should be followed by a space and a value.'`
	`59`	`+ 'Annotation "' . $matches['annotation'] . '" is invalid, "@' . $matches['property'] . '" should be followed by a space and a value.',`
`60`	`60`	`)->identifier('phpunit.invalidPhpDoc')->build();`
`61`	`61`	`}`
`62`	`62`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public static function isMethodOrStaticCallOnAssert(Node $node, Scope $scope): b`
`30`	`30`	`'static',`
`31`	`31`	`'parent',`
`32`	`32`	`],`
`33`		`- true`
	`33`	`+ true,`
`34`	`34`	`)`
`35`	`35`	`) {`
`36`	`36`	`$calledOnType = new ObjectType($scope->getClassReflection()->getName());`