From 96cc49bb7c1ac960df0bc7de8cab68b317310bda Mon Sep 17 00:00:00 2001 From: Vivswan Shah <58091053+Vivswan@users.noreply.github.com> Date: Mon, 21 Sep 2026 19:21:17 -0400 Subject: [PATCH] test(webhooks): drop the case the contract tests already pin The read-port shape case restates the endpoint table at runtime; the canonical read-port case in the contract tests pins the port's derivation for every section. The changed-url identity case stays: its note assertion is the only pin of the section's undeclared action wording. The convergence twin stays: its final-state assertion is the only pin that the resolved secret lands in the stored hook config. Test-only; no source changes. --- src/sections/webhooks/webhooks.test.ts | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/sections/webhooks/webhooks.test.ts b/src/sections/webhooks/webhooks.test.ts index f09f559a..69317551 100644 --- a/src/sections/webhooks/webhooks.test.ts +++ b/src/sections/webhooks/webhooks.test.ts @@ -395,15 +395,4 @@ describe("webhooks plan", () => { ], ]); }); - - test("the read port exposes exactly the list role in its denied posture", () => { - const ctx = planContext(webhooksSection, new MockApi({}), REPO); - expect(Object.keys(ctx.read)).toEqual(["list"]); - // @ts-expect-error a write role is not a read: the port has no `create` - ctx.read.create; - // @ts-expect-error nor an `updateConfig` - ctx.read.updateConfig; - // @ts-expect-error a "denied" primary read offers no 404-tolerant helper - ctx.read.list.probeAbsent; - }); });