Skip to content

Commit c1bfdf2

Browse files
yaoyu-33claude
andcommitted
[test] fix: use absolute coverage path in MiniMax-M2 functional test
Use hardcoded /opt/Megatron-Bridge paths for --data-file and --source, matching the pattern used by the qwen3_moe functional test. The dynamic tmp_path-based coverage file caused issues when sys.executable resolves to the container Python rather than the uv-managed venv. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: yaoyu-33 <yaoyu.094@gmail.com>
1 parent 53f2d66 commit c1bfdf2

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

‎tests/functional_tests/test_groups/models/minimax_m2/test_minimax_m2_conversion.py‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -143,8 +143,7 @@ def test_minimax_m2_conversion_parallelism(self, toy_model_path, tmp_path, tp, p
143143
test_output_dir = tmp_path / f"minimax_m2_{test_name}"
144144
test_output_dir.mkdir(exist_ok=True)
145145

146-
repo_root = Path(__file__).resolve().parents[5]
147-
coverage_file = tmp_path / ".coverage"
146+
repo_root = "/opt/Megatron-Bridge"
148147
cmd = [
149148
sys.executable,
150149
"-m",
@@ -154,10 +153,10 @@ def test_minimax_m2_conversion_parallelism(self, toy_model_path, tmp_path, tp, p
154153
"-m",
155154
"coverage",
156155
"run",
157-
f"--data-file={coverage_file}",
158-
f"--source={repo_root}",
156+
f"--data-file={repo_root}/.coverage",
157+
f"--source={repo_root}/",
159158
"--parallel-mode",
160-
str(repo_root / "examples/conversion/hf_megatron_roundtrip_multi_gpu.py"),
159+
f"{repo_root}/examples/conversion/hf_megatron_roundtrip_multi_gpu.py",
161160
"--hf-model-id",
162161
toy_model_path,
163162
"--output-dir",

0 commit comments

Comments
 (0)