Skip to content

Commit 06ea7bd

Browse files
committed
test(ci): fix autogen coverage gate follow-up
Use a cross-platform Python shell for coverage telemetry and align maf setup tests with the current tool surface and fallback helper access.
1 parent 7e92903 commit 06ea7bd

2 files changed

Lines changed: 8 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ jobs:
5050
5151
- name: Emit branch coverage telemetry
5252
if: always()
53+
shell: python
5354
run: |
54-
python - <<'PY'
5555
import json
5656
import sys
5757
import xml.etree.ElementTree as ET
@@ -78,10 +78,10 @@ jobs:
7878
"required": required,
7979
}))
8080
sys.exit(0 if passed else 1)
81-
PY
8281
8382
- name: Compile Python sources
8483
run: python -m compileall autogen_starter autogen_dashboard maf_starter main.py -q
8584

8685
- name: Validate dashboard JavaScript
8786
run: node --check autogen_dashboard/static/app.js
87+

tests/test_maf_setup.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,13 @@
2424
from maf_starter.devui_overrides import OLD_GD_RENDERER
2525
from maf_starter.devui_overrides import _patch_devui_bundle
2626
from maf_starter.devui_overrides import install_devui_ui_overrides
27+
import maf_starter.provider_fallback as provider_fallback
2728
from maf_starter.provider_fallback import (
2829
ResponseStream,
2930
_is_fallback_worthy_error,
3031
_merge_route_metadata,
3132
_response_from_updates,
3233
_resolve_run_scope,
33-
_sanitize_messages,
3434
_wrap_stream_with_fallback,
3535
)
3636
from maf_starter.routing_policy import RoutingPlan, build_routing_plan
@@ -182,6 +182,9 @@ def test_build_repo_tools_contains_expected_tool_names(self) -> None:
182182
"search_repo",
183183
"request_human_approval",
184184
"apply_repo_write_plan",
185+
"delegate_task_to_claude",
186+
"delegate_task_to_codex",
187+
"delegate_task_to_antigravity",
185188
},
186189
)
187190

@@ -348,7 +351,7 @@ def test_sanitize_messages_strips_unsupported_reasoning_properties(self) -> None
348351
Message(role="user", text="final"),
349352
]
350353

351-
sanitized = _sanitize_messages(messages)
354+
sanitized = provider_fallback._sanitize_messages(messages)
352355

353356
self.assertEqual(len(sanitized), 2)
354357
self.assertNotIn("reasoning_details", sanitized[0].additional_properties)
@@ -618,3 +621,4 @@ def test_env_example_documents_ollama(self) -> None:
618621

619622
if __name__ == "__main__":
620623
unittest.main()
624+

0 commit comments

Comments
 (0)