Skip to content

feat: migrate tests to Testo, require PHP 8.2 - #21

Merged
roxblnfk merged 2 commits into
2.xfrom
testo
Oct 9, 2026
Merged

roxblnfk merged 2 commits into
2.xfrom
testo

Conversation

@roxblnfk

@roxblnfk roxblnfk commented Oct 9, 2026

Copy link
Copy Markdown
Member

What was changed

  • Minimum PHP is now 8.2; vimeo/psalm is ^6 || ^7 so it installs on the whole supported range.
  • Tests run on Testo instead of PHPUnit; phpunit/phpunit and phpunit.xml are gone, composer test runs testo.
  • Mocks stay on Mockery via testo/bridge-mockery. The plugin verifies Mockery expectations after each test, which the PHPUnit suite never did (no Mockery::close()), so the once()/withArgs() expectations are now actually checked.
  • CI calls spiral/gh-actions testo.yml (feat(testo): test PHP 8.2-8.6 by default, add test-server input spiral/gh-actions#10): PHP 8.2–8.6, highest dependencies.

Why?

Testo requires PHP 8.2; moving the suite to it is the first step before coverage, mutation testing and the static analysis pass.

Checklist

  • Tested
    • Testo suite run locally: 9 tests before (PHPUnit) and after (Testo), the same methods

Testo, which the test suite moves to next, needs PHP 8.2. Psalm is constrained to ^6 || ^7 so that it installs on the whole supported PHP range.

Assisted-By: Claude Opus 5.5
Mocks stay on Mockery: Double needs PHP 8.3, and the Testo Mockery plugin now verifies expectations after every test, which the PHPUnit suite never did (Mockery::close() was not called). The empty TestCase base is gone since Testo discovers tests by attribute.

Assisted-By: Claude Opus 5.5
@roxblnfk
roxblnfk requested a review from a team October 9, 2026 20:18
@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2deede85-0add-45b9-838f-95937f5d2a80

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@roxblnfk
roxblnfk merged commit a870150 into 2.x Oct 9, 2026
8 of 9 checks passed
@roxblnfk
roxblnfk deleted the testo branch October 9, 2026 20:20
This was referenced Oct 9, 2026
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.

1 participant