[terabytesoftw]

Pull Request

• Breaking change (fix or feature that would cause existing functionality to change)
• Bugfix (non-breaking change that fixes an issue)
• CI/build configuration
• Documentation update
• New feature (non-breaking change that adds functionality)
• Refactoring (no functional changes)

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Summary

Summary by CodeRabbit

• New Features
  • JUnit report output can now be configured with a file name or disabled.
• Bug Fixes
  • Additional test and PHPUnit arguments containing line breaks are handled more reliably.
  • Test-result and coverage uploads now run only when the workflow is not cancelled and the relevant settings are provided.

The Testo workflow and action now support a configurable JUnit report file. Testo and PHPUnit actions also normalize line breaks in additional arguments and require a nonempty coverage driver before coverage uploads.

Changes: Test action reporting and input handling

Layer / File(s)	Summary
Configurable Testo JUnit reporting .github/workflows/testo.yml, actions/testo/action.yml, CHANGELOG.md	The workflow exposes and passes a JUnit-file input. Linux and Windows commands add the JUnit option when the input is nonempty. Test-results upload requires a nonempty input and uploads that file.
Argument normalization and coverage upload conditions actions/testo/action.yml, actions/phpunit/action.yml	Both actions replace line breaks in additional arguments with spaces. Coverage uploads require a nonempty driver other than none, and a non-cancelled workflow.

Priority: Not assessed

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Sequence Diagram(s)

sequenceDiagram
  participant Workflow as Testo workflow
  participant Action as Testo action
  participant CLI as Testo command
  participant Upload as Test-results upload
  Workflow->>Action: pass junit-file
  Action->>CLI: add --log-junit=<file> when nonempty
  CLI->>Upload: provide the configured report file

Loading

Possibly related PRs

• php-forge/actions#88: Introduces the reusable Testo workflow and action that this change updates.

Suggested labels: enhancement

Merge Risk: Low · up to a9254

Existing workflows may change JUnit behavior or fail to publish custom reports; document the migration and keep report generation aligned with upload before merging.

Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• 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

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit sets a report file just right,
Then sends the test results into sight.
It smooths the args where newlines grew,
And checks the driver is set, too.
The hare hops off beneath the moon.}

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