Skip to content

Raise test coverage toward 90% (autopilot round 10, low blast radius only) #1664

Description

@stranske

Why

Fleet coverage initiative round 10 remains below the 90% objective. The next slice must prove a real behavioral branch in dashboard news-feed loading, not pad the metric.

Scope

Add a discriminating regression in tests/test_dashboard.py for an uncovered SQLite manager-key branch in ui/dashboard.py (load_news_stream).

Non-Goals

  • Do NOT change CI or workflow configuration.
  • Do NOT add trivial getter or import-only tests.

Tasks

  • Run pytest tests/test_dashboard.py -k news --cov=ui/dashboard.py --cov-report=term-missing and record baseline in the PR body.
  • In tests/test_dashboard.py, add a regression for load_news_stream when SQLite databases use managers.id instead of managers.manager_id in ui/dashboard.py, and record deliberate-break plus measured coverage delta in the PR body.

Acceptance Criteria

  • pytest tests/test_dashboard.py -k news --cov=ui/dashboard.py --cov-report=term-missing exits 0 with measured coverage delta recorded in the PR body.
  • The new regression fails when its selected branch is deliberately broken, then passes after revert; both recorded in the PR body.

Implementation Notes

Coverage-autopilot round 10; low blast radius only. If measured coverage is already at or above 90%, comment and close without opening a PR.

Activity

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

    agents:formattedIssue has been formatted to AGENT_ISSUE_TEMPLATE

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions