Skip to content

[Form] Make FormPerformanceTestCase compatible with PHPUnit 10 #51881

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 7, 2023

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Oct 6, 2023

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Part of #49069
License MIT

In PHPUnit 10, TestCase::runTest() has a return type of mixed. I'm adding the return type conditionally on the one class where we override this method.

@carsonbot carsonbot added this to the 5.4 milestone Oct 6, 2023
@derrabus derrabus added Form and removed Bug labels Oct 6, 2023
@carsonbot carsonbot changed the title Make FormPerformanceTestCase compatible with PHPUnit 10 [Form] Make FormPerformanceTestCase compatible with PHPUnit 10 Oct 6, 2023
@derrabus derrabus added the Bug label Oct 6, 2023
@fabpot
Copy link
Member

fabpot commented Oct 7, 2023

Thank you @derrabus.

@fabpot fabpot merged commit 7aaa92a into symfony:5.4 Oct 7, 2023
@derrabus derrabus deleted the bugfix/form-performance-phpunit-10 branch October 7, 2023 07:12
xabbuh added a commit that referenced this pull request Oct 9, 2023
This PR was merged into the 7.0 branch.

Discussion
----------

[Form] Remove RunTestTrait

| Q             | A
| ------------- | ---
| Branch?       | 7.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Follows #51881
| License       | MIT

On 7.0, we can add the new return type directly and remove the trait introduced with #51881.

Commits
-------

3f36287 Remove RunTestTrait
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy