-
Notifications
You must be signed in to change notification settings - Fork 5
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
Change to support PHPUnit 10 #473
Conversation
Important Auto Review SkippedAuto reviews are disabled on base/target branches other than the default branch. Please add the base/target branch pattern to the list of additional branches to be reviewed in the settings. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the TipsChat with CodeRabbit Bot (
|
This reverts commit 6506aa3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GoodJobDude 🐙
@@ -7,7 +7,7 @@ | |||
use Mantle\Testing\Framework_Test_Case; | |||
use Predis\Connection\ConnectionException; | |||
|
|||
class Test_Cache_Manager extends Framework_Test_Case { | |||
class CacheManagerTest extends Framework_Test_Case { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure I like the Pascal case... I must be getting old. New things 😨
@@ -21,7 +21,7 @@ trait Example_Overload { | |||
* This method should be run during the setUpBeforeClass method of | |||
* the Test_Case. | |||
*/ | |||
public static function example_overload_set_up_before_class(): void { | |||
public static function exampleoverload_set_up_before_class(): void { | |||
static::$overloaded_methods[] = 'setUpBeforeClass'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this look weird now?
* Including support for PHPUnit 10 * Upping to nunomaduro/collision * Migrating phpunit config * Upgrading annotation reader for PHPUnit 10.x * Adding PSR-4 autoloader dev * WIP on conversion script * Work to support the PSR4 migration * Script to convert to PSR4 * Extend to collision 6 for older versions * Bump asset manager * Change to support PHPUnit 10 (#473)
Changes in the test suite to support PHPUnit 10. Broken out from #427 due to the size.