From 6025a1d1c377d847bbe84af7ba11857934cbc7bd Mon Sep 17 00:00:00 2001 From: Ed_ Date: Thu, 18 Jun 2026 12:29:43 -0400 Subject: [PATCH] test(extended_sims): Phase 13.4 - switch test_execution_sim_live from gemini_cli to gemini User directive (2026-06-17): do not add skip markers for flaky tests. Instead, switch the test to use a different provider (gemini) and report if it still fails. Original: gemini_cli with mock_gemini_cli.py subprocess New: gemini with gemini-2.5-flash-lite model If the test still fails, REPORT it -- do not add a skip marker. The user wants to start a diff track to fix it. --- tests/test_extended_sims.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/test_extended_sims.py b/tests/test_extended_sims.py index 86df2672..7170847f 100644 --- a/tests/test_extended_sims.py +++ b/tests/test_extended_sims.py @@ -64,8 +64,8 @@ def test_execution_sim_live(live_gui: Any) -> None: sim.setup("LiveExecutionSim") # Enable manual approval to test modals client.set_value('manual_approve', True) - client.set_value('current_provider', 'gemini_cli') - client.set_value('gcli_path', f'"{sys.executable}" "{os.path.abspath("tests/mock_gemini_cli.py")}"') + client.set_value('current_provider', 'gemini') + client.set_value('current_model', 'gemini-2.5-flash-lite') client.set_value('auto_add_history', True) sim.run() time.sleep(2)