Skip to content

Conversation

@TheSecurityDev
Copy link
Contributor

Adds screenshots for most states of the setup, reset, and updater screens. Most of the code is in the system for running these new types of tests (separate UI script), so once you have it, adding the tests for all 3 scripts doesn't add much complexity.

@Copilot Copilot AI review requested due to automatic review settings October 17, 2025 05:32
@github-actions github-actions bot added the ui label Oct 17, 2025
Copy link
Contributor

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 screenshot testing capabilities for setup, reset, and updater UI screens by creating a new testing framework for standalone UI scripts. The changes enable automated screenshot capture of various states of these critical system interfaces.

Key changes:

  • Creates a new TestScriptUI class to handle testing of standalone UI scripts with subprocess management
  • Adds SOFTWARE_SETUP_CASES configuration for setup, updater, and reset screen test scenarios
  • Extends the screenshot testing system to support multi-step UI interactions and state capture

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2025

raylib UI Preview

reset : $${\color{cyan}\text{NEW}}$$
reset_2 : $${\color{cyan}\text{NEW}}$$
reset_3 : $${\color{cyan}\text{NEW}}$$
setup : $${\color{cyan}\text{NEW}}$$
setup_2 : $${\color{cyan}\text{NEW}}$$
setup_3 : $${\color{cyan}\text{NEW}}$$
setup_4 : $${\color{cyan}\text{NEW}}$$
setup_5 : $${\color{cyan}\text{NEW}}$$
setup_6 : $${\color{cyan}\text{NEW}}$$
setup_7 : $${\color{cyan}\text{NEW}}$$
updater : $${\color{cyan}\text{NEW}}$$
updater_2 : $${\color{cyan}\text{NEW}}$$
updater_3 : $${\color{cyan}\text{NEW}}$$
All Screenshots

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant