Skip to content

Commit 61908f0

Browse files
os-zhuangclaude
andcommitted
test(showcase): expect 9 positions after finance/legal added
The v16 approval-demo change added `finance` + `legal` positions to positions.ts (7 → 9), but seed.test.ts still asserted 7, failing Test Core. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 7daf0fa commit 61908f0

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

examples/app-showcase/test/seed.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,11 @@ describe('showcase stack', () => {
2525
// leaving 3 dataset-bound analytics reports.
2626
expect((stack.reports ?? []).length).toBe(3);
2727
expect((stack.flows ?? []).length).toBeGreaterThan(0);
28-
// Seven flat positions (contributor/manager/exec/auditor/ops/
29-
// field_ops_delegate/client_portal_user) — the ADR-0090 distribution
30-
// layer; `everyone` and `guest` are built-in anchors and never declared
31-
// by the app.
32-
expect((stack.positions ?? []).length).toBe(7);
28+
// Nine flat positions (contributor/manager/exec/auditor/ops/
29+
// field_ops_delegate/client_portal_user, plus finance/legal for the v16
30+
// approval sign-off flows) — the ADR-0090 distribution layer; `everyone`
31+
// and `guest` are built-in anchors and never declared by the app.
32+
expect((stack.positions ?? []).length).toBe(9);
3333
expect((stack.agents ?? []).length).toBe(0); // AI agents are an enterprise (service-ai) feature; the open showcase ships none
3434
});
3535
});

0 commit comments

Comments
 (0)