[CodeQuality] Add AddStubIntersectionVarToStubPropertyRector

config/sets/phpunit-code-quality.php

@@ -6,6 +6,7 @@
 use Rector\PHPUnit\CodeQuality\Rector\CallLike\DirectInstanceOverMockArgRector;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\AddParamTypeFromDependsRector;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\AddReturnTypeToDependedRector;
+use Rector\PHPUnit\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\ConstructClassMethodToSetUpTestCaseRector;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\InlineStubPropertyToCreateStubMethodCallRector;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\NarrowUnusedSetUpDefinedPropertyRector;
@@ -57,7 +58,6 @@
 use Rector\PHPUnit\PHPUnit120\Rector\CallLike\CreateStubOverCreateMockArgRector;
 use Rector\PHPUnit\PHPUnit120\Rector\Class_\PropertyCreateMockToCreateStubRector;
 use Rector\PHPUnit\PHPUnit120\Rector\ClassMethod\ExpressionCreateMockToCreateStubRector;
-use Rector\PHPUnit\PHPUnit120\Rector\Property\BareVarToStubIntersectionRector;
 use Rector\PHPUnit\PHPUnit120\Rector\Property\MockObjectVarToStubRector;
 use Rector\PHPUnit\PHPUnit60\Rector\MethodCall\GetMockBuilderGetMockToCreateMockRector;
 use Rector\PHPUnit\PHPUnit90\Rector\MethodCall\ReplaceAtMethodWithDesiredMatcherRector;
@@ -139,7 +139,7 @@
         ExpressionCreateMockToCreateStubRector::class,
         PropertyCreateMockToCreateStubRector::class,
         MockObjectVarToStubRector::class,
-        BareVarToStubIntersectionRector::class,
+        AddStubIntersectionVarToStubPropertyRector::class,
         InlineStubPropertyToCreateStubMethodCallRector::class,
 
         // @test first, enable later

config/sets/phpunit-mock-to-stub.php

@@ -4,11 +4,11 @@
 
 use Rector\Config\RectorConfig;
 use Rector\PHPUnit\CodeQuality\Rector\Class_\AddIntersectionVarToMockObjectPropertyRector;
+use Rector\PHPUnit\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector;
 use Rector\PHPUnit\PHPUnit120\Rector\CallLike\CreateStubInCoalesceArgRector;
 use Rector\PHPUnit\PHPUnit120\Rector\CallLike\CreateStubOverCreateMockArgRector;
 use Rector\PHPUnit\PHPUnit120\Rector\Class_\PropertyCreateMockToCreateStubRector;
 use Rector\PHPUnit\PHPUnit120\Rector\ClassMethod\ExpressionCreateMockToCreateStubRector;
-use Rector\PHPUnit\PHPUnit120\Rector\Property\BareVarToStubIntersectionRector;
 use Rector\PHPUnit\PHPUnit120\Rector\Property\MockObjectVarToStubRector;
 
 return static function (RectorConfig $rectorConfig): void {
@@ -19,7 +19,7 @@
         ExpressionCreateMockToCreateStubRector::class,
         PropertyCreateMockToCreateStubRector::class,
         MockObjectVarToStubRector::class,
-        BareVarToStubIntersectionRector::class,
         AddIntersectionVarToMockObjectPropertyRector::class,
+        AddStubIntersectionVarToStubPropertyRector::class,
     ]);
 };

rules-tests/CodeQuality/Rector/Class_/AddIntersectionVarToMockObjectPropertyRector/Fixture/skip_existing_intersection_var.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddIntersectionVarToMockObjectPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipExistingIntersectionVarTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\MockObject&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\MockObject $someServiceMock;
+
+    protected function setUp(): void
+    {
+        $this->someServiceMock = $this->createMock(\stdClass::class);
+    }
+}

rules-tests/CodeQuality/Rector/Class_/AddIntersectionVarToMockObjectPropertyRector/Fixture/static_call.php.inc

@@ -0,0 +1,38 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddIntersectionVarToMockObjectPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class StaticCallTest extends TestCase
+{
+    private \PHPUnit\Framework\MockObject\MockObject $someServiceMock;
+
+    protected function setUp(): void
+    {
+        $this->someServiceMock = self::createMock(\stdClass::class);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddIntersectionVarToMockObjectPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class StaticCallTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\MockObject&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\MockObject $someServiceMock;
+
+    protected function setUp(): void
+    {
+        $this->someServiceMock = self::createMock(\stdClass::class);
+    }
+}
+
+?>

rules-tests/PHPUnit120/Rector/Property/BareVarToStubIntersectionRector/BareVarToStubIntersectionRectorTest.php → rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/AddStubIntersectionVarToStubPropertyRectorTest.php

@@ -2,13 +2,13 @@
 
 declare(strict_types=1);
 
-namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Property\BareVarToStubIntersectionRector;
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector;
 
 use Iterator;
 use PHPUnit\Framework\Attributes\DataProvider;
 use Rector\Testing\PHPUnit\AbstractRectorTestCase;
 
-final class BareVarToStubIntersectionRectorTest extends AbstractRectorTestCase
+final class AddStubIntersectionVarToStubPropertyRectorTest extends AbstractRectorTestCase
 {
     #[DataProvider('provideData')]
     public function test(string $filePath): void

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/bare_var_without_setup.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class BareVarWithoutSetupTest extends TestCase
+{
+    /**
+     * @var FormBuilderInterface
+     */
+    private \PHPUnit\Framework\MockObject\Stub $formBuilder;
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class BareVarWithoutSetupTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\Stub&FormBuilderInterface
+     */
+    private \PHPUnit\Framework\MockObject\Stub $formBuilder;
+}
+
+?>

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/fixture.php.inc

@@ -0,0 +1,38 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SomeTest extends TestCase
+{
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SomeTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\Stub&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/overwrite_existing_var.php.inc

@@ -0,0 +1,41 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class OverwriteExistingVarTest extends TestCase
+{
+    /**
+     * @var \stdClass
+     */
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class OverwriteExistingVarTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\Stub&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/skip_existing_intersection_var.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipExistingIntersectionVarTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\Stub&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/skip_mock_object_property.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipMockObjectPropertyTest extends TestCase
+{
+    private \PHPUnit\Framework\MockObject\MockObject $someServiceMock;
+
+    protected function setUp(): void
+    {
+        $this->someServiceMock = $this->createStub(\stdClass::class);
+    }
+}

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/skip_non_test_class.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+final class SkipNonTestClass
+{
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    public function setUp(): void
+    {
+        $this->someServiceStub = $this->createStub(\stdClass::class);
+    }
+}

rules-tests/PHPUnit120/Rector/Property/BareVarToStubIntersectionRector/Fixture/skip_stub_short_name.php.inc → rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/skip_stub_short_name_var.php.inc

@@ -1,11 +1,11 @@
 <?php
 
-namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Property\BareVarToStubIntersectionRector\Fixture;
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
 
 use PHPUnit\Framework\MockObject\Stub;
 use PHPUnit\Framework\TestCase;
 
-final class SkipStubShortNameTest extends TestCase
+final class SkipStubShortNameVarTest extends TestCase
 {
     /**
      * @var Stub

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/Fixture/static_call.php.inc

@@ -0,0 +1,38 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class StaticCallTest extends TestCase
+{
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = self::createStub(\stdClass::class);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\Class_\AddStubIntersectionVarToStubPropertyRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class StaticCallTest extends TestCase
+{
+    /**
+     * @var \PHPUnit\Framework\MockObject\Stub&\stdClass
+     */
+    private \PHPUnit\Framework\MockObject\Stub $someServiceStub;
+
+    protected function setUp(): void
+    {
+        $this->someServiceStub = self::createStub(\stdClass::class);
+    }
+}
+
+?>

rules-tests/CodeQuality/Rector/Class_/AddStubIntersectionVarToStubPropertyRector/config/configured_rule.php

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