chore(ci): restore coverage threshold to 97% after TDD integration test fixes #4337

New issue

Closed

opened 2026-04-07 19:52:38 +00:00 by freemo · 2 comments

freemo commented

2026-04-07 19:52:38 +00:00

Owner

Coverage Restoration Issue

Priority: High - Critical for CI pipeline integrity

The coverage threshold was temporarily lowered to 50% to enable CI to pass while implementing TDD expected-fail tags for failing integration tests. Now that the 8 failing integration tests have been properly tagged with TDD expected-fail tags, the coverage threshold must be restored to the original 97%.

Acceptance Criteria

Update noxfile.py COVERAGE_THRESHOLD constant from 50% to 97%
Verify all integration tests pass with TDD tags in place
Verify coverage reports properly enforce 97% threshold
Ensure CI quality pipeline passes completely

Metadata

Commit Message: chore(ci): restore coverage threshold to 97%
Branch: chore/restore-coverage-threshold

Context

This issue was created after implementing TDD expected-fail tags for 8 failing integration tests (issues #4329-#4336). The TDD system inverts failing test results to pass when tagged with tdd_expected_fail, allowing CI to pass while maintaining traceability of known issues.

Issues #4329-#4336: TDD expected-fail integration test issues
Original PR: #4175 (fix/restore-ci-quality-tests)

Definition of Done

This issue is complete when:

COVERAGE_THRESHOLD = 97 in noxfile.py
Full nox quality pipeline passes
Integration tests pass via TDD inversion system
CI enforces proper coverage standards
PR submitted, reviewed, and merged

## Coverage Restoration Issue **Priority:** High - Critical for CI pipeline integrity The coverage threshold was temporarily lowered to 50% to enable CI to pass while implementing TDD expected-fail tags for failing integration tests. Now that the 8 failing integration tests have been properly tagged with TDD expected-fail tags, the coverage threshold must be restored to the original 97%. ## Acceptance Criteria - [ ] Update `noxfile.py` COVERAGE_THRESHOLD constant from 50% to 97% - [ ] Verify all integration tests pass with TDD tags in place - [ ] Verify coverage reports properly enforce 97% threshold - [ ] Ensure CI quality pipeline passes completely ## Metadata - **Commit Message:** `chore(ci): restore coverage threshold to 97%` - **Branch:** `chore/restore-coverage-threshold` ## Context This issue was created after implementing TDD expected-fail tags for 8 failing integration tests (issues #4329-#4336). The TDD system inverts failing test results to pass when tagged with `tdd_expected_fail`, allowing CI to pass while maintaining traceability of known issues. ## Related Issues - Issues #4329-#4336: TDD expected-fail integration test issues - Original PR: #4175 (fix/restore-ci-quality-tests) ## Definition of Done This issue is complete when: - COVERAGE_THRESHOLD = 97 in noxfile.py - Full nox quality pipeline passes - Integration tests pass via TDD inversion system - CI enforces proper coverage standards - PR submitted, reviewed, and merged

freemo added the

Points

label

2026-04-07 19:52:53 +00:00

HAL9000 commented

2026-04-08 11:24:15 +00:00

Owner

Closing as duplicate of #4183. Both issues track restoring the test coverage threshold to 97%. Issue #4183 is the canonical tracking issue with proper labels and MoSCoW priority.

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

Closing as duplicate of #4183. Both issues track restoring the test coverage threshold to 97%. Issue #4183 is the canonical tracking issue with proper labels and MoSCoW priority. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 closed this issue

2026-04-08 11:24:52 +00:00

HAL9000 commented

2026-04-08 11:32:35 +00:00

Owner

Issue triaged by project owner:

State: Verified ✅ — CI coverage threshold needs restoration after TDD integration test fixes
Priority: High — Coverage threshold enforcement is critical for code quality
Milestone: v3.5.0 (M6: Autonomy Hardening) — Coverage ≥97% is an acceptance criterion for all milestones
Story Points: 3 — M (already set) — Requires fixing underlying test failures first
MoSCoW: Must Have ✅ (already set) — Coverage threshold is a hard requirement
Assignee: HAL9000

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

Issue triaged by project owner: - **State**: Verified ✅ — CI coverage threshold needs restoration after TDD integration test fixes - **Priority**: High — Coverage threshold enforcement is critical for code quality - **Milestone**: v3.5.0 (M6: Autonomy Hardening) — Coverage ≥97% is an acceptance criterion for all milestones - **Story Points**: 3 — M (already set) — Requires fixing underlying test failures first - **MoSCoW**: Must Have ✅ (already set) — Coverage threshold is a hard requirement - **Assignee**: HAL9000 --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added this to the v3.5.0 milestone

2026-04-08 11:33:18 +00:00