Skip to content

Commit 1fc26af

Browse files
committed
fix(merge): VerifyOutcome mock in TestEarlyAckOrdering (lv←main sync)
Same drift as prior sync: my aws-samples#616 early-ACK test mocked verify_build/verify_lint with a bare bool, but the fork's resolved source returns VerifyOutcome (build_outcome.passed) → AttributeError. Fixed both mocks to VerifyOutcome(passed=True). Gates: cdk 173 suites/3416 tests + synth + eslint; agent 1412 tests + ruff + ty (no hang).
1 parent 09fe5fd commit 1fc26af

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

agent/tests/test_pipeline.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1988,8 +1988,8 @@ async def fake_run_agent(_prompt, _system_prompt, _config, cwd=None, trajectory=
19881988
patch("pipeline.transition_task_started") as m_transition,
19891989
patch("pipeline.configure_channel_mcp"),
19901990
patch("pipeline.ensure_committed", return_value=False),
1991-
patch("pipeline.verify_build", return_value=True),
1992-
patch("pipeline.verify_lint", return_value=True),
1991+
patch("pipeline.verify_build", return_value=VerifyOutcome(passed=True)),
1992+
patch("pipeline.verify_lint", return_value=VerifyOutcome(passed=True)),
19931993
patch("pipeline.ensure_pr", return_value="https://github.com/o/r/pull/1"),
19941994
patch("pipeline.get_disk_usage", return_value=0),
19951995
patch("pipeline.print_metrics"),

0 commit comments

Comments
 (0)