Skip to content

Split unit and integration tests in CI and collect coverage - #29

Merged
pavog merged 1 commit into
masterfrom
align-tests-workflow
Oct 16, 2026
Merged

pavog merged 1 commit into
masterfrom
align-tests-workflow

Conversation

@pavog

@pavog pavog commented Oct 16, 2026

Copy link
Copy Markdown
Member

This PR splits the CI workflow into a unit test job and an integration test job.

Unit tests

  • Run on every push and pull request, on PHP 8.4 and 8.5.
  • PHP is set up with pcov, and the tests write a clover report to coverage.xml.

Integration tests

  • The integration tests send requests to the live Modrinth API. They now run in a separate job on PHP 8.5, only on pushes to master and when the workflow is started manually.
  • Pull requests no longer run them, so a slow or unavailable API does not block a PR.

phpunit.xml.dist

  • Coverage is only collected for src/Client. The generated code in src/Api and src/Model is not tested directly.

@pavog
pavog merged commit e6382d9 into master Oct 16, 2026
3 checks passed
@pavog
pavog deleted the align-tests-workflow branch October 16, 2026 13:05
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