From 1a89325fd56b1b72e181a853e62181c15b2f14b9 Mon Sep 17 00:00:00 2001 From: Tomas Votruba Date: Sat, 1 Aug 2026 10:11:50 +0200 Subject: [PATCH] [DeadCode] Remove whole assign statement on unused createStub() in tests --- .../Fixture/create_stub_in_test.php.inc | 30 +++++++++++++++++++ .../SideEffect/SideEffectNodeDetector.php | 2 +- 2 files changed, 31 insertions(+), 1 deletion(-) create mode 100644 rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Fixture/create_stub_in_test.php.inc diff --git a/rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Fixture/create_stub_in_test.php.inc b/rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Fixture/create_stub_in_test.php.inc new file mode 100644 index 00000000000..c893c833c4b --- /dev/null +++ b/rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Fixture/create_stub_in_test.php.inc @@ -0,0 +1,30 @@ +createStub('SomeClass'); + } +} + +?> +----- + diff --git a/rules/DeadCode/SideEffect/SideEffectNodeDetector.php b/rules/DeadCode/SideEffect/SideEffectNodeDetector.php index 6e7b83276d6..5e3c60a3b0f 100644 --- a/rules/DeadCode/SideEffect/SideEffectNodeDetector.php +++ b/rules/DeadCode/SideEffect/SideEffectNodeDetector.php @@ -96,7 +96,7 @@ private function isTestMock(MethodCall|StaticCall $node): bool return false; } - return $this->nodeNameResolver->isName($node->name, 'createMock'); + return $this->nodeNameResolver->isNames($node->name, ['createMock', 'createStub']); } private function isPhpParser(New_ $new): bool