Skip to content

chore: automatic test run resolving - #2786

Merged
bitsandfoxes merged 5 commits into
mainfrom
chore/headless-testing
Jul 29, 2026
Merged

chore: automatic test run resolving#2786
bitsandfoxes merged 5 commits into
mainfrom
chore/headless-testing

Conversation

@bitsandfoxes

Copy link
Copy Markdown
Contributor

This change allows running the tests to automatically resolve to either connecting to a running editor instance or to run the tests headless.

Updated the agent docs to make it clear what to pick up.
Updated the bootstrapper to notify about missing tooling.

#skip-changelog

Comment thread scripts/unity-test-cli.ps1
Comment thread scripts/unity-test-cli.ps1
@bitsandfoxes
bitsandfoxes merged commit a62823e into main Jul 29, 2026
17 of 18 checks passed
@bitsandfoxes
bitsandfoxes deleted the chore/headless-testing branch July 29, 2026 14:33

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 67220b4. Configure here.

Comment thread scripts/test-utils.ps1
Message = $msg
Name = $_.fullname
Message = $msg
StackTrace = $stackTrace

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Parent failures inflate headless output

Low Severity

Parse-TestResults collects every test-case with result='Failed', including parent nodes that wrap child cases. The new headless harness prints that list directly, so nested Unity failures can show duplicate or less useful parent entries. report-test-results.ps1 already skips those parents when rendering the same XML.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 67220b4. Configure here.

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