fix(housekeeping): keep rank reset transactional
This commit is contained in:
1 parent
3b568e30b4
commit
16ebc4a09e
2 files changed
+8
-7
No files matched your search
@@ -104,12 +104,13 @@ describe("transaction-aware rank lifecycle helpers", () => {
|
|||||||
});
|
});
|
||||||
await deleteEmulatorRankInTransaction(executor as never, 7);
|
await deleteEmulatorRankInTransaction(executor as never, 7);
|
||||||
|
|
||||||
expect(statements).toHaveLength(5);
|
expect(statements).toHaveLength(6);
|
||||||
expect(statements[0]).toContain("INSERT INTO permission_ranks");
|
expect(statements[0]).toContain("UPDATE permission_definitions");
|
||||||
expect(statements[1]).toContain("INSERT INTO permissions");
|
expect(statements[1]).toContain("INSERT INTO permission_ranks");
|
||||||
expect(statements[2]).toContain("UPDATE permission_definitions");
|
expect(statements[2]).toContain("INSERT INTO permissions");
|
||||||
expect(statements[3]).toContain("DELETE FROM permission_ranks");
|
expect(statements[3]).toContain("UPDATE permission_definitions");
|
||||||
expect(statements[4]).toContain("DELETE FROM permissions");
|
expect(statements[4]).toContain("DELETE FROM permission_ranks");
|
||||||
|
expect(statements[5]).toContain("DELETE FROM permissions");
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -216,7 +216,6 @@ export async function prepareEmulatorRankCreation(
|
|||||||
);
|
);
|
||||||
const id = Number(rows[0]?.next_id ?? 1);
|
const id = Number(rows[0]?.next_id ?? 1);
|
||||||
await ensurePermissionRankColumn(db, id);
|
await ensurePermissionRankColumn(db, id);
|
||||||
await resetPermissionRankColumn(db, id);
|
|
||||||
return id;
|
return id;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -225,6 +224,7 @@ export async function createEmulatorRankInTransaction(
|
|||||||
id: number,
|
id: number,
|
||||||
data: { rank_name: string; level: number },
|
data: { rank_name: string; level: number },
|
||||||
): Promise<void> {
|
): Promise<void> {
|
||||||
|
await resetPermissionRankColumn(db, id);
|
||||||
await rawExec(
|
await rawExec(
|
||||||
db,
|
db,
|
||||||
sql`
|
sql`
|
||||||
|
|||||||
Reference in new issue
Block a user