build(deps-dev): bump phpunit/phpunit from 9.5.28 to 13.1.8#355
build(deps-dev): bump phpunit/phpunit from 9.5.28 to 13.1.8#355dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [phpunit/phpunit](https://github.com/sebastianbergmann/phpunit) from 9.5.28 to 13.1.8. - [Release notes](https://github.com/sebastianbergmann/phpunit/releases) - [Changelog](https://github.com/sebastianbergmann/phpunit/blob/13.1.8/ChangeLog-13.1.md) - [Commits](sebastianbergmann/phpunit@9.5.28...13.1.8) --- updated-dependencies: - dependency-name: phpunit/phpunit dependency-version: 13.1.8 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 across four major versions (9 to 13) without any corresponding updates to the test codebase or project configuration. This update is highly likely to break the build for two reasons: the strict requirement for PHP 8.4.1+, which is not currently defined in composer.json, and the extensive breaking changes in PHPUnit's internal API (including the move from annotations to attributes and the new Event system).
Although Codacy marks the PR as 'up to standards', this is likely due to the analysis being restricted to the dependency files themselves. This PR should not be merged until the environment compatibility is confirmed and the test suite has been migrated to handle the removed assertions and updated configuration schema.
About this PR
- The minimum requirement of PHP 8.4.1 for PHPUnit 13.1 may unexpectedly break development environments or CI pipelines that are not yet on the latest PHP release, as the project does not explicitly define its own PHP requirement in
composer.json. - The PR jumps across four major versions of PHPUnit without any adjustments to test files or configuration. Given the breaking changes introduced in PHPUnit 10 and later, it is highly probable that the current tests will fail to execute without a significant refactoring effort.
Test suggestions
- Execute the existing test suite using PHPUnit 13 to detect breakages caused by API removals or behavior changes (e.g. transition from annotations to attributes).
- Verify that the CI/CD pipeline and local environments support PHP 8.4.1, as necessitated by this version of PHPUnit.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Execute the existing test suite using PHPUnit 13 to detect breakages caused by API removals or behavior changes (e.g. transition from annotations to attributes).
2. Verify that the CI/CD pipeline and local environments support PHP 8.4.1, as necessitated by this version of PHPUnit.
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 version jump (9 to 13) introduces a requirement for PHP >= 8.4.1 and many breaking changes.
- Explicitly declare the minimum PHP version requirement in the
requiresection ofcomposer.json(>= 8.4.1). - Update
phpunit.xmlby runningvendor/bin/phpunit --migrate-configuration. - Migrate test files to use Attributes instead of Docblock annotations and replace removed assertions like
assertRegExporassertInternalType.
Try running the following prompt in your coding agent:
Update composer.json to require php 8.4.1 or higher and migrate the tests in the tests/ directory to be compatible with PHPUnit 13 attributes and configuration.
|
Superseded by #358. |
Bumps phpunit/phpunit from 9.5.28 to 13.1.8.
Release notes
Sourced from phpunit/phpunit's releases.
... (truncated)
Changelog
Sourced from phpunit/phpunit's changelog.
... (truncated)
Commits
f49a2b5Prepare release423461fMerge branch '12.5' into 13.1d75dd30Prepare release451a35bMerge branch '12.5' into 13.1419e088Closes #6599172f1cdCloses #66038a8e7bbMerge branch '12.5' into 13.1f86f987Merge branch '11.5' into 12.5078b608Update toolsfebf07fIncrease timeoutDependabot 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)