Skip to content
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

IsUnitTestTrait: update test class list #2050

Merged
merged 1 commit into from
May 16, 2022

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented May 15, 2022

Update the test class list based on changes in WP Core:

  • The Block_Supported_Styles_Test was never a TestCase, but was a concrete test placed in the wrong directory.
  • The PHPUnit_Adapter_TestCase class was added to allow for PHPUnit cross-version support.

Other than that, the list is now alphabetized within each category and the categories are annotated with comments.

@jrfnl
Copy link
Member Author

jrfnl commented May 16, 2022

@GaryJones Just checking - any particular reason why you didn't merge this after approving (as the second approver) ? Same for the other open PR.

@jrfnl
Copy link
Member Author

jrfnl commented May 16, 2022

FYI: As I've now changed the required statuses, I will rebase this PR (and the other open one) without changes, so the required statuses pass again.

Update the test class list based on changes in WP Core:
* The `Block_Supported_Styles_Test` was never a `TestCase`, but was a concrete test placed in the wrong directory.
* The `PHPUnit_Adapter_TestCase` class was added to allow for PHPUnit cross-version support.

Other than that, the list is now alphabetized within each category and the categories are annotated with comments.
@jrfnl jrfnl force-pushed the feature/unittesttrait-update-class-list branch from a4c2f90 to 20d906d Compare May 16, 2022 11:40
@dingo-d dingo-d merged commit 8ea3a0a into develop May 16, 2022
@dingo-d dingo-d deleted the feature/unittesttrait-update-class-list branch May 16, 2022 12:40
@jrfnl jrfnl mentioned this pull request Aug 20, 2023
36 tasks
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.

3 participants