Skip to content

test(coverage): Improve global test coverage after ADE implementation #52

Description

@Pedrovaleriolopez

Problem

After merging PR #51 (AIOS Dashboard & ADE Core Implementation), global test coverage dropped below thresholds due to +108k lines of new code.

Current Coverage vs Thresholds

Metric Current Threshold Gap
Statements 26.56% 30% -3.44%
Branches 23.7% 25% -1.3%
Lines 26.71% 30% -3.29%
Functions 28.15% 30% -1.85%
.aios-core/core/ 43.35% 45% -1.65%

Scope

New modules needing tests:

ADE Core (~15k lines)

  • .aios-core/core/execution/ - Build orchestrator, wave executor, rate limiter
  • .aios-core/core/orchestration/ - Master orchestrator, gate evaluator, recovery handler
  • .aios-core/core/memory/ - Gotchas memory, context snapshot
  • .aios-core/core/ideation/ - Ideation engine

Infrastructure (~10k lines)

  • .aios-core/infrastructure/scripts/ - Plan tracker, worktree manager, QA orchestrator

Workflow Intelligence (~5k lines)

  • .aios-core/workflow-intelligence/learning/ - Gotcha registry, QA feedback, semantic search

Acceptance Criteria

  • Global statement coverage ≥ 30%
  • Global branch coverage ≥ 25%
  • Global line coverage ≥ 30%
  • Global function coverage ≥ 30%
  • .aios-core/core/ line coverage ≥ 45%

Priority

Medium - Tests passing, coverage is a quality metric not a blocker.

Related

Activity

  1. oalanicolas commented on Feb 2, 2026

    @oalanicolas
    Collaborator

    🔗 Consolidation Note

    This issue is related to #50 (core coverage 60%).

    Recommended approach: Work on both issues together as a single coverage improvement effort:

    1. Focus on .aios-core/core/ first (fix(coverage): Increase .aios-core/core/ test coverage to meet 60% threshold #50)
    2. Then expand to global coverage (this issue)

    The .aios-core/core/ coverage improvement will directly contribute to global coverage targets.

  2. oalanicolas commented on Feb 2, 2026

    @oalanicolas
    Collaborator

    Progress Update

    Current Coverage (commit 9e23f63):

    Metric Current Threshold Status
    Statements 28.17% 30% ❌ -1.83%
    Branches 25.17% 25% ✅ Met
    Functions 30.57% 30% ✅ Met
    Lines 28.44% 30% ❌ -1.56%

    Done:

    • ✅ Branches coverage meets threshold
    • ✅ Functions coverage meets threshold
    • ✅ Removed broken integration test (tools-system.test.js)

    Remaining:

    • Statements needs +1.83% (~770 lines)
    • Lines needs +1.56% (~630 lines)

    Will continue improving coverage in future sessions.

  3. Pedrovaleriolopez commented on Feb 3, 2026

    @Pedrovaleriolopez
    ContributorAuthor

    Update: Issue #50 mesclada aqui

    A issue #50 (.aios-core/core/ coverage) foi fechada e seus requisitos foram consolidados nesta issue.

    Requisitos adicionais (de #50):

    • .aios-core/core/ line coverage ≥ 60% (atualmente ~47%)

    Priorização sugerida:

    1. Primeiro: Focar em .aios-core/core/ (módulos críticos)
    2. Depois: Expandir para ADE e Infrastructure

    Atualizado por @devops (Gage)

  4. 24 remaining items

  5. rafaelscosta commented on May 8, 2026

    @rafaelscosta
    Collaborator

    Coverage debt is now resolved on main after the merged coverage replacement PRs.

    Validation run:
    npm run test:coverage -- --runInBand --forceExit

    Result:

    • Test Suites: 11 skipped, 328 passed, 328 of 339 total
    • Tests: 149 skipped, 8314 passed, 8463 total

    Coverage summary from coverage/coverage-final.json via Istanbul:

    I also restored the local generated entity registry mutation from the coverage run, so no runtime/generated state drift is being carried forward.

    Closing as completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: coreCore framework (.aios-core/core/)priority: P3Medium — affects some usersstatus: confirmedIssue confirmed, ready for worktype: testTest coverage and quality

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions