fix(coverage): make coverage analyze fail when tests fail

[peter-jerry-ye]

Summary

Fixed the issue where moon coverage analyze would continue to execute the coverage report analysis even when moon test --enable-coverage failed.

Problem

The run_coverage_analyze function was discarding the exit code from run_test using the ? operator, which only propagated errors (not Ok values). This meant that when tests failed (returning exit code 2), the coverage analysis would still proceed, generating reports based on incomplete or invalid coverage data.

Solution

Modified crates/moon/src/cli/coverage.rs to:

1. Capture the test result exit code instead of discarding it
2. Check if the result is non-zero and return early with the failure exit code

Changes

• Fix: crates/moon/src/cli/coverage.rs - Added early return when tests fail
• Test: Added test_moon_coverage_analyze_fails_on_test_failure regression test
• Test data: Added test_coverage_fail_test.in test case with a failing test

Verification

• All 272 moon tests pass (including 3 coverage tests)
• Clippy shows no warnings
• Code is properly formatted

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/moon/src/cli/coverage.rs	48% smaller
crates/moon/tests/test_cases/moon_coverage.rs	0% smaller
crates/moon/tests/test_cases/test_coverage_fail_test.in/lib/hello.mbt	Unsupported file format
crates/moon/tests/test_cases/test_coverage_fail_test.in/lib/moon.pkg.json	0% smaller
crates/moon/tests/test_cases/test_coverage_fail_test.in/moon.mod.json	0% smaller