Skip to content

IBX-12606: Migrated the test suites to PHPUnit 11 - #51

Merged
konradoboza merged 2 commits into
6.0from
phpunit-11
Sep 16, 2026
Merged

konradoboza merged 2 commits into
6.0from
phpunit-11

Conversation

@ViniTou

@ViniTou ViniTou commented Sep 15, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12606

Related PRs:

Important

PHPUnit 11 only. phpunit/phpunit ^11.5 moves to require, because src/ ships PHPUnit code (ContainsPath constraint, MenuAssert, AbstractTranslationCase, IbexaKernelTestCase). A consumer still on PHPUnit ^9 cannot resolve ibexa/test-core ~6.0.x-dev after this merges — merge it once the consumers have their PHPUnit 11 PRs ready (or last in the wave).

Description:

PHPUnit 9 → 11 for ibexa/test-core (Symfony 8 upgrade prerequisite; the org-wide wave migrates every 6.0 repo).

  • phpunit/phpunit: require-dev ^9require ^11.5; symfony/phpunit-bridge ^5.4^7.4.
  • phpunit.xml.dist on the 11.5 schema: <listeners> SymfonyTestsListener<extensions><bootstrap class="Symfony\Bridge\PhpUnit\SymfonyExtension"/>, cacheDirectory=".phpunit.cache" (gitignored).
  • src/contracts/.../ContainsPath::matches($other)matches(mixed $other): PHPUnit 11 typed the parent Constraint::matches() signature (a real PHPStan error, not test noise).
  • AbstractTranslationCase: the @dataProvider docblock kept for PHPUnit 9 next to the #[DataProvider] attribute is gone.
  • Tests: @dataProvider/@covers annotations → attributes (13 files); one stale phpstan-baseline.neon entry (attribute.notFound for DataProvider) pruned.

Verified: 45 tests green, 0 PHPUnit deprecations; PHPStan no errors; code style clean; rector dry-run clean. Consumer smoke test: ibexa/connector-qualifio (phpunit-11 branch) unit suite green with this branch pinned as a path repository.

For QA:

N/A — test tooling only.

Documentation:

N/A

@ViniTou ViniTou changed the title Migrated the test suites to PHPUnit 11 IBX-12606: Migrated the test suites to PHPUnit 11 Sep 15, 2026

@mikadamczyk mikadamczyk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, only the conflict needs to be resolved

@konradoboza
konradoboza force-pushed the phpunit-11 branch 2 times, most recently from 919e16d to 4c2802f Compare September 16, 2026 12:33
@konradoboza
konradoboza merged commit 7d11546 into 6.0 Sep 16, 2026
5 checks passed
@konradoboza
konradoboza deleted the phpunit-11 branch September 16, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants