Skip to content

Commit 0f857bf

authored

Introduce phpstan-deprecation-rules

1 parent 17bbfd3 commit 0f857bfCopy full SHA for 0f857bf

File tree

9 files changed

+11

-9

lines changed

composer.json
phpstan.neon
src/Rules/PHPUnit

9 files changed

+11

-9

lines changed

`‎composer.json`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@`
`14`	`14`	`},`
`15`	`15`	`"require-dev": {`
`16`	`16`	`"php-parallel-lint/php-parallel-lint": "^1.2",`
	`17`	`+ "phpstan/phpstan-deprecation-rules": "^2.0",`
`17`	`18`	`"phpstan/phpstan-strict-rules": "^2.0",`
`18`	`19`	`"phpunit/phpunit": "^9.6"`
`19`	`20`	`},`

`‎phpstan.neon`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@ includes:`
`2`	`2`	`- extension.neon`
`3`	`3`	`- rules.neon`
`4`	`4`	`- vendor/phpstan/phpstan-strict-rules/rules.neon`
	`5`	`+ - vendor/phpstan/phpstan-deprecation-rules/rules.neon`
`5`	`6`	`- phar://phpstan.phar/conf/bleedingEdge.neon`
`6`	`7`	`- phpstan-baseline.neon`
`7`	`8`

`‎src/Rules/PHPUnit/ClassCoversExistsRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function processNode(Node $node, Scope $scope): array`
`50`	`50`	`{`
`51`	`51`	`$classReflection = $node->getClassReflection();`
`52`	`52`
`53`		`- if (!$classReflection->isSubclassOf(TestCase::class)) {`
	`53`	`+ if (!$classReflection->is(TestCase::class)) {`
`54`	`54`	`return [];`
`55`	`55`	`}`
`56`	`56`

`‎src/Rules/PHPUnit/ClassMethodCoversExistsRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public function processNode(Node $node, Scope $scope): array`
`56`	`56`	`return [];`
`57`	`57`	`}`
`58`	`58`
`59`		`- if (!$classReflection->isSubclassOf(TestCase::class)) {`
	`59`	`+ if (!$classReflection->is(TestCase::class)) {`
`60`	`60`	`return [];`
`61`	`61`	`}`
`62`	`62`

`‎src/Rules/PHPUnit/DataProviderDeclarationRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function processNode(Node $node, Scope $scope): array`
`52`	`52`	`{`
`53`	`53`	`$classReflection = $scope->getClassReflection();`
`54`	`54`
`55`		`- if ($classReflection === null \|\| !$classReflection->isSubclassOf(TestCase::class)) {`
	`55`	`+ if ($classReflection === null \|\| !$classReflection->is(TestCase::class)) {`
`56`	`56`	`return [];`
`57`	`57`	`}`
`58`	`58`

`‎src/Rules/PHPUnit/DataProviderHelper.php`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public function getDataProviderMethods(`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`$dataProviderMethod = $this->parseDataProviderAnnotationValue($scope, $dataProviderValue);`
`79`		`- $dataProviderMethod[] = $node->getLine();`
	`79`	`+ $dataProviderMethod[] = $node->getStartLine();`
`80`	`80`
`81`	`81`	`yield $dataProviderValue => $dataProviderMethod;`
`82`	`82`	`}`
`@@ -257,7 +257,7 @@ private function parseDataProviderExternalAttribute(Attribute $attribute): ?arra`
`257`	`257`	`sprintf('%s::%s', $className, $methodNameArg->value) => [`
`258`	`258`	`$dataProviderClassReflection,`
`259`	`259`	`$methodNameArg->value,`
`260`		`- $attribute->getLine(),`
	`260`	`+ $attribute->getStartLine(),`
`261`	`261`	`],`
`262`	`262`	`];`
`263`	`263`	`}`
`@@ -279,7 +279,7 @@ private function parseDataProviderAttribute(Attribute $attribute, ClassReflectio`
`279`	`279`	`$methodNameArg->value => [`
`280`	`280`	`$classReflection,`
`281`	`281`	`$methodNameArg->value,`
`282`		`- $attribute->getLine(),`
	`282`	`+ $attribute->getStartLine(),`
`283`	`283`	`],`
`284`	`284`	`];`
`285`	`285`	`}`

`‎src/Rules/PHPUnit/NoMissingSpaceInClassAnnotationRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function getNodeType(): string`
`33`	`33`	`public function processNode(Node $node, Scope $scope): array`
`34`	`34`	`{`
`35`	`35`	`$classReflection = $scope->getClassReflection();`
`36`		`- if ($classReflection === null \|\| $classReflection->isSubclassOf(TestCase::class) === false) {`
	`36`	`+ if ($classReflection === null \|\| $classReflection->is(TestCase::class) === false) {`
`37`	`37`	`return [];`
`38`	`38`	`}`
`39`	`39`

`‎src/Rules/PHPUnit/NoMissingSpaceInMethodAnnotationRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function getNodeType(): string`
`33`	`33`	`public function processNode(Node $node, Scope $scope): array`
`34`	`34`	`{`
`35`	`35`	`$classReflection = $scope->getClassReflection();`
`36`		`- if ($classReflection === null \|\| $classReflection->isSubclassOf(TestCase::class) === false) {`
	`36`	`+ if ($classReflection === null \|\| $classReflection->is(TestCase::class) === false) {`
`37`	`37`	`return [];`
`38`	`38`	`}`
`39`	`39`

`‎src/Rules/PHPUnit/ShouldCallParentMethodsRule.php`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function processNode(Node $node, Scope $scope): array`
`33`	`33`	`return [];`
`34`	`34`	`}`
`35`	`35`
`36`		`- if (!$scope->getClassReflection()->isSubclassOf(TestCase::class)) {`
	`36`	`+ if (!$scope->getClassReflection()->is(TestCase::class)) {`
`37`	`37`	`return [];`
`38`	`38`	`}`
`39`	`39`

0 commit comments

Comments

(0)