Skip to content

Commit da2c5e1

Browse files
committed
test(objectql): control-plane leg of ADR-0029 D9's pin asserts the full ADR-0112 envelope (#7470)
The control-plane leg of protocol-object-overlay-layer.test.ts asserted only a message substring, because deleteMetaItem's catch re-wrap used to drop `code` (a repository refusal reached the caller as 403 with `code: undefined`). PR #7466 (2c28df9) fixed the producer, so the constraint is gone: the leg now asserts `code` + `status`, exactly like its project-kernel sibling, and the message-substring check is dropped (the sibling carries none). Reverse-verified: with packages/metadata-protocol/src/protocol.ts reverted to 2c28df9^ and rebuilt, exactly this one test goes red (expected undefined to be 'NOT_OVERRIDABLE'); restored, 16/16 green. Refs #7470, #7426, #7466. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MwoubC3jL271FYt9rGXwxb
1 parent c6a4eeb commit da2c5e1

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

packages/objectql/src/protocol-object-overlay-layer.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -400,7 +400,8 @@ describe('ADR-0029 D9.7 — the delete is a SUBTRACTION, and #7012\'s guard is r
400400
.deleteMetaItem({ type: 'object', name: 'myapp_invoice' })
401401
.then(() => null, (e: any) => e);
402402
expect(refused).toBeInstanceOf(Error);
403-
expect(String(refused.message)).toContain('NOT_OVERRIDABLE');
403+
expect(refused.code).toBe('NOT_OVERRIDABLE');
404+
expect(refused.status).toBe(403);
404405

405406
const b = await bootWithPackage(seed, { controlPlane: true });
406407
await withObjectWritable(() => b.protocol.deleteMetaItem({ type: 'object', name: 'myapp_invoice' }));

0 commit comments

Comments
 (0)