Skip to content

test_now_render: test_a_spec_now_line_is_picked_up is RED on main — f1cffbe9 rewrote the spec it asserts on #388

Description

@localai-bot

Reproduced on origin/main b580452d, and it is main's, not any open PR's.

FAIL: test_a_spec_now_line_is_picked_up (test_now_render.DerivedFromPerRowRecords)
AssertionError: 'W2' not found in 'DONE at implementation merge `dbd0d51c`: W1-W5 shipped.
Progressive `## Now` backfill is the selected compatibility policy, not remaining work;
use `scripts/now.py` for the derived live position.'
 : this row's own spec carries a `## Now`, so it must render
Ran 11 tests -- FAILED (failures=1)

tests/scripts/test_now_render.py:92 asserts the literal "W2" appears in
now.spec_now("now-derived.md"). f1cffbe9 ("close ENG-NOW-DERIVED honestly",
#374) rewrote that spec's ## Now section to the DONE wording above, which no
longer contains W2 (it says W1-W5), and did not update the test.

Why this is main's and not a merge artifact: now.spec_now() reads exactly one
file, .agents/specs/now-derived.md. In the tree where this was observed,
tests/scripts/test_now_render.py, .agents/specs/now-derived.md and
scripts/now.py are all byte-identical to b580452d (diff against
git show b580452d:<path> is empty for all three), so nothing else in the tree
can influence the result.

Not fixed in the flow that found it, deliberately. The test is a guard, and the
repair is a judgement about what it should assert now that the prose it was
coupled to has legitimately changed -- assert non-empty, assert a ## Now
heading exists, or re-couple to a different token. AGENTS.md holds that changing
a checker's semantics takes a spec plus red-before/green-after evidence, and
that a red gate is never turned green by weakening an assertion, so this belongs
to the ENG-NOW-DERIVED owner rather than to a passing landing.

Found while landing PRs #324, #282 and #267.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions