Skip to content

[BUG][P0] test_node_count stale after hourly_tracking HITL — CI failing on main #25

Description

@mishrapravin114

Summary

CI on main is red because tests/test_hourly_tracking_agent.py::test_node_count still expects 5 nodes after human_review was added (graph now has 6 nodes).

Evidence

FAILED tests/test_hourly_tracking_agent.py::test_node_count
assert 6 == 5

Latest failing CI run on main after hourly_tracking HITL merge.

Root cause

Agent graph gained human_review_node; test was not updated.

Fix

File: core/tests/test_hourly_tracking_agent.py

def test_node_count():
    assert len(graph.nodes) == 6

Optional additions:

  • Assert "human_review" in [n.id for n in graph.nodes]
  • Assert graph.pause_nodes == ["human_review"]

Acceptance criteria

  • ./engine test suite passes locally: cd core && uv run python -m pytest tests/test_hourly_tracking_agent.py -q
  • CI Test job green on PR merge

Effort

~5 minutes

Type

Bug fix — not investigation.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingphase-1Phase 1 — pilot / GTMpriority-p0P0 — critical path / do nowtype-agentVertical agent template

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions