Skip to content

630:P1 Add UI test for ESC toggling menu overlay#3

Closed
riley-1995 wants to merge 27 commits into
kblaschke:masterfrom
riley-1995:p1/issue-18-ui-esc-toggle-test
Closed

630:P1 Add UI test for ESC toggling menu overlay#3
riley-1995 wants to merge 27 commits into
kblaschke:masterfrom
riley-1995:p1/issue-18-ui-esc-toggle-test

Conversation

@riley-1995
Copy link
Copy Markdown

This PR adds a UI-level test that verifies the ESC key toggles the menu overlay visibility exactly once per press.

The test:

  • Initializes the SDL + ImGui subsystems in-process.
  • Injects SDL KEYDOWN / KEYUP events for SDLK_ESCAPE.
  • Verifies overlay visibility via the deterministic ProjectMGUI::Visible() getter.

Confirms:

  • First ESC press flips overlay visibility.
  • Second ESC press restores the original state.
  • The application does not exit unexpectedly.
  • Ensures ImGui is not capturing keyboard input before injecting ESC to simulate a realistic user interaction state.

The test runs in headless environments (CI) by configuring dummy SDL drivers where needed.

This protects against regressions where:

  • ESC no longer toggles the overlay,
  • ESC triggers unrelated actions (e.g., quit),
  • The overlay does not restore correctly on the second press.

Closes projectM-visualizer#18.

riley-1995 and others added 27 commits February 10, 2026 14:59
Enabled blank issues and removed contact links.
…integration

630:P1 Enable GoogleTest integration via vcpkg manifest when BUILD_TESTING=ON
Add ListAudioDevices test and update test core lib
…ces config before and after calling ListAudioDevices().
Add unit test for --listAudioDevices CLI option behavior
…ild-check-time

Improve Windows CI performance with fork-safe vcpkg binary caching
630:P1 Add integration render smoke test to prevent blank render regressions.
Update render test OpenGL import for Apple, silence depreciation warnings
Add CLI integration test to ensure --help exits without entering RenderLoop
630:P1 Application enters and exits main render loop once
@riley-1995 riley-1995 closed this Feb 27, 2026
@riley-1995 riley-1995 deleted the p1/issue-18-ui-esc-toggle-test branch February 27, 2026 00:06
@riley-1995
Copy link
Copy Markdown
Author

Accidentally made PR to your fork rather than mine. Please ignore.

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.

Occasional Graphical Corruption

2 participants