Incorrect behavior of ParentTestClassConstructorRector

[gisostallenberg]

Bug Report

Subject	Details
Rector version	last dev-main
Installed as	composer dependency

Minimal PHP Code Causing Issue

See https://getrector.com/demo/919b7847-fa96-4779-8160-37d400c35745

<?php

 use PHPUnit\Framework\TestCase;

 final class SomeHelper extends TestCase
 {
 }

Responsible rules

• ParentTestClassConstructorRector

Description

Because in PHPUnit 11 the constructor of \PHPUnit\Framework\TestCase is marked final, applying this rule results in a PHPStan error 'Method SomeHelper::__construct() overrides @Final method PHPUnit\Framework\TestCase::__construct()

Could the rule detect if the parent method is final?

[gisostallenberg]

The added value of this rule is also very low. PHPUnit does not detect classes without *Test when a filter or directory is given. When a direct test on the class (file) is ran, PHPUnit executes the test cases in the class, even without the constructor.

I would suggest a rule that checks if child classes of TestCase have a Test suffix, which would add them to the detected classes. Otherwise removing this rule from (at least PHPUnit 11 set) would be better.

[samsonasik]

@TomasVotruba this is why I was suggesting to remove this rule:

• [PHPUnit 10] Drop ParentTestClassConstructorRector rector-phpunit#490

that you closed, as you want this to upgrade phpunit 9 to 10,

so we add special rule on phpunit 12 to remove override final construct:

• [PHPUnit 12] Add RemoveOverrideFinalConstructTestCaseRector  rector-phpunit#491

[TomasVotruba]

This rule should only trigger, if these is a __construct method in the helper class.

What is the expeced content per PHPUnit version from 9 through 12? The class should be always initializable via new SomeHelper()

[gisostallenberg]

This rule should only trigger, if these is a __construct method in the helper class

It now adds the __construct method if there isn't one and the class is not named *Test.

[github-actions]

This issue has been automatically marked as stale due to inactivity. It will be closed in next 30 days if no further activity occurs.