Skip to content

Conversation

@StephenHodgson
Copy link
Member

  • add additional unity utp log handling

- add additional unity utp log handling
@StephenHodgson StephenHodgson marked this pull request as ready for review December 20, 2025 21:38
Copilot AI review requested due to automatic review settings December 20, 2025 21:38
Copy link

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

This PR adds enhanced Unity Test Platform (UTP) log handling capabilities to unity-cli version 1.8.2. The changes introduce test scenario files that intentionally generate various types of failures and warnings to validate the CLI's error handling and logging infrastructure.

  • Adds six Unity test scenario files covering compiler errors/warnings, build errors/warnings, and playmode/editmode test failures
  • Implements UTP log artifact upload in the CI workflow to capture test execution logs
  • Updates package version from 1.8.1 to 1.8.2 with related dependency bumps

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
unity-tests/PlaymodeTestsErrors.cs Playmode test that intentionally fails to validate test failure reporting
unity-tests/EditmodeTestsErrors.cs Editmode test that intentionally fails to validate test failure reporting
unity-tests/CompilerWarnings.cs Generates obsolete API warnings for compiler warning scenario testing
unity-tests/CompilerErrors.cs Uses #error directive to force compilation errors for error scenario testing
unity-tests/BuildWarnings.cs Build preprocessor that emits warnings during Unity build pipeline
unity-tests/BuildErrors.cs Build preprocessor that throws exceptions to simulate build failures
package.json Version bump to 1.8.2
package-lock.json Version update and automatic dependency updates for baseline-browser-mapping, caniuse-lite, dedent, and update-browserslist-db
.github/workflows/build-options.json Adds tests array listing all new test scenario files
.github/workflows/unity-build.yml Adds UTP log artifact upload step and removes trailing whitespace
Comments suppressed due to low confidence (1)

unity-tests/CompilerWarnings.cs:12

            ObsoleteApi(); // CS0618: call to obsolete member

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

@StephenHodgson StephenHodgson marked this pull request as draft December 22, 2025 00:33
@StephenHodgson StephenHodgson marked this pull request as ready for review December 22, 2025 00:33
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