Skip to content

Fix/unity test capture - #114

Merged
kochounoyume merged 5 commits into
mainfrom
fix/unity-test-capture
Jul 16, 2026
Merged

Fix/unity test capture#114
kochounoyume merged 5 commits into
mainfrom
fix/unity-test-capture

Conversation

@syogakusya

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 16, 2026 15:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates Unity UI Toolkit playmode tests to better validate user interaction and improve screenshot capture stability, while also refactoring input fixture usage in the shared test base.

Changes:

  • Add toggle state assertions in UssTest to confirm clicks are actually registered before capturing screenshots.
  • Refactor TestBase to use a private InputTestFixture instance and update input helpers accordingly.
  • Adjust hard-coded click Y coordinates in tests to match the expected UI layout.

Reviewed changes

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

File Description
Assets/Tests/UssTest.cs Adds UI interaction assertions before screenshots; updates click coordinates; minor screenshot helper edits.
Assets/Tests/TestBase.cs Refactors InputTestFixture field usage and updates helper methods.
Assets/Tests/CustomElementTest.cs Updates click coordinates used by console view tests.

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

Comment thread Assets/Tests/UssTest.cs
Comment thread Assets/Tests/UssTest.cs
Comment thread Assets/Tests/UssTest.cs
Comment thread Assets/Tests/UssTest.cs
Comment thread Assets/Tests/UssTest.cs
Comment thread Assets/Tests/TestBase.cs
Comment thread Assets/Tests/UssTest.cs
@kochounoyume
kochounoyume merged commit cd86707 into main Jul 16, 2026
7 checks passed
@kochounoyume
kochounoyume deleted the fix/unity-test-capture branch July 16, 2026 15:31
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.

3 participants