build(deps-dev): bump phpunit/phpunit from 9.5.28 to 13.1.12#361
build(deps-dev): bump phpunit/phpunit from 9.5.28 to 13.1.12#361dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 9.5.28 to 13.1.12. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/13.1.12/ChangeLog-13.1.md) - [Commits](sebastianbergmann/phpunit@9.5.28...13.1.12) --- updated-dependencies: - dependency-name: phpunit/phpunit dependency-version: 13.1.12 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades phpunit/phpunit from version 9.5.28 to 13.1.12. This is a massive major version jump that spans several generations of the framework. PHPUnit 10+ introduced a completely new event system, removed legacy methods (such as expectError() and expectWarning()), and changed the XML configuration schema.
In its current state, the PR is likely to break the build because it lacks the accompanying migration for phpunit.xml and the transition from docblock annotations to PHP attributes in the test suite. These gaps should be addressed before merging to ensure compatibility with the new version.
About this PR
- Updating PHPUnit from v9 to v13 without performing a migration of the test suite and configuration will cause immediate failure in any environment running these tests. The transition requires migrating the
phpunit.xmlschema and updating test metadata from docblocks to PHP 8 attributes.
Test suggestions
- Execute the full test suite to ensure compatibility with PHPUnit 13 breaking changes.
- Verify that the
phpunit.xmlconfiguration file is migrated to the new schema required for PHPUnit 10 and above.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Execute the full test suite to ensure compatibility with PHPUnit 13 breaking changes.
2. Verify that the `phpunit.xml` configuration file is migrated to the new schema required for PHPUnit 10 and above.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
| }, | ||
| "require-dev": { | ||
| "phpunit/phpunit": "^9.0", | ||
| "phpunit/phpunit": "^13.1", |
There was a problem hiding this comment.
🔴 HIGH RISK
This major upgrade requires a complete migration of the phpunit.xml configuration and test metadata which is missing from this PR. PHPUnit 10 and above requires a different configuration schema and encourages a shift from docblock annotations to PHP attributes. It is recommended to use a migration tool or a coding agent to convert the test suite and configuration file to be compatible with version 10+.
|
Superseded by #362. |
Bumps phpunit/phpunit from 9.5.28 to 13.1.12.
Release notes
Sourced from phpunit/phpunit's releases.
... (truncated)
Changelog
Sourced from phpunit/phpunit's changelog.
... (truncated)
Commits
ca98c9fPrepare released8d329dMerge branch '12.5' into 13.1f37c01ePrepare release76ced3fMerge branch '12.5' into 13.15426398Add end-to-end test for forwarding of non-runtime-changeable INI settings to ...85d1ce8Merge branch '12.5' into 13.1c1a8c8aImprove check for a usable code coverage driver in PHPT skip conditionsd9baa1fMerge branch '12.5' into 13.12ca6914Update dependencies3046e58Merge branch '12.5' into 13.1Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)