Skip to content

Commit 0cdc1a1

Browse files
committed
test(plugin-security): run the two consequence cells on both driver families
This file's whole thesis is that a claim about the write gate must not turn out to depend on which backend a deployment runs. The two cells the previous commit added ran on one; they now run on both, like the other twenty-three. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012zTkyNHJ7TkuN2oXtP5x37
1 parent c0d87c0 commit 0cdc1a1

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

packages/plugins/plugin-security/src/insert-check-post-image.test.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -938,7 +938,8 @@ for (const [driverName, makeDriver] of DRIVERS) {
938938
* entirely — they are about the ENGINE's pass order, which is what the F1 fix
939939
* moved. They live here because this file is where that reorder is justified.
940940
*/
941-
describe('[#16608 F1] what moving the strips ahead of the credential channel changes', () => {
941+
for (const [credDriverName, makeCredDriver] of DRIVERS) {
942+
describe(`[#16608 F1] ${credDriverName} — what moving the strips ahead of the credential channel changes`, () => {
942943
const OBJ = 'qa_ro_cred';
943944
// The same non-system caller the cells above use — granted create on this
944945
// object, and governed by NO `check`, so nothing here is the RLS gate's doing.
@@ -951,7 +952,7 @@ describe('[#16608 F1] what moving the strips ahead of the credential channel cha
951952
// value test then compared a REF against the caller's plaintext, read the
952953
// difference as "a hook rewrote this key", and KEPT it — the one input
953954
// where that test inverts. Pre-existing on 17.3.0, closed by the reorder.
954-
const booted = await boot(DRIVERS[0]![1]);
955+
const booted = await boot(makeCredDriver);
955956

956957
const outcome = await attempt(() =>
957958
booted.engine.insert(
@@ -985,7 +986,7 @@ describe('[#16608 F1] what moving the strips ahead of the credential channel cha
985986
// `refuseEmptyPasswordFields` up too would let a field-level validation
986987
// verdict answer a write that RLS refuses, which is the wrong precedence
987988
// for a security gate.
988-
const booted = await boot(DRIVERS[0]![1]);
989+
const booted = await boot(makeCredDriver);
989990

990991
const outcome = await attempt(() =>
991992
booted.engine.insert(OBJ, { id: 'cred_2', name: 'n', pw: '' }, { context: CALLER } as never),
@@ -997,3 +998,4 @@ describe('[#16608 F1] what moving the strips ahead of the credential channel cha
997998
expect(rows[0]!.pw, 'the empty credential is not stored — the ruling’s guarantee, unchanged').toBeNull();
998999
});
9991000
});
1001+
}

0 commit comments

Comments
 (0)