Jedes Glied einer Liste kostet, auch in einem Gruppen-CC

Bisher zahlte nur, wer neu einstieg: wer in einer Kette mitgezogen
wurde oder einen Gruppen-CC mitbenutzte, stand darin ohne eigene
Buchung. Damit ging die Rechnung nicht auf - verbrauchte Redesigns sind
alle Glieder aller Listen minus die Anzahl der Listen, und das gilt fuer
jeden Nutzer einzeln.

Jetzt bucht jedes Glied: das unterste einen Platz, jedes weitere ein
Redesign. Wo eine Buchung fehlt, traegt der Abgleich sie nach, und das
Redesign eines Gruppen-CCs kostet jeden Nutzer sein eigenes.

Am Pfeil stehen alle offenen Buchungen eines Eintrags, nicht nur die
juengste - sonst bleibt eine zweite unsichtbar.

server/test/ketten.test.ts

@@ -716,8 +716,9 @@ describe('Die Stelle in der Kette bestimmt, was ein Eintrag kostet', () => {
     );
   });
 
-  it('lässt einen Eintrag ohne Buchung in Ruhe', async () => {
-    // Nacherfassung ohne Guthaben-Angabe - da gibt es nichts umzustellen.
+  it('trägt eine fehlende Buchung nach', async () => {
+    // Jedes Glied einer Liste kostet: das unterste ein Paket, jedes weitere
+    // ein Redesign. Wo nichts gebucht ist, fehlt es in der Rechnung.
     const spieler = await spielerMitPaketen('Ohne Buchung', 1);
     const eins = await characters.createCharacter(spieler, { name: 'Mit Buchung' }, ctx);
 
@@ -735,7 +736,19 @@ describe('Die Stelle in der Kette bestimmt, was ein Eintrag kostet', () => {
       ctx,
     );
 
-    assert.deepEqual(await credits.getBalances(spieler), vorher, 'nichts gebucht, nichts geändert');
+    const danach = await credits.getBalances(spieler);
+    assert.equal(danach.CC, vorher.CC, 'der Platz bleibt, wie er war');
+    assert.equal(
+      danach.REDESIGN_PAID,
+      vorher.REDESIGN_PAID - 1,
+      'das zweite Glied kostet ein Redesign',
+    );
+
+    // Und allein gestellt kostet er wieder einen eigenen Platz.
+    await characters.detachFromChain(ohne.id, ctx);
+    const allein = await credits.getBalances(spieler);
+    assert.equal(allein.CC, vorher.CC - 1, 'eine eigene Liste, ein eigenes Paket');
+    assert.equal(allein.REDESIGN_PAID, vorher.REDESIGN_PAID, 'das Redesign ist wieder frei');
   });
 });
 
@@ -1345,11 +1358,13 @@ describe('Die Umstellung auf Listen je Nutzer', () => {
       );
     }
 
-    assert.equal(
-      await characters.richteBuchungenAus(),
-      0,
-      'und die Buchungen passen danach zu den Listen',
-    );
+    for (const wer of [wirt, gast]) {
+      assert.equal(
+        (await characters.rebuildFunding(wer, ctx)).umgebucht,
+        0,
+        'und die Buchungen passen danach zu den Listen',
+      );
+    }
 
     // Der Gast zahlt für seinen Einstieg ein Redesign, nicht einen Platz.
     assert.equal(

server/test/nacherfassung.test.ts

@@ -107,16 +107,20 @@ describe('Nachträglich aufräumen, was längst passiert ist', () => {
     assert.deepEqual(await mitglieder(alpha), ['Spieler A', 'Spieler B']);
   });
 
-  it('zieht ihn durch die Kette, ohne ein zweites Mal zu buchen', async () => {
+  it('zieht ihn durch die Kette und bucht jedes Glied einzeln', async () => {
+    // Für ihn ist jedes Glied ein eigener Eintrag in seiner Liste: der
+    // unterste kostet den Platz, jedes weitere ein Redesign.
     const vorher = await credits.getBalances(a);
 
     const ergebnis = await characters.addMemberToChain(beta, a, {}, ctx);
     assert.equal(ergebnis.added, 2, 'Beta und Gamma kommen dazu');
 
-    assert.deepEqual(
-      await credits.getBalances(a),
-      vorher,
-      'sein Platz wurde einmal bezahlt und ist derselbe geblieben',
+    const danach = await credits.getBalances(a);
+    assert.equal(danach.CC, vorher.CC, 'sein Platz wurde einmal bezahlt');
+    assert.equal(
+      danach.REDESIGN_PAID,
+      vorher.REDESIGN_PAID - 2,
+      'für Beta und Gamma je ein Redesign',
     );
     assert.deepEqual(await mitglieder(gamma), ['Spieler A', 'Spieler B']);
   });