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

Updated PHPUnit to v10 #15232

Merged
merged 12 commits into from
Aug 8, 2024
Merged

Updated PHPUnit to v10 #15232

merged 12 commits into from
Aug 8, 2024

Conversation

marcusmoore
Copy link
Collaborator

Description

This is a maintenance PR that runs the Shift for upgrading to PHPUnit 10. The notes that Shift created are in my fork that Shift included because they were really noisy.

I reverted adding void to each method and making classes final.

In addition, per Shift's recommendations I ran:

  • composer update phpunit/phpunit nunomaduro/collision brianium/paratest --with-all-dependencies
  • vendor/bin/phpunit --migrate-configuration (then alphabetized the properties)

Type of change

  • Chore

@marcusmoore marcusmoore requested a review from snipe as a code owner August 6, 2024 20:54
@probot-autolabeler probot-autolabeler bot added backend dependencies Pull requests that update a dependency file ldap tests labels Aug 6, 2024
Copy link

what-the-diff bot commented Aug 6, 2024

PR Summary

  • Enhanced Git ignores
    The .gitignore file was updated to include .phpunit.cache, helping to prevent unnecessary files from being tracked by Git.

  • Upgraded Test Error Handler
    The nunomaduro/collision package (which is used for handling errors during testing) in composer.json was upgraded to a newer version (^7.0).

  • Refined PHPUnit Configuration
    The configuration file for PHPUnit, a testing tool, was tweaked. The backupStaticAttributes attribute has been renamed to backupStaticProperties and a new cacheDirectory attribute was added pointing to .phpunit.cache.

  • Improved Test Methods
    Numerous method visibility changes were made, and static modifiers were added in multiple test files (AssetCheckoutTest.php, EmailNotificationsUponCheckinTest.php, EmailNotificationsUponCheckoutTest.php, SlackNotificationsUponCheckinTest.php, and ShowSetUpPageTest.php). This increases the accessibility of these methods and improves overall testing efficiency.

  • Added Test Grouping
    A new attribute [Group('ldap')] was added to LdapTest.php, allowing for better organization and targeting of test running.

Note: The repeated information for SlackNotificationsUponCheckinTest.php appears to be a duplication and is therefore not detailed again.

@marcusmoore
Copy link
Collaborator Author

Whelp doesn't work on 8.1. Moving back to draft for now.

@marcusmoore marcusmoore marked this pull request as draft August 6, 2024 20:56
@snipe
Copy link
Owner

snipe commented Aug 6, 2024

Can't win 'em all :P

@marcusmoore marcusmoore marked this pull request as ready for review August 6, 2024 22:04
@snipe snipe merged commit cc1e356 into snipe:develop Aug 8, 2024
9 checks passed
@marcusmoore marcusmoore deleted the shift-126036 branch August 12, 2024 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend dependencies Pull requests that update a dependency file ldap tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants