Skip to content

feat: Add domain exceptions to skip or cancel test#191

Merged
roxblnfk merged 1 commit into
1.xfrom
skipped-cancelled
May 27, 2026
Merged

feat: Add domain exceptions to skip or cancel test#191
roxblnfk merged 1 commit into
1.xfrom
skipped-cancelled

Conversation

@roxblnfk
Copy link
Copy Markdown
Member

What was changed

Added domain exceptions to skip or cancel test

Checklist

  • Tested
    • Unit tests added

@roxblnfk roxblnfk requested a review from a team as a code owner May 27, 2026 10:46
@roxblnfk roxblnfk requested a review from Copilot May 27, 2026 10:46
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds status-bearing domain exceptions so tests can explicitly end as skipped or cancelled, and documents the behavior for Testo users and plugin authors.

Changes:

  • Adds SkipTest / CancelTest exceptions and maps them in TestRunner to Status::Skipped / Status::Cancelled.
  • Adds feature coverage using a real runner and a stub suite.
  • Updates Testo skills documentation and contributor guidance for the new public behavior.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
core/Core/Exception/SkipTest.php Adds skip marker exception.
core/Core/Exception/CancelTest.php Adds cancellation marker exception.
core/Application/Internal/Runner/TestRunner.php Maps marker exceptions to non-error statuses.
core/Core/Context/CaseInfo.php Renames test execution callback from invoker to handler.
core/Application/Internal/Runner/SuiteRunner.php Updates CaseInfo construction for renamed callback.
tests/Application/suites.php Adds Application feature test suite.
tests/Application/Stub/Runner/StatusFromException.php Adds runner stub cases for exception status mapping.
tests/Application/Feature/Runner/StatusFromExceptionTest.php Adds feature tests for skip/cancel/error mapping.
skills/testo-write-tests/SKILL.md Documents skip/cancel usage in tests.
skills/testo-plugin-author/SKILL.md Documents interceptor behavior for skip/cancel.
skills/testo-migrate-from-phpunit/SKILL.md Adds PHPUnit migration guidance for skipped/incomplete tests.
skills/testo-flaky-tests/SKILL.md Notes retry/repeat behavior with skip/cancel.
skills/README.md Adds skills index and source-of-truth notes.
CLAUDE.md Adds contributor guidance to update skills with public API changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread core/Core/Context/CaseInfo.php
Comment thread core/Core/Context/CaseInfo.php
Comment thread core/Core/Context/CaseInfo.php
@roxblnfk roxblnfk merged commit 6d800e0 into 1.x May 27, 2026
20 checks passed
@roxblnfk roxblnfk deleted the skipped-cancelled branch May 27, 2026 10:55
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.

2 participants