Skip to content

fix: coverage timeout#9

Merged
wasabeef merged 1 commit intomainfrom
fix-coverage
Jul 14, 2025
Merged

fix: coverage timeout#9
wasabeef merged 1 commit intomainfrom
fix-coverage

Conversation

@wasabeef
Copy link
Owner

@wasabeef wasabeef commented Jul 14, 2025

Description

Fixes CI coverage generation timeout issues by improving cargo-tarpaulin configuration.

Changes

  • Increased timeout from 180s to 300s
  • Added --engine llvm for stability
  • Excluded test files from coverage calculation
  • Added error handling to prevent CI failures
  • Coverage now reports source code only (more accurate)

Impact

  • CI stability improved
  • Coverage metrics more accurate (14.6% vs 17%)
  • Build failures prevented when tarpaulin encounters issues

@github-actions
Copy link

📊 CI Results

✅ All Checks Passed

📋 Coverage & Testing

  • Coverage: 14.6%
  • Total Tests: 248
  • Security Tests: 55
  • Performance Tests: 17
  • MockDeviceManager Tests: 21

🎯 Quality Metrics

⚠️ Coverage: 14.6%
✅ Linting: All clippy warnings resolved
✅ Formatting: Code properly formatted
✅ Security: Comprehensive protection validated

🚀 Build Status

  • Ubuntu: ✅ Passed
  • macOS: ✅ Passed
  • Artifacts: ✅ Generated

🧪 Test Suite Innovation

  • Test Files: 49 total (5 MockDeviceManager)
  • Emulator Independent: ✅ 21 tests (< 6 seconds)
  • Mock Coverage: 10% of test files are emulator-independent
  • CI Performance: Fast feedback with MockDeviceManager integration

@wasabeef wasabeef marked this pull request as ready for review July 14, 2025 02:58
@wasabeef wasabeef merged commit 3a16baf into main Jul 14, 2025
6 checks passed
@wasabeef wasabeef deleted the fix-coverage branch July 14, 2025 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments