Skip to content

Commit e899b80

Browse files
committed
test(approvals): declare only the verbs the snapshot test double is driven through
Claude-Session: https://claude.ai/code/session_018zT8d8NpiQ1ExhuNd5TxY6 Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 0a68f75 commit e899b80

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎packages/plugins/plugin-approvals/src/approval-snapshot-credential-mask.test.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,9 @@ function makeEngine(withSchema = true) {
4141
return rows.slice(options?.offset ?? 0, (options?.offset ?? 0) + (options?.limit ?? 1000));
4242
},
4343
async insert(object: string, data: any) { ensure(object).push({ ...data }); return { ...data }; },
44-
async update(_object: string, data: any) { return data; },
45-
async delete() { return null; },
44+
// Opening a request and reading it back reach no update/delete verb, so
45+
// the double declares none (the same reasoning as
46+
// `approval-payload-masked-field.test.ts`).
4647
};
4748
if (withSchema) {
4849
engine.getSchema = (object: string) => object !== OBJECT ? undefined : {

0 commit comments

Comments
 (0)