Closes#215 (reopened by the adversarial-reviewer#18 post-merge verification for exactly this gap).
Adds the two missing locked-path behavioral tests to PlayoutControllerTests, mirroring the existing five (409 ConflictObjectResult + ProblemDetails 409 + mediator DidNotReceive). Contract/metadata 409 rows for both PUTs already existed — no OpenAPI change.
Targeted controller+contract filter 404/404; full ErsatzTV.Tests 1026/1026.
Closes #215 (reopened by the adversarial-reviewer#18 post-merge verification for exactly this gap).
Adds the two missing locked-path behavioral tests to `PlayoutControllerTests`, mirroring the existing five (409 ConflictObjectResult + ProblemDetails 409 + mediator `DidNotReceive`). Contract/metadata 409 rows for both PUTs already existed — no OpenAPI change.
Targeted controller+contract filter 404/404; full ErsatzTV.Tests 1026/1026.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Closes the reviewer-verification gap from adversarial-reviewer#18: the
locked-path 409 guard tests already existed for Delete, EraseItems,
EraseItemsAndHistory, Update, and UpdateDefaultDeco, but were missing for
PlayoutController.ReplaceAlternateSchedules and .ReplaceTemplates even
though their production guards (entityLocker.IsPlayoutLocked) were in
place. Added two tests mirroring the existing pattern exactly (409
ConflictObjectResult + ProblemDetails + DidNotReceive() on the mediator
command). ApiErrorResponseMetadataTests and OpenApiErrorResponseContractTests
already carried 409 rows for both PUT endpoints, so no changes were needed
there.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
timothy
merged commit c4b7babbe8 into main2026-07-11 11:59:44 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Closes #215 (reopened by the adversarial-reviewer#18 post-merge verification for exactly this gap).
Adds the two missing locked-path behavioral tests to
PlayoutControllerTests, mirroring the existing five (409 ConflictObjectResult + ProblemDetails 409 + mediatorDidNotReceive). Contract/metadata 409 rows for both PUTs already existed — no OpenAPI change.Targeted controller+contract filter 404/404; full ErsatzTV.Tests 1026/1026.
🤖 Generated with Claude Code