From b36a30de35d8dc0368450be5e768fa435fd1caeb Mon Sep 17 00:00:00 2001 From: stash Date: Fri, 26 Jun 2026 12:38:20 -0700 Subject: [PATCH 1/2] test: skip dimsim path-replanning e2e when OPENAI_API_KEY unset test_path_replanning drives the unitree-go2-agentic blueprint, which deploys SpeakSkill -> OpenAITTSNode and the agentic stack, both of which require OPENAI_API_KEY at module start. Without the key the blueprint fails to start and the test errors out instead of skipping. Add @pytest.mark.skipif_no_openai (already used by every other dimos/e2e_tests test) so it skips cleanly when the key is absent, while still running on the self-hosted-large runner where the secret is set. --- dimos/e2e_tests/test_dimsim_path_replaning.py | 1 + 1 file changed, 1 insertion(+) diff --git a/dimos/e2e_tests/test_dimsim_path_replaning.py b/dimos/e2e_tests/test_dimsim_path_replaning.py index d222da30bd..ae15fe1904 100644 --- a/dimos/e2e_tests/test_dimsim_path_replaning.py +++ b/dimos/e2e_tests/test_dimsim_path_replaning.py @@ -15,6 +15,7 @@ import pytest +@pytest.mark.skipif_no_openai @pytest.mark.self_hosted_large def test_path_replanning( lcm_spy, start_blueprint, dim_sim, direct_cmd_vel_explorer, spawn_wall_on_pose From 579a453c436130a7fdeb5cced0d452009053ace5 Mon Sep 17 00:00:00 2001 From: stash Date: Fri, 26 Jun 2026 13:02:36 -0700 Subject: [PATCH 2/2] test: skip remaining dimsim e2e tests when OPENAI_API_KEY unset test_walk_forward and test_go_to_the_bed both drive the unitree-go2-agentic blueprint (same OpenAI-backed agentic stack + SpeakSkill as test_path_replanning), so they also error at blueprint startup without OPENAI_API_KEY. Add @pytest.mark.skipif_no_openai to both, matching every other agentic e2e test. This covers all three dimos/e2e_tests/test_dimsim_*.py tests. --- dimos/e2e_tests/test_dimsim_spatial_memory.py | 1 + dimos/e2e_tests/test_dimsim_walk_forward.py | 1 + 2 files changed, 2 insertions(+) diff --git a/dimos/e2e_tests/test_dimsim_spatial_memory.py b/dimos/e2e_tests/test_dimsim_spatial_memory.py index df2d2477f6..a20c316e15 100644 --- a/dimos/e2e_tests/test_dimsim_spatial_memory.py +++ b/dimos/e2e_tests/test_dimsim_spatial_memory.py @@ -15,6 +15,7 @@ import pytest +@pytest.mark.skipif_no_openai @pytest.mark.self_hosted_large def test_go_to_the_bed(lcm_spy, start_blueprint, human_input, dim_sim, explore_house) -> None: start_blueprint( diff --git a/dimos/e2e_tests/test_dimsim_walk_forward.py b/dimos/e2e_tests/test_dimsim_walk_forward.py index fe8a73e94f..b8c110b628 100644 --- a/dimos/e2e_tests/test_dimsim_walk_forward.py +++ b/dimos/e2e_tests/test_dimsim_walk_forward.py @@ -15,6 +15,7 @@ import pytest +@pytest.mark.skipif_no_openai @pytest.mark.self_hosted_large def test_walk_forward(lcm_spy, start_blueprint, human_input, dim_sim) -> None: start_blueprint(