-
-
Notifications
You must be signed in to change notification settings - Fork 6
Open
Description
Problem
The CI detection test in crates/mandrel-mcp-th/src/cli/mod.rs is currently ignored because it fails in local environments where GitHub Actions environment variables are not set.
Details
- Test:
test_ci_system_detection - Issue: The test sets environment variables but the detection logic may not be working correctly
- Current Status: Temporarily ignored with
#[ignore]attribute
Expected Behavior
The test should:
- Properly set GitHub Actions environment variables
- Detect the CI system correctly
- Pass consistently across different environments
Suggested Fix
- Review the CI detection logic in the
EnvironmentDetector - Ensure environment variable setup in the test is complete
- Add proper cleanup of environment variables
- Consider using mock environments for testing
Priority
Medium - This is a test infrastructure issue that affects CI/CD validation.
Metadata
Metadata
Assignees
Labels
No labels