Assert*Rules: Do cheap checks first

Author: staabm

src/Rules/PHPUnit/AssertEqualsIsDiscouragedRule.php

@@ -27,24 +27,24 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
-		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+		if (count($node->getArgs()) < 2) {
 			return [];
 		}
-
 		if ($node->isFirstClassCallable()) {
 			return [];
 		}
 
-		if (count($node->getArgs()) < 2) {
-			return [];
-		}
 		if (
 			!$node->name instanceof Node\Identifier
 			|| !in_array(strtolower($node->name->name), ['assertequals', 'assertnotequals'], true)
 		) {
 			return [];
 		}
 
+		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+			return [];
+		}
+
 		$leftType = TypeCombinator::removeNull($scope->getType($node->getArgs()[0]->value));
 		$rightType = TypeCombinator::removeNull($scope->getType($node->getArgs()[1]->value));
 

src/Rules/PHPUnit/AssertSameBooleanExpectedRule.php

@@ -23,17 +23,12 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
-		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+		if (count($node->getArgs()) < 2) {
 			return [];
 		}
-
 		if ($node->isFirstClassCallable()) {
 			return [];
 		}
-
-		if (count($node->getArgs()) < 2) {
-			return [];
-		}
 		if (!$node->name instanceof Node\Identifier || $node->name->toLowerString() !== 'assertsame') {
 			return [];
 		}
@@ -43,6 +38,10 @@ public function processNode(Node $node, Scope $scope): array
 			return [];
 		}
 
+		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+			return [];
+		}
+
 		if ($expectedArgumentValue->name->toLowerString() === 'true') {
 			return [
 				RuleErrorBuilder::message('You should use assertTrue() instead of assertSame() when expecting "true"')->identifier('phpunit.assertTrue')->build(),

src/Rules/PHPUnit/AssertSameNullExpectedRule.php

@@ -23,18 +23,17 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
-		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+		if (count($node->getArgs()) < 2) {
 			return [];
 		}
-
 		if ($node->isFirstClassCallable()) {
 			return [];
 		}
-
-		if (count($node->getArgs()) < 2) {
+		if (!$node->name instanceof Node\Identifier || $node->name->toLowerString() !== 'assertsame') {
 			return [];
 		}
-		if (!$node->name instanceof Node\Identifier || $node->name->toLowerString() !== 'assertsame') {
+
+		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
 			return [];
 		}
 

src/Rules/PHPUnit/AssertSameWithCountRule.php

@@ -24,23 +24,21 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
-		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
+		if (count($node->getArgs()) < 2) {
 			return [];
 		}
-
 		if ($node->isFirstClassCallable()) {
 			return [];
 		}
-
-		if (count($node->getArgs()) < 2) {
+		if (!$node->name instanceof Node\Identifier || $node->name->toLowerString() !== 'assertsame') {
 			return [];
 		}
-		if (!$node->name instanceof Node\Identifier || $node->name->toLowerString() !== 'assertsame') {
+
+		if (!AssertRuleHelper::isMethodOrStaticCallOnAssert($node, $scope)) {
 			return [];
 		}
 
 		$right = $node->getArgs()[1]->value;
-
 		if (
 			$right instanceof Node\Expr\FuncCall
 			&& $right->name instanceof Node\Name