[DeadCode] Add RemoveParentDelegatingClassMethodRector

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_added_native_types.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentWithDocblockTypes;
+
+final class SkipAddedNativeTypes extends SomeParentWithDocblockTypes
+{
+    public function compute(int $value): int
+    {
+        return parent::compute($value);
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_attributed_method.php.inc

@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipAttributedMethod extends SomeParentCommand
+{
+    #[\Deprecated]
+    protected function configure(): void
+    {
+        parent::configure();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_construct.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentWithConstructor;
+
+final class SkipConstruct extends SomeParentWithConstructor
+{
+    public function __construct(int $value)
+    {
+        parent::__construct($value);
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_deprecated_docblock.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipDeprecatedDocblock extends SomeParentCommand
+{
+    /**
+     * @deprecated
+     */
+    protected function configure(): void
+    {
+        parent::configure();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_different_parent_method.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipDifferentParentMethod extends SomeParentCommand
+{
+    protected function configure(): void
+    {
+        parent::anotherMethod();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_extra_stmt.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipExtraStmt extends SomeParentCommand
+{
+    protected function configure(): void
+    {
+        parent::configure();
+
+        echo 'more';
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_final_method.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+class SkipFinalMethod extends SomeParentCommand
+{
+    final protected function configure(): void
+    {
+        parent::configure();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_narrowed_param_type.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipNarrowedParamType extends SomeParentCommand
+{
+    public function process(int|float $value): int
+    {
+        return parent::process($value);
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_no_parent_class.php.inc

@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+final class SkipNoParentClass
+{
+    protected function configure(): void
+    {
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_param_docblock.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipParamDocblock extends SomeParentCommand
+{
+    /**
+     * @param positive-int $value
+     */
+    public function process(int $value): int
+    {
+        return parent::process($value);
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_private_parent_method.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentWithPrivateMethod;
+
+final class SkipPrivateParentMethod extends SomeParentWithPrivateMethod
+{
+    protected function resolve(): void
+    {
+        parent::resolve();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_swapped_args.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentWithTwoParams;
+
+final class SkipSwappedArgs extends SomeParentWithTwoParams
+{
+    public function run(int $first, int $second): int
+    {
+        return parent::run($second, $first);
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/skip_wider_visibility.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SkipWiderVisibility extends SomeParentCommand
+{
+    public function configure(): void
+    {
+        parent::configure();
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/some_command.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SomeCommand extends SomeParentCommand
+{
+    protected function configure(): void
+    {
+        parent::configure();
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class SomeCommand extends SomeParentCommand
+{
+}
+
+?>

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Fixture/with_params_and_return.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class WithParamsAndReturn extends SomeParentCommand
+{
+    public function process(int $value): int
+    {
+        return parent::process($value);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source\SomeParentCommand;
+
+final class WithParamsAndReturn extends SomeParentCommand
+{
+}
+
+?>

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/RemoveParentDelegatingClassMethodRectorTest.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class RemoveParentDelegatingClassMethodRectorTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Source/SomeParentCommand.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source;
+
+abstract class SomeParentCommand
+{
+    protected function configure(): void
+    {
+    }
+
+    public function process(int $value): int
+    {
+        return $value;
+    }
+
+    public static function make(): void
+    {
+    }
+
+    public function anotherMethod(): void
+    {
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Source/SomeParentWithConstructor.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source;
+
+abstract class SomeParentWithConstructor
+{
+    public function __construct(int $value)
+    {
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Source/SomeParentWithDocblockTypes.php

@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source;
+
+abstract class SomeParentWithDocblockTypes
+{
+    /**
+     * @param int $value
+     * @return int
+     */
+    public function compute($value)
+    {
+        return $value;
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Source/SomeParentWithPrivateMethod.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source;
+
+abstract class SomeParentWithPrivateMethod
+{
+    private function resolve(): void
+    {
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/Source/SomeParentWithTwoParams.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector\Source;
+
+abstract class SomeParentWithTwoParams
+{
+    public function run(int $first, int $second): int
+    {
+        return $first - $second;
+    }
+}

rules-tests/DeadCode/Rector/ClassMethod/RemoveParentDelegatingClassMethodRector/config/configured_rule.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\DeadCode\Rector\ClassMethod\RemoveParentDelegatingClassMethodRector;
+
+return RectorConfig::configure()
+    ->withRules([RemoveParentDelegatingClassMethodRector::class]);