From dc967efa34109cfafcce6b4ad46536bcdb998601 Mon Sep 17 00:00:00 2001
From: Aleksei Kankov <akankov@gmail.com>
Date: Thu, 14 May 2026 21:04:50 +0200
Subject: [PATCH 1/2] [PHPUnit12] Add fixture reproducing AllowMockObjects
 false positive on ConstraintValidatorTestCase

Refs rectorphp/rector#9650
---
 .../skip_constraint_validator_test_case.php.inc        | 10 ++++++++++
 1 file changed, 10 insertions(+)
 create mode 100644 rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case.php.inc

diff --git a/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case.php.inc b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case.php.inc
new file mode 100644
index 00000000..2c5393a1
--- /dev/null
+++ b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case.php.inc
@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Class_\AllowMockObjectsWhereParentClassRector\Fixture;
+
+use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+
+final class SkipConstraintValidatorTestCase extends ConstraintValidatorTestCase
+{
+    private \PHPUnit\Framework\MockObject\MockObject $someMock;
+}

From e93ad3f28ff98a4f204a17a4cbf085db8e42d0d4 Mon Sep 17 00:00:00 2001
From: Aleksei Kankov <akankov@gmail.com>
Date: Thu, 14 May 2026 21:04:50 +0200
Subject: [PATCH 2/2] [PHPUnit12] Drop ConstraintValidatorTestCase from
 AllowMockObjectsWhereParentClassRector triggers

The Symfony fix in symfony/symfony#62745 (shipped in symfony/validator
6.4.31 / 7.3.9 / 7.4.3 / 8.0.3) removed the underlying need for the
attribute on this lineage. Rule now fires only on TypeTestCase.

Includes: code sample updated to TypeTestCase, orphaned enum constant
removed, existing fixture converted to TypeTestCase, stub added so the
converted fixture autoloads, and a transitive-inheritance skip fixture.

Closes rectorphp/rector#9650
---
 ...nt_validator_test_case_via_intermediate.php.inc | 14 ++++++++++++++
 .../Fixture/some_class.php.inc                     |  8 ++++----
 .../AllowMockObjectsWhereParentClassRector.php     | 13 +++++--------
 src/Enum/PHPUnitClassName.php                      |  2 --
 stubs/Symfony/Component/Form/Test/TypeTestCase.php |  9 +++++++++
 5 files changed, 32 insertions(+), 14 deletions(-)
 create mode 100644 rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case_via_intermediate.php.inc
 create mode 100644 stubs/Symfony/Component/Form/Test/TypeTestCase.php

diff --git a/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case_via_intermediate.php.inc b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case_via_intermediate.php.inc
new file mode 100644
index 00000000..ae71a3ac
--- /dev/null
+++ b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/skip_constraint_validator_test_case_via_intermediate.php.inc
@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Class_\AllowMockObjectsWhereParentClassRector\Fixture;
+
+use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+
+abstract class IntermediateConstraintValidatorBase extends ConstraintValidatorTestCase
+{
+}
+
+final class SkipConstraintValidatorTestCaseViaIntermediate extends IntermediateConstraintValidatorBase
+{
+    private \PHPUnit\Framework\MockObject\MockObject $someMock;
+}
diff --git a/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/some_class.php.inc b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/some_class.php.inc
index f796ef81..d8720b45 100644
--- a/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/some_class.php.inc
+++ b/rules-tests/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector/Fixture/some_class.php.inc
@@ -2,9 +2,9 @@
 
 namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Class_\AllowMockObjectsWhereParentClassRector\Fixture;
 
-use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+use Symfony\Component\Form\Test\TypeTestCase;
 
-final class SomeClass extends ConstraintValidatorTestCase
+final class SomeClass extends TypeTestCase
 {
     private \PHPUnit\Framework\MockObject\MockObject $someMock;
 }
@@ -15,10 +15,10 @@ final class SomeClass extends ConstraintValidatorTestCase
 
 namespace Rector\PHPUnit\Tests\PHPUnit120\Rector\Class_\AllowMockObjectsWhereParentClassRector\Fixture;
 
-use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+use Symfony\Component\Form\Test\TypeTestCase;
 
 #[\PHPUnit\Framework\Attributes\AllowMockObjectsWithoutExpectations]
-final class SomeClass extends ConstraintValidatorTestCase
+final class SomeClass extends TypeTestCase
 {
     private \PHPUnit\Framework\MockObject\MockObject $someMock;
 }
diff --git a/rules/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector.php b/rules/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector.php
index e354c8af..a091dd81 100644
--- a/rules/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector.php
+++ b/rules/PHPUnit120/Rector/Class_/AllowMockObjectsWhereParentClassRector.php
@@ -30,10 +30,7 @@ final class AllowMockObjectsWhereParentClassRector extends AbstractRector
     /**
      * @var string[]
      */
-    private const array PARENT_CLASSES = [
-        PHPUnitClassName::SYMFONY_CONSTRAINT_VALIDATOR_TEST_CASE,
-        PHPUnitClassName::SYMFONY_TYPE_TEST_CASE,
-    ];
+    private const array PARENT_CLASSES = [PHPUnitClassName::SYMFONY_TYPE_TEST_CASE];
 
     public function __construct(
         private readonly TestsNodeAnalyzer $testsNodeAnalyzer,
@@ -75,18 +72,18 @@ public function getRuleDefinition(): RuleDefinition
             [
                 new CodeSample(
                     <<<'CODE_SAMPLE'
-use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+use Symfony\Component\Form\Test\TypeTestCase;
 
-final class SomeTest extends ConstraintValidatorTestCase
+final class SomeTest extends TypeTestCase
 {
 }
 CODE_SAMPLE
                     ,
                     <<<'CODE_SAMPLE'
-use Symfony\Component\Validator\Test\ConstraintValidatorTestCase;
+use Symfony\Component\Form\Test\TypeTestCase;
 
 #[\PHPUnit\Framework\Attributes\AllowMockObjectsWithoutExpectations]
-final class SomeTest extends ConstraintValidatorTestCase
+final class SomeTest extends TypeTestCase
 {
 }
 CODE_SAMPLE
diff --git a/src/Enum/PHPUnitClassName.php b/src/Enum/PHPUnitClassName.php
index d5c9e555..6aad1917 100644
--- a/src/Enum/PHPUnitClassName.php
+++ b/src/Enum/PHPUnitClassName.php
@@ -27,8 +27,6 @@ final class PHPUnitClassName
 
     public const string STUB = 'PHPUnit\Framework\MockObject\Stub';
 
-    public const string SYMFONY_CONSTRAINT_VALIDATOR_TEST_CASE = 'Symfony\Component\Validator\Test\ConstraintValidatorTestCase';
-
     public const string SYMFONY_TYPE_TEST_CASE = 'Symfony\Component\Form\Test\TypeTestCase';
 
     /**
diff --git a/stubs/Symfony/Component/Form/Test/TypeTestCase.php b/stubs/Symfony/Component/Form/Test/TypeTestCase.php
new file mode 100644
index 00000000..312dc61e
--- /dev/null
+++ b/stubs/Symfony/Component/Form/Test/TypeTestCase.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Symfony\Component\Form\Test;
+
+use PHPUnit\Framework\TestCase;
+
+abstract class TypeTestCase extends TestCase
+{
+}