Rewrite in backward compatibility compliant way

phpstan · Jun 9, 2021 · 63cec25 · 63cec25
1 parent 0208ab7
commit 63cec25
Show file tree

Hide file tree

Showing 14 changed files with 14 additions and 65 deletions.
diff --git a/tests/Rules/BooleansInConditions/BooleanInBooleanAndRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInBooleanAndRuleTest.php
@@ -15,12 +15,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInBooleanAndRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			),
 			true
 		);

diff --git a/tests/Rules/BooleansInConditions/BooleanInBooleanNotRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInBooleanNotRuleTest.php
@@ -12,12 +12,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInBooleanNotRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/BooleansInConditions/BooleanInBooleanOrRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInBooleanOrRuleTest.php
@@ -15,12 +15,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInBooleanOrRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			),
 			true
 		);

diff --git a/tests/Rules/BooleansInConditions/BooleanInElseIfConditionRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInElseIfConditionRuleTest.php
@@ -12,12 +12,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInElseIfConditionRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/BooleansInConditions/BooleanInIfConditionRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInIfConditionRuleTest.php
@@ -12,12 +12,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInIfConditionRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/BooleansInConditions/BooleanInTernaryOperatorRuleTest.php b/tests/Rules/BooleansInConditions/BooleanInTernaryOperatorRuleTest.php
@@ -12,12 +12,7 @@ protected function getRule(): Rule
 	{
 		return new BooleanInTernaryOperatorRule(
 			new BooleanRuleHelper(
-				new RuleLevelHelper(
-					$this->createBroker(),
-					true,
-					false,
-					true
-				)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandInArithmeticIncrementOrDecrementRuleTest.php b/tests/Rules/Operators/OperandInArithmeticIncrementOrDecrementRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return $this->createRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticAdditionRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticAdditionRuleTest.php
@@ -13,7 +13,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticAdditionRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticDivisionRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticDivisionRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticDivisionRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticExponentiationRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticExponentiationRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticExponentiationRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticModuloRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticModuloRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticModuloRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticMultiplicationRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticMultiplicationRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticMultiplicationRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/Operators/OperandsInArithmeticSubtractionRuleTest.php b/tests/Rules/Operators/OperandsInArithmeticSubtractionRuleTest.php
@@ -12,7 +12,7 @@ protected function getRule(): Rule
 	{
 		return new OperandsInArithmeticSubtractionRule(
 			new OperatorRuleHelper(
-				new RuleLevelHelper($this->createBroker(), true, false, true)
+				self::getContainer()->getByType(RuleLevelHelper::class)
 			)
 		);
 	}

diff --git a/tests/Rules/StrictCalls/DynamicCallOnStaticMethodsRuleTest.php b/tests/Rules/StrictCalls/DynamicCallOnStaticMethodsRuleTest.php
@@ -8,19 +8,13 @@
 class DynamicCallOnStaticMethodsRuleTest extends \PHPStan\Testing\RuleTestCase
 {
 
-	/** @var bool */
-	private $checkThisOnly;
-
 	protected function getRule(): Rule
 	{
-		$broker = $this->createBroker();
-		$ruleLevelHelper = new RuleLevelHelper($broker, true, $this->checkThisOnly, true);
-		return new DynamicCallOnStaticMethodsRule($ruleLevelHelper);
+		return new DynamicCallOnStaticMethodsRule(self::getContainer()->getByType(RuleLevelHelper::class));
 	}
 
 	public function testRule(): void
 	{
-		$this->checkThisOnly = false;
 		$this->analyse([__DIR__ . '/data/dynamic-calls-on-static-methods.php'], [
 			[
 				'Dynamic call to static method StrictCalls\ClassWithStaticMethod::foo().',
@@ -41,19 +35,4 @@ public function testRule(): void
 		]);
 	}
 
-	public function testRuleOnThisOnly(): void
-	{
-		$this->checkThisOnly = true;
-		$this->analyse([__DIR__ . '/data/dynamic-calls-on-static-methods.php'], [
-			[
-				'Dynamic call to static method StrictCalls\ClassWithStaticMethod::foo().',
-				14,
-			],
-			[
-				'Dynamic call to static method StrictCalls\ClassUsingTrait::foo().',
-				34,
-			],
-		]);
-	}
-
 }