External/SnapOtter
1
0
Fork 0
mirror of https://github.com/snapotter-hq/SnapOtter.git synced 2026-08-03 07:46:42 +02:00
Code Issues Packages Projects Releases Wiki Activity

test: update permission count tests for new enterprise permissions

Browse Source
This commit is contained in:
SnapOtter
2026-06-13 17:07:39 +08:00
parent 1cf1f47d6f
commit de7bbcc18d
3 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/unit/api/effective-permissions.test.ts
+2 -2
View File
@@ -192,8 +192,8 @@ describe("hasEffectivePermission", () => {
describe("getPermissions", () => { describe("getPermissions", () => {
describe("exact counts for built-in roles", () => { describe("exact counts for built-in roles", () => {
it("admin has exactly 14 permissions", async () => { it("admin has exactly 17 permissions", async () => {
expect(await getPermissions("admin")).toHaveLength(14); expect(await getPermissions("admin")).toHaveLength(17);
}); });
it("editor has exactly 7 permissions", async () => { it("editor has exactly 7 permissions", async () => {
tests/unit/api/permissions.test.ts
+5 -2
View File
@@ -25,9 +25,9 @@ import { getPermissions, hasPermission } from "../../../apps/api/src/permissions
describe("permissions", () => { describe("permissions", () => {
describe("getPermissions", () => { describe("getPermissions", () => {
it("returns all 14 permissions for admin", async () => { it("returns all 17 permissions for admin", async () => {
const perms = await getPermissions("admin"); const perms = await getPermissions("admin");
expect(perms).toHaveLength(14); expect(perms).toHaveLength(17);
expect(perms).toContain("tools:use"); expect(perms).toContain("tools:use");
expect(perms).toContain("files:own"); expect(perms).toContain("files:own");
expect(perms).toContain("files:all"); expect(perms).toContain("files:all");
@@ -42,6 +42,9 @@ describe("permissions", () => {
expect(perms).toContain("features:manage"); expect(perms).toContain("features:manage");
expect(perms).toContain("system:health"); expect(perms).toContain("system:health");
expect(perms).toContain("audit:read"); expect(perms).toContain("audit:read");
expect(perms).toContain("compliance:manage");
expect(perms).toContain("webhooks:manage");
expect(perms).toContain("security:manage");
}); });
it("returns only basic permissions for user role", async () => { it("returns only basic permissions for user role", async () => {
tests/unit/api/rbac-enforcement.test.ts
+5 -2
View File
@@ -16,7 +16,7 @@ vi.mock("../../../apps/api/src/plugins/auth.js", () => ({
import { getPermissions, hasPermission } from "../../../apps/api/src/permissions.js"; import { getPermissions, hasPermission } from "../../../apps/api/src/permissions.js";
describe("role permissions", () => { describe("role permissions", () => {
it("admin has all 14 permissions", async () => { it("admin has all 17 permissions", async () => {
const perms = await getPermissions("admin"); const perms = await getPermissions("admin");
expect(perms).toContain("tools:use"); expect(perms).toContain("tools:use");
expect(perms).toContain("files:all"); expect(perms).toContain("files:all");
@@ -24,7 +24,10 @@ describe("role permissions", () => {
expect(perms).toContain("features:manage"); expect(perms).toContain("features:manage");
expect(perms).toContain("system:health"); expect(perms).toContain("system:health");
expect(perms).toContain("audit:read"); expect(perms).toContain("audit:read");
expect(perms.length).toBe(14); expect(perms).toContain("compliance:manage");
expect(perms).toContain("webhooks:manage");
expect(perms).toContain("security:manage");
expect(perms.length).toBe(17);
}); });
it("editor has collaborative but not admin permissions", async () => { it("editor has collaborative but not admin permissions", async () => {