Suche zeigt alle Arten, nur Arbeitsrollen öffnen die Akte
Punkt 1 und 2 der Wünsche, plus zwei Lücken, die dabei sichtbar wurden. **Suche.** Wer nach Alvarez sucht, landet auf den Fallakten und erfährt nie, dass es dazu auch eine Personalakte gibt: die Suche fand sie, aber hinter einem Reiter, auf den nichts hindeutete. Die Reiter tragen jetzt ihre Trefferzahl, und ein leeres Ergebnis nennt ausdrücklich, wo etwas liegt. Dabei fiel auf, dass searchPersonnel das Recht personnel.read nicht prüfte. Die Kommandopalette tat es, die Aktenseite tat es, die Suche nicht — über /suche?kind=personnel kam jeder mit ausreichender Stufe an eine Liste aller Personalakten mit Namen, Dienstgrad und Etiketten. Die Prüfung steht jetzt dort, wo diese Akten herausgegeben werden. Die Trefferzahl einer Art, die es für den Nutzer nicht gibt, ist null und nicht 0: eine Null wäre die Auskunft „hier ist nichts", und die steht ihm nicht zu. **Täter und Mittäter** als Beteiligtenrollen — und damit wurde eine ältere Lücke offensichtlich: ein Beteiligter durfte die Akte lesen, gleich in welcher Rolle. Bei „Betroffener" fiel das nicht auf, bei „Täter" wäre es offensichtlich falsch. Der Beschuldigte hätte die Ermittlung gegen sich selbst mitgelesen. Die Beteiligtenliste beantwortet zwei Fragen: wer bearbeitet den Fall, und um wen geht es darin. Nur die erste ist eine Berechtigung. WORKING_ROLES nennt die drei, die die Akte öffnen; Zeuge, Betroffener, Täter und Mittäter öffnen nichts. Das gilt an allen vier Stellen, die auf Beteiligung prüften: Aktenzugriff, Dokumentzugriff über die besitzende Akte, „meine Fälle" und der Filter „nur meine". Zwei bestehende Tests hatten die alte Annahme festgeschrieben und benutzten WITNESS, um Zugriff zu belegen; sie prüfen jetzt mit INVESTIGATOR. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
2fb120dd77
commit
970be0e320
@@ -92,9 +92,10 @@ suite('Sichtbarkeit von Akten', () => {
|
||||
},
|
||||
})
|
||||
|
||||
// Beteiligter an der geheimen Akte, obwohl seine Stufe nicht reicht.
|
||||
// Als Ermittler an der geheimen Akte, obwohl seine Stufe nicht reicht.
|
||||
// Die Rolle ist hier wesentlich: nur wer an der Akte arbeitet, öffnet sie.
|
||||
await db.caseParticipant.create({
|
||||
data: { caseId: ids.geheim, userId: nutzer.beteiligt, role: 'WITNESS' },
|
||||
data: { caseId: ids.geheim, userId: nutzer.beteiligt, role: 'INVESTIGATOR' },
|
||||
})
|
||||
|
||||
// Ausgeschlossen von der offenen Akte, obwohl Stufe 5 und Admin.
|
||||
@@ -144,7 +145,7 @@ suite('Sichtbarkeit von Akten', () => {
|
||||
expect(treffer).toContain(ids.geheim)
|
||||
})
|
||||
|
||||
it('lässt einen Beteiligten auch über seiner Stufe lesen', async () => {
|
||||
it('lässt einen Ermittler auch über seiner Stufe lesen', async () => {
|
||||
const treffer = await sichtbar(perms(nutzer.beteiligt, 1))
|
||||
expect(treffer).toContain(ids.geheim)
|
||||
})
|
||||
@@ -186,4 +187,52 @@ suite('Sichtbarkeit von Akten', () => {
|
||||
expect(gefunden).toContain(`${marke}-eintrag-offen`)
|
||||
expect(gefunden).not.toContain(`${marke}-eintrag-hoch`)
|
||||
})
|
||||
|
||||
it('öffnet die Akte nicht für Zeugen, Betroffene und Täter', async () => {
|
||||
// Die Beteiligtenliste beantwortet zwei Fragen: wer bearbeitet den Fall,
|
||||
// und um wen geht es darin. Nur die erste verschafft Zugang — sonst läse
|
||||
// der Beschuldigte die Ermittlung gegen sich selbst mit.
|
||||
const vorlage = await db.documentTemplate.findFirstOrThrow({
|
||||
where: { key: { startsWith: marke } },
|
||||
})
|
||||
const akte = await db.case.create({
|
||||
data: {
|
||||
templateId: vorlage.id,
|
||||
templateKey: vorlage.key,
|
||||
createdById: nutzer.stufe5,
|
||||
caseNumber: `Rol-2026-${Date.now().toString(36)}`,
|
||||
title: 'Ermittlung',
|
||||
classification: 4,
|
||||
status: 'OPEN',
|
||||
},
|
||||
})
|
||||
|
||||
const p = perms(nutzer.stufe1, 1)
|
||||
|
||||
for (const rolle of ['WITNESS', 'SUBJECT', 'SUSPECT', 'ACCOMPLICE'] as const) {
|
||||
await db.caseParticipant.create({
|
||||
data: { caseId: akte.id, userId: nutzer.stufe1, role: rolle },
|
||||
})
|
||||
|
||||
const sichtbar = await db.case.findFirst({
|
||||
where: { AND: [visibleCaseWhere(p), { id: akte.id }] },
|
||||
select: { id: true },
|
||||
})
|
||||
expect(sichtbar, `Rolle ${rolle} darf die Akte nicht öffnen`).toBeNull()
|
||||
|
||||
await db.caseParticipant.deleteMany({
|
||||
where: { caseId: akte.id, userId: nutzer.stufe1, role: rolle },
|
||||
})
|
||||
}
|
||||
|
||||
// Gegenprobe: als Ermittler geht es, sonst prüft der Test nichts.
|
||||
await db.caseParticipant.create({
|
||||
data: { caseId: akte.id, userId: nutzer.stufe1, role: 'INVESTIGATOR' },
|
||||
})
|
||||
const alsErmittler = await db.case.findFirst({
|
||||
where: { AND: [visibleCaseWhere(p), { id: akte.id }] },
|
||||
select: { id: true },
|
||||
})
|
||||
expect(alsErmittler).not.toBeNull()
|
||||
})
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user