mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
AshimandGitHub
5a45bcbc8f
* feat(rbac): add editor role, 3 new permissions, ownership helper * feat(rbac): add audit_log table, apiKeys.permissions column, editor role to schema * feat(rbac): wire requirePermission into all routes, add editor role support * refactor(rbac): replace ad-hoc role checks with permission-based ownership * feat(rbac): add audit log DB writes + query endpoint Dual-write audit events to stdout (existing) and SQLite audit_log table. Add GET /api/v1/audit-log with pagination, action filter, and date range filtering, gated behind audit:read permission. * feat(rbac): add API key permission scoping with ceiling enforcement * feat(rbac): add escalation prevention and last-admin protection * feat(rbac): add editor role to UI, API key permission scoping in settings * test(rbac): add full permission matrix integration test * test(rbac): add editor role E2E tests * feat(rbac): add custom roles with CRUD API and DB-backed permission lookup * feat(rbac): add API key expiration * feat(rbac): add roles management UI and API key expiration to settings * feat(rbac): add audit log UI to settings * fix: remove any cast in API key permission validation * test(rbac): add unit tests for username validation rules * test(rbac): add unit tests for effective permissions and ownership * test(rbac): add comprehensive route permission matrix (all routes × all roles) * test(rbac): add auth route edge case tests (login failures, session expiry, password side effects) * test(rbac): add escalation prevention tests (register, update, self-demote, last-admin) * test(rbac): add ownership enforcement tests (files, pipelines, editor access, cross-user isolation) * test(rbac): add API key edge cases (name validation, delete behavior, key revocation) * test(rbac): add audit log edge cases (all events, pagination clamping, structure) * test(rbac): add custom roles edge case tests (validation, CRUD, functional permissions) * test(rbac): add comprehensive E2E tests (roles UI, audit log, custom role, API key scoping)
54 lines
2.0 KiB
TypeScript
54 lines
2.0 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { getPermissions, hasPermission } from "../../../apps/api/src/permissions.js";
|
|
|
|
describe("role permissions", () => {
|
|
it("admin has all 15 permissions", () => {
|
|
const perms = getPermissions("admin");
|
|
expect(perms).toContain("tools:use");
|
|
expect(perms).toContain("files:all");
|
|
expect(perms).toContain("users:manage");
|
|
expect(perms).toContain("features:manage");
|
|
expect(perms).toContain("system:health");
|
|
expect(perms).toContain("audit:read");
|
|
expect(perms.length).toBe(15);
|
|
});
|
|
|
|
it("editor has collaborative but not admin permissions", () => {
|
|
const perms = getPermissions("editor");
|
|
expect(perms).toContain("tools:use");
|
|
expect(perms).toContain("files:own");
|
|
expect(perms).toContain("files:all");
|
|
expect(perms).toContain("pipelines:all");
|
|
expect(perms).toContain("settings:read");
|
|
expect(perms).not.toContain("users:manage");
|
|
expect(perms).not.toContain("settings:write");
|
|
expect(perms).not.toContain("teams:manage");
|
|
expect(perms).not.toContain("features:manage");
|
|
expect(perms).not.toContain("system:health");
|
|
expect(perms).not.toContain("audit:read");
|
|
});
|
|
|
|
it("user has basic permissions only", () => {
|
|
const perms = getPermissions("user");
|
|
expect(perms).toContain("tools:use");
|
|
expect(perms).toContain("files:own");
|
|
expect(perms).toContain("apikeys:own");
|
|
expect(perms).toContain("pipelines:own");
|
|
expect(perms).toContain("settings:read");
|
|
expect(perms).not.toContain("files:all");
|
|
expect(perms).not.toContain("users:manage");
|
|
});
|
|
|
|
it("unknown role returns empty permissions", () => {
|
|
const perms = getPermissions("bogus" as any);
|
|
expect(perms).toEqual([]);
|
|
});
|
|
|
|
it("hasPermission checks correctly", () => {
|
|
expect(hasPermission("admin", "users:manage")).toBe(true);
|
|
expect(hasPermission("editor", "users:manage")).toBe(false);
|
|
expect(hasPermission("user", "tools:use")).toBe(true);
|
|
expect(hasPermission("user", "files:all")).toBe(false);
|
|
});
|
|
});
|