Erstzugang schuetzen, Kettenstatus richtigstellen, CCs nachtraeglich aendern
Der Zugang aus der Erstinstallation ist ab Werk unantastbar: weder Rolle noch Zustand lassen sich aendern, sein Passwort nicht von aussen zuruecksetzen, und loeschen laesst er sich nicht - auch nicht durch sich selbst. So bleibt ein Weg zurueck, wenn sich jemand aus Versehen die Rechte entzieht. Sein eigenes Passwort aendert er weiterhin unter "Konto". BOOTSTRAP_ADMIN_EDITABLE=true hebt den Schutz auf. Welcher Zugang gemeint ist, traegt der Start nach: den Namen kennt nur er, die Migration nicht. Ein CC behielt nach dem Verschieben seines Redesigns den Vermerk "Redesignt", obwohl ueber ihm nichts mehr stand. Der Vermerk stammte von einem Nachfolger, der inzwischen woanders lag. Jetzt folgt er der Kette: wer einen Nachfolger hat, wurde ersetzt - wer ans Ende rueckt, nicht mehr. Ein Platz entsteht dabei nicht aus dem Nichts, die Deckung wird geprueft wie sonst auch. Neu bearbeitbar: Name, Beschreibung und der verknuepfte Forums-Antrag eines CCs. Ein Antrag, den der Abgleich noch nicht kennt, wird dabei vorgemerkt - sonst staende er beim naechsten Lauf als offener Antrag da, obwohl er laengst eingetragen ist. Ebenso laesst sich nachtraeglich umstellen, ob ein Redesign bezahlt oder Kulanz war; umgebucht wird ueber eine Gegenbuchung, ueberschrieben wird nichts. Zuletzt: Eintraege eines anderen Editors kamen nicht an, solange die Seite offen stand. Jetzt wird der Stand geholt, sobald der Tab wieder in den Vordergrund kommt - das trifft den Moment, in dem jemand wieder hinsieht, ohne im Hintergrund dauernd zu fragen.
This commit is contained in:
@@ -219,6 +219,115 @@ describe('Rollen', () => {
|
||||
});
|
||||
});
|
||||
|
||||
describe('Der Zugang aus der Erstinstallation', () => {
|
||||
/** Legt einen geschützten Admin an, wie es der erste Start tut. */
|
||||
async function erstzugang() {
|
||||
return db.prisma.user.create({
|
||||
data: {
|
||||
username: 'erstzugang',
|
||||
passwordHash: 'x',
|
||||
role: 'ADMIN',
|
||||
isProtected: true,
|
||||
},
|
||||
});
|
||||
}
|
||||
|
||||
it('lässt sich seine Rolle nicht nehmen', async () => {
|
||||
const geschuetzt = await erstzugang();
|
||||
const credentials = await login('chef');
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'PATCH',
|
||||
url: `/api/admin/users/${geschuetzt.id}`,
|
||||
headers: authHeaders(credentials),
|
||||
payload: { role: 'VIEWER' },
|
||||
});
|
||||
|
||||
assert.equal(response.statusCode, 409);
|
||||
assert.equal(response.json().error, 'PROTECTED_ACCOUNT');
|
||||
|
||||
const unveraendert = await db.prisma.user.findUniqueOrThrow({ where: { id: geschuetzt.id } });
|
||||
assert.equal(unveraendert.role, 'ADMIN');
|
||||
});
|
||||
|
||||
it('lässt sich nicht sperren', async () => {
|
||||
const geschuetzt = await db.prisma.user.findFirstOrThrow({ where: { isProtected: true } });
|
||||
const credentials = await login('chef');
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'PATCH',
|
||||
url: `/api/admin/users/${geschuetzt.id}`,
|
||||
headers: authHeaders(credentials),
|
||||
payload: { isActive: false },
|
||||
});
|
||||
|
||||
assert.equal(response.statusCode, 409);
|
||||
});
|
||||
|
||||
it('bekommt sein Passwort nicht von aussen zurückgesetzt', async () => {
|
||||
const geschuetzt = await db.prisma.user.findFirstOrThrow({ where: { isProtected: true } });
|
||||
const credentials = await login('chef');
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'POST',
|
||||
url: `/api/admin/users/${geschuetzt.id}/reset-password`,
|
||||
headers: authHeaders(credentials),
|
||||
payload: { newPassword: 'EinLangesPasswort1' },
|
||||
});
|
||||
|
||||
assert.equal(response.statusCode, 409);
|
||||
});
|
||||
|
||||
it('lässt sich nicht löschen', async () => {
|
||||
const geschuetzt = await db.prisma.user.findFirstOrThrow({ where: { isProtected: true } });
|
||||
const credentials = await login('chef');
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'DELETE',
|
||||
url: `/api/admin/users/${geschuetzt.id}`,
|
||||
headers: authHeaders(credentials),
|
||||
});
|
||||
|
||||
assert.equal(response.statusCode, 409);
|
||||
assert.ok(await db.prisma.user.findUnique({ where: { id: geschuetzt.id } }));
|
||||
});
|
||||
|
||||
it('ist in der Liste als geschützt ausgewiesen', async () => {
|
||||
const credentials = await login('chef');
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'GET',
|
||||
url: '/api/admin/users',
|
||||
headers: authHeaders(credentials),
|
||||
});
|
||||
|
||||
const eintrag = response
|
||||
.json()
|
||||
.items.find((u: { username: string }) => u.username === 'erstzugang');
|
||||
assert.equal(eintrag.isLocked, true, 'die Oberfläche soll die Knöpfe gar nicht erst zeigen');
|
||||
});
|
||||
|
||||
it('lässt gewöhnliche Konten unangetastet', async () => {
|
||||
const credentials = await login('chef');
|
||||
const normal = await db.prisma.user.findFirstOrThrow({ where: { username: 'gast' } });
|
||||
|
||||
const response = await app.inject({
|
||||
method: 'PATCH',
|
||||
url: `/api/admin/users/${normal.id}`,
|
||||
headers: authHeaders(credentials),
|
||||
payload: { role: 'EDITOR' },
|
||||
});
|
||||
|
||||
assert.equal(response.statusCode, 200, 'der Schutz gilt nur für den Erstzugang');
|
||||
await app.inject({
|
||||
method: 'PATCH',
|
||||
url: `/api/admin/users/${normal.id}`,
|
||||
headers: authHeaders(credentials),
|
||||
payload: { role: 'VIEWER' },
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('Erstpasswort', () => {
|
||||
it('sperrt schreibende Aktionen bis zum Wechsel', async () => {
|
||||
const credentials = await login('neu');
|
||||
|
||||
+224
-7
@@ -152,15 +152,23 @@ describe('Lücken in einer Redesign-Kette schliessen', () => {
|
||||
verkettungStimmt(zweite);
|
||||
});
|
||||
|
||||
it('löst einen Eintrag wieder heraus', async () => {
|
||||
it('holt einen falsch einsortierten Eintrag wieder heraus', async () => {
|
||||
// Zwei eigenständige CCs, einer davon versehentlich in die Kette des
|
||||
// anderen gezogen. Beide sind aus einem eigenen Paket bezahlt.
|
||||
const spieler = await spielerMitPaketen('Loeser', 2);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'Bleibt' }, ctx);
|
||||
await credits.bookRedesignPackage(spieler, ctx);
|
||||
const zwei = await characters.redesignCharacter(
|
||||
eins.character.id,
|
||||
{ name: 'Geht wieder', useKind: 'REDESIGN_PAID' },
|
||||
const zwei = await characters.createCharacter(spieler, { name: 'Gehoert woanders hin' }, ctx);
|
||||
|
||||
await characters.moveInChain(
|
||||
zwei.character.id,
|
||||
{ lineageId: eins.character.lineageId, afterId: eins.character.id },
|
||||
ctx,
|
||||
);
|
||||
// In der Kette gilt: wer einen Nachfolger hat, wurde ersetzt.
|
||||
const eingereiht = await db.prisma.character.findUniqueOrThrow({
|
||||
where: { id: eins.character.id },
|
||||
});
|
||||
assert.equal(eingereiht.inactiveReason, 'REWORKED');
|
||||
|
||||
const { lineageId } = await characters.detachFromChain(zwei.character.id, ctx);
|
||||
assert.notEqual(lineageId, eins.character.lineageId);
|
||||
@@ -168,12 +176,67 @@ describe('Lücken in einer Redesign-Kette schliessen', () => {
|
||||
const alte = await kette(spieler, eins.character.lineageId);
|
||||
assert.deepEqual(alte.map((g) => g.name), ['Bleibt']);
|
||||
|
||||
// Und ohne Nachfolger ist "redesignt" hinfällig.
|
||||
const zurueck = await db.prisma.character.findUniqueOrThrow({
|
||||
where: { id: eins.character.id },
|
||||
});
|
||||
assert.equal(zurueck.status, 'ACTIVE');
|
||||
assert.equal(zurueck.inactiveReason, null);
|
||||
|
||||
const allein = await db.prisma.character.findUniqueOrThrow({
|
||||
where: { id: zwei.character.id },
|
||||
});
|
||||
assert.equal(allein.predecessorId, null, 'allein heisst ohne Vorgänger');
|
||||
});
|
||||
|
||||
it('lässt ein Redesign nicht ohne eigenen Platz allein stehen', async () => {
|
||||
// Ein Redesign hat keine eigene CC-Buchung - es tauscht einen Platz aus.
|
||||
// Allein gestellt braucht es einen, und der muss gedeckt sein.
|
||||
const spieler = await spielerMitPaketen('Knapp', 1);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'Original' }, ctx);
|
||||
await credits.bookRedesignPackage(spieler, ctx);
|
||||
const zwei = await characters.redesignCharacter(
|
||||
eins.character.id,
|
||||
{ name: 'Neue Fassung', useKind: 'REDESIGN_PAID' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
await assert.rejects(
|
||||
() => characters.detachFromChain(zwei.character.id, ctx),
|
||||
(error: Error & { code?: string }) => error.code === 'NO_FREE_SLOT',
|
||||
'sonst stünden zwei aktive CCs auf einem Paket',
|
||||
);
|
||||
});
|
||||
|
||||
it('räumt die Redesign-Markierung ab, wenn der Nachfolger wegzieht', async () => {
|
||||
// Genau der Fall aus dem Betrieb: das Redesign gehörte in eine andere
|
||||
// Kette. Was zurückbleibt, ist von niemandem mehr ersetzt worden.
|
||||
const spieler = await spielerMitPaketen('Umsortiert', 2);
|
||||
const platzA = await characters.createCharacter(spieler, { name: 'Platz A' }, ctx);
|
||||
const platzB = await characters.createCharacter(spieler, { name: 'Platz B' }, ctx);
|
||||
|
||||
await credits.bookRedesignPackage(spieler, ctx);
|
||||
const redesign = await characters.redesignCharacter(
|
||||
platzB.character.id,
|
||||
{ name: 'Redesign von B', useKind: 'REDESIGN_PAID' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
// B ist jetzt redesignt. Nun stellt sich heraus: das Redesign gehört zu A.
|
||||
await characters.moveInChain(
|
||||
redesign.character.id,
|
||||
{ lineageId: platzA.character.lineageId, afterId: platzA.character.id },
|
||||
ctx,
|
||||
);
|
||||
|
||||
const b = await db.prisma.character.findUniqueOrThrow({ where: { id: platzB.character.id } });
|
||||
assert.equal(b.status, 'ACTIVE', 'ohne Nachfolger ist B wieder der aktuelle CC');
|
||||
assert.equal(b.inactiveReason, null);
|
||||
|
||||
const a = await db.prisma.character.findUniqueOrThrow({ where: { id: platzA.character.id } });
|
||||
assert.equal(a.inactiveReason, 'REWORKED', 'A hat jetzt einen Nachfolger');
|
||||
});
|
||||
|
||||
it('lässt keinen CC in die Kette eines anderen Spielers', async () => {
|
||||
const einer = await spielerMitPaketen('Einer', 1);
|
||||
const anderer = await spielerMitPaketen('Anderer', 1);
|
||||
@@ -206,7 +269,7 @@ describe('Lücken in einer Redesign-Kette schliessen', () => {
|
||||
);
|
||||
});
|
||||
|
||||
it('rührt den Status nicht an', async () => {
|
||||
it('macht aus einem eingereihten CC einen redesignten', async () => {
|
||||
const spieler = await spielerMitPaketen('Unberuehrt', 2);
|
||||
const aktiv = await characters.createCharacter(spieler, { name: 'Aktiv' }, ctx);
|
||||
const ziel = await characters.createCharacter(spieler, { name: 'Ziel' }, ctx);
|
||||
@@ -220,7 +283,10 @@ describe('Lücken in einer Redesign-Kette schliessen', () => {
|
||||
const danach = await db.prisma.character.findUniqueOrThrow({
|
||||
where: { id: aktiv.character.id },
|
||||
});
|
||||
assert.equal(danach.status, 'ACTIVE', 'verschieben ordnet nur ein, es entscheidet nichts');
|
||||
assert.equal(danach.status, 'ACTIVE', 'am Ende der Kette bleibt er der aktuelle');
|
||||
|
||||
const davor = await db.prisma.character.findUniqueOrThrow({ where: { id: ziel.character.id } });
|
||||
assert.equal(davor.inactiveReason, 'REWORKED', 'wer einen Nachfolger bekommt, ist ersetzt');
|
||||
});
|
||||
});
|
||||
|
||||
@@ -277,3 +343,154 @@ describe('Verweise an einem CC', () => {
|
||||
assert.equal(await db.prisma.characterLink.count({ where: { characterId: cc.character.id } }), 0);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Redesign nachträglich umbuchen', () => {
|
||||
it('stellt von bezahlt auf Kulanz um, ohne die Geschichte zu verbiegen', async () => {
|
||||
const spieler = await spielerMitPaketen('Umbucher', 1);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'Erste Fassung' }, ctx);
|
||||
const zwei = await characters.redesignCharacter(
|
||||
eins.character.id,
|
||||
{ name: 'Zweite Fassung', useKind: 'REDESIGN_PAID' },
|
||||
ctx,
|
||||
);
|
||||
assert.equal(zwei.balances.REDESIGN_PAID, 0, 'das bezahlte ist verbraucht');
|
||||
|
||||
await credits.bookKulanz(spieler, 1, 'Absprache aufgetaucht', ctx);
|
||||
const danach = await characters.changeRedesignKind(zwei.character.id, 'REDESIGN_KULANZ', ctx);
|
||||
|
||||
assert.equal(danach.balances.REDESIGN_PAID, 1, 'das bezahlte ist wieder frei');
|
||||
assert.equal(danach.balances.REDESIGN_KULANZ, 0, 'dafür ist die Kulanz verbraucht');
|
||||
|
||||
// Korrigiert wird durch Gegenbuchung, nicht durch Überschreiben.
|
||||
const zeilen = await db.prisma.creditLedger.findMany({
|
||||
where: { characterId: zwei.character.id },
|
||||
orderBy: { id: 'asc' },
|
||||
});
|
||||
assert.equal(zeilen.length, 3, 'Verbrauch, Storno, neuer Verbrauch');
|
||||
assert.equal(zeilen[1]?.source, 'REVERSAL');
|
||||
assert.equal(zeilen[2]?.kind, 'REDESIGN_KULANZ');
|
||||
});
|
||||
|
||||
it('und wieder zurück', async () => {
|
||||
const spieler = await spielerMitPaketen('Hin und her', 1);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'A' }, ctx);
|
||||
await credits.bookKulanz(spieler, 1, 'Erstmal Kulanz', ctx);
|
||||
const zwei = await characters.redesignCharacter(
|
||||
eins.character.id,
|
||||
{ name: 'B', useKind: 'REDESIGN_KULANZ' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
const danach = await characters.changeRedesignKind(zwei.character.id, 'REDESIGN_PAID', ctx);
|
||||
assert.equal(danach.balances.REDESIGN_KULANZ, 1);
|
||||
assert.equal(danach.balances.REDESIGN_PAID, 0, 'das aus dem CC-Paket ist jetzt dran');
|
||||
});
|
||||
|
||||
it('weist dieselbe Art ab', async () => {
|
||||
const spieler = await spielerMitPaketen('Unveraendert', 1);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'X' }, ctx);
|
||||
const zwei = await characters.redesignCharacter(
|
||||
eins.character.id,
|
||||
{ name: 'Y', useKind: 'REDESIGN_PAID' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
await assert.rejects(
|
||||
() => characters.changeRedesignKind(zwei.character.id, 'REDESIGN_PAID', ctx),
|
||||
(error: Error & { code?: string }) => error.code === 'SAME_KIND',
|
||||
);
|
||||
});
|
||||
|
||||
it('weist einen CC ohne Redesign-Buchung ab', async () => {
|
||||
const spieler = await spielerMitPaketen('Ohne Redesign', 1);
|
||||
const eins = await characters.createCharacter(spieler, { name: 'Direkt aus dem Paket' }, ctx);
|
||||
|
||||
await assert.rejects(
|
||||
() => characters.changeRedesignKind(eins.character.id, 'REDESIGN_KULANZ', ctx),
|
||||
(error: Error & { code?: string }) => error.code === 'NO_REDESIGN_BOOKING',
|
||||
'ein CC aus dem Paket ist kein Redesign',
|
||||
);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Angaben eines CCs nachtragen', () => {
|
||||
it('ändert Namen und Beschreibung', async () => {
|
||||
const spieler = await spielerMitPaketen('Korrigierer', 1);
|
||||
const cc = await characters.createCharacter(spieler, { name: 'Tipfehler' }, ctx);
|
||||
|
||||
await characters.updateCharacter(
|
||||
cc.character.id,
|
||||
{ name: 'Tippfehler behoben', loreText: 'Jetzt mit Beschreibung' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
const danach = await db.prisma.character.findUniqueOrThrow({ where: { id: cc.character.id } });
|
||||
assert.equal(danach.name, 'Tippfehler behoben');
|
||||
assert.equal(danach.loreText, 'Jetzt mit Beschreibung');
|
||||
});
|
||||
|
||||
it('hängt einen Antrag an, den der Abgleich schon kennt', async () => {
|
||||
const spieler = await spielerMitPaketen('Verknuepfer', 1);
|
||||
const cc = await characters.createCharacter(spieler, { name: 'Mit Antrag' }, ctx);
|
||||
|
||||
await db.prisma.forumThread.create({
|
||||
data: {
|
||||
threadId: 7771,
|
||||
title: 'Sein Antrag',
|
||||
url: 'https://forum.example.org/index.php?thread/7771-sein-antrag/',
|
||||
currentBoard: 'CC_ARCHIV',
|
||||
},
|
||||
});
|
||||
|
||||
await characters.updateCharacter(
|
||||
cc.character.id,
|
||||
{ threadRef: 'https://forum.example.org/index.php?thread/7771-sein-antrag/' },
|
||||
ctx,
|
||||
);
|
||||
|
||||
const thread = await db.prisma.forumThread.findUniqueOrThrow({ where: { threadId: 7771 } });
|
||||
assert.equal(thread.linkedCharacterId, cc.character.id);
|
||||
assert.equal(thread.needsClassification, false, 'er ist damit erledigt');
|
||||
});
|
||||
|
||||
it('merkt einen unbekannten Antrag vor', async () => {
|
||||
const spieler = await spielerMitPaketen('Vormerker', 1);
|
||||
const cc = await characters.createCharacter(spieler, { name: 'Noch ungelesen' }, ctx);
|
||||
|
||||
// Nur die Nummer, und der Abgleich kennt den Thread nicht.
|
||||
await characters.updateCharacter(cc.character.id, { threadRef: '8888' }, ctx);
|
||||
|
||||
const thread = await db.prisma.forumThread.findUniqueOrThrow({ where: { threadId: 8888 } });
|
||||
assert.equal(thread.linkedCharacterId, cc.character.id);
|
||||
assert.equal(
|
||||
thread.needsClassification,
|
||||
false,
|
||||
'sonst stünde er beim nächsten Lauf als offener Antrag da',
|
||||
);
|
||||
});
|
||||
|
||||
it('löst eine Verknüpfung wieder', async () => {
|
||||
const spieler = await spielerMitPaketen('Loeser2', 1);
|
||||
const cc = await characters.createCharacter(spieler, { name: 'Doch nicht' }, ctx);
|
||||
await characters.updateCharacter(cc.character.id, { threadRef: '9999' }, ctx);
|
||||
|
||||
await characters.updateCharacter(cc.character.id, { threadRef: null }, ctx);
|
||||
|
||||
const thread = await db.prisma.forumThread.findUniqueOrThrow({ where: { threadId: 9999 } });
|
||||
assert.equal(thread.linkedCharacterId, null);
|
||||
|
||||
const liste = await characters.listCharacters(spieler);
|
||||
const eintrag = liste.find((c) => c.id === cc.character.id);
|
||||
assert.deepEqual(eintrag?.forumThreads, [], 'am CC hängt kein Antrag mehr');
|
||||
});
|
||||
|
||||
it('weist eine Eingabe ab, die keine Thread-Angabe ist', async () => {
|
||||
const spieler = await spielerMitPaketen('Vertipper', 1);
|
||||
const cc = await characters.createCharacter(spieler, { name: 'Sauber' }, ctx);
|
||||
|
||||
await assert.rejects(
|
||||
() => characters.updateCharacter(cc.character.id, { threadRef: 'keine ahnung' }, ctx),
|
||||
(error: Error & { code?: string }) => error.code === 'INVALID_THREAD',
|
||||
);
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user