Private
Public Access
0
0
Files
manual_slop/tests
ed 63e4e54e1b test(palette): use deterministic close in 3 test functions
3 tests fail because _toggle_command_palette is non-deterministic AND the
tests depend on prior fixture state. The toggle only flips the boolean,
so the test's behavior depends on whether palette starts open or closed.

Fixed all 3 tests by adding a force-close preamble that:
  if client.get_value("show_command_palette") is True:
      client.push_event("custom_callback", {"callback": "_toggle_command_palette", "args": []})
      poll for False with 2s deadline

Tests fixed:
- test_palette_starts_hidden: replaced unconditional toggle (which opened
  the palette from default-closed state) with conditional force-close
- test_palette_toggles_via_callback: added force-close preamble before
  the "assert initial state is False" check
- test_palette_query_state_resets_on_open: added force-close preamble
  before the 3-toggle sequence (so toggle sequence starts from closed
  state and ends open, matching the assertion)

Verification: 7 of 7 tests pass in tests/test_command_palette_sim.py
(was 3 failed, 4 passed). Also passes in batch with other live_gui
tests (12 of 12 pass) - no isolation-pass fallacy.
2026-06-24 11:14:46 -04:00
..