Vier der acht Änderungswünsche. **Knöpfe.** Rund vierzig der siebzig Knöpfe waren blosser Text in Akzentfarbe, von einem Link nicht zu unterscheiden. Jeder trägt jetzt Rahmen und Grundfläche, über die Klasse .btn an einer Stelle definiert statt siebzigmal als Klassenkette. Beim Drücken zieht der Knopf sich zusammen und bekommt einen inneren Ring — er wirkt kurz dicker, ohne dass die Rahmenbreite sich ändert und das Umfeld springt; die Rückmeldung setzt ohne Verzögerung ein und klingt langsam ab, sonst bliebe ein kurzer Klick unsichtbar. Dazu die neue Regel: Flächen bleiben eckig, alles Bedienbare ist rund. Damit ist die Rundung eine Auskunft und keine Verzierung. **Verwaltung.** Rollen, Vorlagen und Änderungsrichtlinie waren drei Einträge in der Kopfzeile für etwas, das man selten und dann zusammenhängend braucht. Jetzt ein Eintrag /admin mit Reiterleiste, dazu eine gemeinsame Tabs-Komponente, die auch der Fallakte zeigt, welcher Reiter offen ist. **Kanäle.** Ersatzlos entfernt, mitsamt Tabelle. Der Chat gehört dorthin, wo der Vorgang steht; ein eigener Reiter mit freien Räumen war der Rückweg nach Discord. Die Personalakte hat jetzt denselben Chat wie die Fallakte — ohne „als Vermerk übernehmen", denn dafür braucht es einen Verlauf, und Meldungen gehen nur an Erwähnte, nie in den allgemeinen Kanal: dass es zu jemandem eine Personalakte gibt, ist selbst eine Auskunft. Ein CHECK erzwingt jetzt, dass eine Nachricht zu genau einem Ort gehört. **Kopfzeile.** Suchbalken statt Suchlink, als gewöhnliches Formular ohne ein Byte im Bündel. Thema, Benachrichtigungen und Abmelden sitzen hinter dem Profilbild, das von Discord kommt; wer keins hat, bekommt seine Initialen. Nebenbei: Aktenzeichen in Tests waren nur auf vier Stellen eindeutig und kollidierten zwischen Läufen, weil Akten nicht löschbar sind.
190 lines
6.0 KiB
TypeScript
190 lines
6.0 KiB
TypeScript
// @vitest-environment node
|
|
import { afterAll, beforeAll, describe, expect, it } from 'vitest'
|
|
import { createPrismaClient, loadEnvFromRepoRoot, type PrismaClient } from '@afis/db'
|
|
import { visibleCaseWhere, visibleEntryWhere } from '@/lib/access/caseFilter'
|
|
import type { EffectivePermissions, Permission } from '@/lib/auth/permissions'
|
|
|
|
/**
|
|
* Die Negativtests aus docs/03-auth-und-rechte.md.
|
|
*
|
|
* Sie laufen gegen eine echte Datenbank, weil genau das der Punkt ist: die
|
|
* Prüfung muss in der Abfrage stecken, nicht in einem Filter danach. Ein Test
|
|
* gegen eine ausgedachte Liste würde die Aussage nicht treffen.
|
|
*/
|
|
loadEnvFromRepoRoot()
|
|
const url = process.env.DIRECT_DATABASE_URL ?? process.env.DATABASE_URL
|
|
const suite = url ? describe : describe.skip
|
|
|
|
let db: PrismaClient
|
|
const marke = `zugriff-${Date.now()}`
|
|
|
|
function perms(
|
|
userId: string,
|
|
clearanceLevel: number,
|
|
options: { isAdmin?: boolean; permissions?: Permission[] } = {},
|
|
): EffectivePermissions {
|
|
return {
|
|
userId,
|
|
clearanceLevel,
|
|
permissions: new Set<Permission>(options.permissions ?? []),
|
|
isAdmin: options.isAdmin ?? false,
|
|
hasMembership: true,
|
|
}
|
|
}
|
|
|
|
suite('Sichtbarkeit von Akten', () => {
|
|
const ids = {
|
|
offen: `${marke}-offen`,
|
|
geheim: `${marke}-geheim`,
|
|
versiegelt: `${marke}-versiegelt`,
|
|
}
|
|
const nutzer = {
|
|
stufe1: `${marke}-stufe1`,
|
|
stufe2: `${marke}-stufe2`,
|
|
stufe5: `${marke}-stufe5`,
|
|
beteiligt: `${marke}-beteiligt`,
|
|
admin: `${marke}-admin`,
|
|
}
|
|
|
|
beforeAll(async () => {
|
|
db = createPrismaClient(url)
|
|
|
|
const template = await db.documentTemplate.create({
|
|
data: { key: `${marke}-vorlage`, version: 1, name: 'Prüfvorlage', fields: [] },
|
|
})
|
|
|
|
for (const id of Object.values(nutzer)) {
|
|
await db.user.create({ data: { id, username: id, displayName: id } })
|
|
}
|
|
|
|
const gemeinsam = { templateId: template.id, templateKey: template.key, createdById: nutzer.stufe5 }
|
|
|
|
await db.case.create({
|
|
data: {
|
|
...gemeinsam,
|
|
id: ids.offen,
|
|
caseNumber: `AfiS-9001-${Date.now().toString(36)}${Math.random().toString(36).slice(2, 6)}`,
|
|
title: 'Offener Vorgang',
|
|
classification: 1,
|
|
status: 'OPEN',
|
|
},
|
|
})
|
|
|
|
await db.case.create({
|
|
data: {
|
|
...gemeinsam,
|
|
id: ids.geheim,
|
|
caseNumber: `AfiS-9002-${Date.now().toString(36)}${Math.random().toString(36).slice(2, 6)}`,
|
|
title: 'Geheimer Vorgang',
|
|
classification: 4,
|
|
status: 'OPEN',
|
|
},
|
|
})
|
|
|
|
await db.case.create({
|
|
data: {
|
|
...gemeinsam,
|
|
id: ids.versiegelt,
|
|
caseNumber: `AfiS-9003-${Date.now().toString(36)}${Math.random().toString(36).slice(2, 6)}`,
|
|
title: 'Versiegelter Vorgang',
|
|
classification: 1,
|
|
status: 'SEALED',
|
|
},
|
|
})
|
|
|
|
// Beteiligter an der geheimen Akte, obwohl seine Stufe nicht reicht.
|
|
await db.caseParticipant.create({
|
|
data: { caseId: ids.geheim, userId: nutzer.beteiligt, role: 'WITNESS' },
|
|
})
|
|
|
|
// Ausgeschlossen von der offenen Akte, obwohl Stufe 5 und Admin.
|
|
await db.caseExclusion.createMany({
|
|
data: [
|
|
{ caseId: ids.offen, userId: nutzer.stufe5, reason: 'Betroffener' },
|
|
{ caseId: ids.offen, userId: nutzer.admin, reason: 'Betroffener' },
|
|
],
|
|
})
|
|
})
|
|
|
|
afterAll(async () => {
|
|
await db.$disconnect()
|
|
})
|
|
|
|
async function sichtbar(p: EffectivePermissions): Promise<string[]> {
|
|
const rows = await db.case.findMany({
|
|
where: { AND: [visibleCaseWhere(p), { id: { in: Object.values(ids) } }] },
|
|
select: { id: true },
|
|
})
|
|
return rows.map((r) => r.id).sort()
|
|
}
|
|
|
|
it('zeigt einer Stufe 2 die Stufe-4-Akte nicht, auch nicht als Treffer', async () => {
|
|
const p = perms(nutzer.stufe2, 2)
|
|
const treffer = await sichtbar(p)
|
|
expect(treffer).not.toContain(ids.geheim)
|
|
expect(treffer).toContain(ids.offen)
|
|
|
|
// Auch die Zählung darf sie nicht enthalten, sonst verrät die Lücke sie.
|
|
const anzahl = await db.case.count({
|
|
where: { AND: [visibleCaseWhere(p), { id: { in: Object.values(ids) } }] },
|
|
})
|
|
expect(anzahl).toBe(treffer.length)
|
|
})
|
|
|
|
it('gibt einem ausgeschlossenen Nutzer mit Stufe 5 keinen Zugriff', async () => {
|
|
const treffer = await sichtbar(perms(nutzer.stufe5, 5))
|
|
expect(treffer).not.toContain(ids.offen)
|
|
})
|
|
|
|
it('schlägt selbst den Admin, wenn er ausgeschlossen ist', async () => {
|
|
// Der Ausschluss deckt den Fall ab, dass gegen einen hochrangigen
|
|
// Mitarbeiter ermittelt wird. Der Administrator ist einer.
|
|
const treffer = await sichtbar(perms(nutzer.admin, 5, { isAdmin: true }))
|
|
expect(treffer).not.toContain(ids.offen)
|
|
expect(treffer).toContain(ids.geheim)
|
|
})
|
|
|
|
it('lässt einen Beteiligten auch über seiner Stufe lesen', async () => {
|
|
const treffer = await sichtbar(perms(nutzer.beteiligt, 1))
|
|
expect(treffer).toContain(ids.geheim)
|
|
})
|
|
|
|
it('verbirgt versiegelte Akten unterhalb von Stufe 5', async () => {
|
|
expect(await sichtbar(perms(nutzer.stufe2, 2))).not.toContain(ids.versiegelt)
|
|
expect(await sichtbar(perms(nutzer.stufe1, 1))).not.toContain(ids.versiegelt)
|
|
expect(await sichtbar(perms(nutzer.stufe5, 5))).toContain(ids.versiegelt)
|
|
})
|
|
|
|
it('verbirgt Verlaufseinträge oberhalb der Stufe des Lesers', async () => {
|
|
await db.caseEntry.createMany({
|
|
data: [
|
|
{
|
|
id: `${marke}-eintrag-offen`,
|
|
caseId: ids.offen,
|
|
type: 'NOTE',
|
|
body: 'Für alle lesbar',
|
|
authorId: nutzer.stufe1,
|
|
minClearance: 0,
|
|
},
|
|
{
|
|
id: `${marke}-eintrag-hoch`,
|
|
caseId: ids.offen,
|
|
type: 'NOTE',
|
|
body: 'Nur ab Stufe 4',
|
|
authorId: nutzer.stufe1,
|
|
minClearance: 4,
|
|
},
|
|
],
|
|
})
|
|
|
|
const p = perms(nutzer.stufe1, 1)
|
|
const eintraege = await db.caseEntry.findMany({
|
|
where: { AND: [visibleEntryWhere(p), { caseId: ids.offen }] },
|
|
select: { id: true },
|
|
})
|
|
const gefunden = eintraege.map((e) => e.id)
|
|
expect(gefunden).toContain(`${marke}-eintrag-offen`)
|
|
expect(gefunden).not.toContain(`${marke}-eintrag-hoch`)
|
|
})
|
|
})
|