@if ($cancellationScheduled)
- {{ __('settings.cancel_scheduled_body', ['date' => $instance?->service_ends_at?->local()->isoFormat('LL')]) }}
+ {{-- An die Antwort des Kunden gebunden, statt jedem
+ denselben Export zu versprechen. Der Kunde, der im
+ Dialog gerade „Nein" gewaehlt hat, wird direkt
+ hierher umgeleitet — und las als Erstes, dass er
+ ihn doch bekommt. Drei Zustaende, drei Saetze; bei
+ `null` (nie gefragt) fragt der Satz, statt zu
+ versprechen. --}}
+ {{ __(match ($instance?->export_wish) {
+ true => 'settings.cancel_scheduled_body_yes',
+ false => 'settings.cancel_scheduled_body_no',
+ default => 'settings.cancel_scheduled_body_open',
+ }, ['date' => $instance?->service_ends_at?->local()->isoFormat('LL')]) }}
@elseif ($hasActivePackage)
{{ __('settings.package_active', ['plan' => $instance ? __('billing.plan.'.$instance->plan) : '—']) }}
@else
diff --git a/resources/views/mail/service-ending-soon.blade.php b/resources/views/mail/service-ending-soon.blade.php
index c019e94..47b9f33 100644
--- a/resources/views/mail/service-ending-soon.blade.php
+++ b/resources/views/mail/service-ending-soon.blade.php
@@ -23,7 +23,9 @@
{{-- Nur, wer beim Kündigen "ja" gesagt hat. Ein "nein" bedeutet ausdrücklich
"bereitet nichts vor" (ConfirmCancelPackage) — ihm hier trotzdem einen
- Export anzukündigen wäre ein Versprechen, das niemand eingelöst hätte. --}}
+ Export anzukündigen wäre ein Versprechen, das niemand eingelöst hätte.
+ Und `null` heißt "nie gefragt": auch dafür ist der Satz falsch, deshalb
+ der strikte Vergleich auf true. --}}
@if ($exportWish === true)
diff --git a/tests/Feature/Cancellation/DashboardCountdownTest.php b/tests/Feature/Cancellation/DashboardCountdownTest.php
index 5502476..70c9051 100644
--- a/tests/Feature/Cancellation/DashboardCountdownTest.php
+++ b/tests/Feature/Cancellation/DashboardCountdownTest.php
@@ -87,13 +87,48 @@ it('zeigt den Streifen nicht, solange nicht gekuendigt wurde', function () {
it('zeigt ihn auch nach dem Laufzeitende nicht mehr', function () {
// Danach ist der Zugang zu, und ein Streifen, der eine Restzeit nennt,
- // wäre schlicht falsch. Was dann gilt, sagt die Seite an anderer Stelle.
+ // wäre schlicht falsch. Was dann gilt, sagt der Kasten aus der Prüfung
+ // darunter — bis zur Fix-Welle behauptete dieser Kommentar das, und die
+ // Seite sagte das Gegenteil.
[$user] = gekuendigteInstanz(endetIn: now()->subDay(), status: 'ended');
Livewire::actingAs($user)->test(Dashboard::class)
->assertDontSee(__('dashboard.ending.title'));
});
+it('sagt nach dem Laufzeitende, dass die Laufzeit beendet ist', function () {
+ // K2, zweite Hälfte: Dashboard::render() holt eine `ended`-Instanz nicht
+ // mehr, und der Kunde fiel damit in denselben Zweig wie jemand, der noch
+ // nie etwas bestellt hat — „Ihre Cloud wird eingerichtet." samt „Paket
+ // buchen", am Tag, an dem ihm EndInstanceService die Adresse eingezogen
+ // hat. Das ist kein Kunde ohne Paket, sondern einer, dessen Paket vorbei
+ // ist, und die Seite muss das sagen, samt dem Datum.
+ [$user, $instance] = gekuendigteInstanz(endetIn: now()->subDay(), status: 'ended');
+
+ Livewire::actingAs($user)->test(Dashboard::class)
+ ->assertSee(__('dashboard.ended.label'))
+ ->assertSee($instance->service_ends_at->local()->isoFormat('LL'))
+ ->assertDontSee(__('dashboard.title_pending'))
+ ->assertDontSee(__('dashboard.no_instance_label'));
+});
+
+it('zeigt einem Kunden ohne jede Instanz weiterhin den leeren Zustand', function () {
+ // Die Gegenprobe zur Prüfung darüber: der beendete Fall darf den ehrlichen
+ // Leerzustand nicht verschlucken. Wer nie etwas bestellt hat, bekommt
+ // weiterhin „Noch keine Instanz" — und keine Rede von einer Laufzeit, die
+ // es nie gab.
+ $user = User::factory()->create(['email' => 'ohne@example.test', 'email_verified_at' => now()]);
+ Customer::factory()->create([
+ 'email' => 'ohne@example.test',
+ 'user_id' => $user->id,
+ 'status' => 'active',
+ ]);
+
+ Livewire::actingAs($user)->test(Dashboard::class)
+ ->assertSee(__('dashboard.no_instance_label'))
+ ->assertDontSee(__('dashboard.ended.label'));
+});
+
it('nennt keine Restzeit, die schon abgelaufen ist', function () {
// Der Fall zwischen den Stühlen: das Laufzeitende ist vorbei, aber der
// stündliche Lauf hat die Instanz noch nicht auf `ended` gestellt. Eine
@@ -130,12 +165,58 @@ it('zaehlt am letzten Tag in Stunden, nicht mehr in Tagen', function () {
->assertSee(trans_choice('dashboard.ending.remaining_hours', 5));
});
-it('traegt die Antwort zum Export und laesst sie aendern', function () {
+/*
+ * Die drei Zustände am Streifen, je einer pro Prüfung.
+ *
+ * Vorher stand hier EINE Prüfung, und sie konnte nicht fehlschlagen:
+ * `x-ui.switch` rendert beide Wörter und überlässt dem CSS, welches sichtbar
+ * ist — `assertSee('Kein Export gewünscht')` war deshalb bei `true`, bei
+ * `false` UND bei `null` grün. Genau deshalb kam K1 durch: `(bool) null` ist
+ * `false`, der Schalter behauptete einem nie gefragten Menschen seine eigene
+ * Antwort, und nichts hielt die drei Zustände auseinander.
+ *
+ * Die Auswahlfelder tragen ihren Zustand dagegen im Markup (`checked`), also
+ * prüft das hier wirklich das, was der Kunde sieht.
+ */
+it('zeigt am Streifen ein Ja als Ja', function () {
+ [$user, $instance] = gekuendigteInstanz(endetIn: now()->addDays(12));
+ $instance->update(['export_wish' => true]);
+
+ Livewire::actingAs($user)->test(Dashboard::class)
+ ->assertSeeHtml('id="export-wish-yes" checked')
+ ->assertDontSeeHtml('id="export-wish-no" checked');
+});
+
+it('zeigt am Streifen ein Nein als Nein', function () {
+ [$user, $instance] = gekuendigteInstanz(endetIn: now()->addDays(12));
+ $instance->update(['export_wish' => false]);
+
+ Livewire::actingAs($user)->test(Dashboard::class)
+ ->assertSeeHtml('id="export-wish-no" checked')
+ ->assertDontSeeHtml('id="export-wish-yes" checked');
+});
+
+it('behauptet keine Antwort, wenn niemand gefragt wurde', function () {
+ // Der Schaden, gegen den der Kopfkommentar der Wanderung geschrieben
+ // wurde: eine vor dieser Ausrollung gekündigte Instanz trägt `null`, und
+ // der Streifen sagte darunter „Kein Export gewünscht" — eine Aussage über
+ // einen Wunsch, den dieser Mensch nie geäußert hat, dargestellt als seine
+ // eigene Antwort. Keine Auswahl ist markiert, und der Satz daneben fragt.
+ [$user, $instance] = gekuendigteInstanz(endetIn: now()->addDays(12));
+
+ expect($instance->export_wish)->toBeNull();
+
+ Livewire::actingAs($user)->test(Dashboard::class)
+ ->assertDontSeeHtml('id="export-wish-yes" checked')
+ ->assertDontSeeHtml('id="export-wish-no" checked')
+ ->assertSee(__('dashboard.ending.export_open'));
+});
+
+it('laesst die Antwort am Streifen aendern', function () {
[$user, $instance] = gekuendigteInstanz(endetIn: now()->addDays(12));
$instance->update(['export_wish' => false]);
Livewire::actingAs($user)->test(Dashboard::class)
- ->assertSee(__('dashboard.ending.export_no'))
->call('setExportWish', true);
expect($instance->fresh()->export_wish)->toBeTrue();
diff --git a/tests/Feature/Cancellation/ExportAnswerHasReadersTest.php b/tests/Feature/Cancellation/ExportAnswerHasReadersTest.php
new file mode 100644
index 0000000..718ed31
--- /dev/null
+++ b/tests/Feature/Cancellation/ExportAnswerHasReadersTest.php
@@ -0,0 +1,137 @@
+create(['email' => 'vertrag@example.test', 'email_verified_at' => now()]);
+
+ $customer = Customer::factory()->create([
+ 'email' => 'vertrag@example.test',
+ 'user_id' => $user->id,
+ 'status' => 'active',
+ ]);
+
+ $order = Order::factory()->create(['customer_id' => $customer->id, 'plan' => 'team']);
+
+ $instance = Instance::factory()->create([
+ 'customer_id' => $customer->id,
+ 'order_id' => $order->id,
+ 'plan' => 'team',
+ 'status' => 'cancellation_scheduled',
+ 'subdomain' => 'vertrag',
+ 'cancel_requested_at' => now()->subDay(),
+ 'service_ends_at' => now()->addDays(20),
+ 'export_wish' => $exportWunsch,
+ ]);
+
+ return [$user, $instance->refresh()];
+}
+
+it('verspricht keinen Export, wenn der Kunde gerade Nein gesagt hat', function () {
+ // K2: zwei richtige Bauteile, ein Bildschirm dazwischen, der beiden
+ // widersprach. Der Dialog schreibt korrekt `false` und leitet auf genau
+ // diese Seite um — und das Erste, was der Kunde dort las, war, dass er
+ // den Export bekommt.
+ [$user, $instance] = kundeMitVorgemerkterKuendigung(false);
+
+ Livewire::actingAs($user)->test(Settings::class, ['tab' => 'contract'])
+ ->assertSee(__('settings.cancel_scheduled_body_no', [
+ 'date' => $instance->service_ends_at->local()->isoFormat('LL'),
+ ]))
+ ->assertDontSee(__('settings.cancel_scheduled_body_yes', [
+ 'date' => $instance->service_ends_at->local()->isoFormat('LL'),
+ ]));
+});
+
+it('nennt den Export nur dem, der ihn wollte', function () {
+ [$user, $instance] = kundeMitVorgemerkterKuendigung(true);
+
+ Livewire::actingAs($user)->test(Settings::class, ['tab' => 'contract'])
+ ->assertSee(__('settings.cancel_scheduled_body_yes', [
+ 'date' => $instance->service_ends_at->local()->isoFormat('LL'),
+ ]));
+});
+
+it('fragt auf der Vertragsseite nach, wenn niemand gefragt wurde', function () {
+ // `null` heißt: vor dieser Ausrollung gekündigt. Der Satz fragt dann,
+ // statt zu versprechen — und sagt, was passiert, wenn die Frage offen
+ // bleibt.
+ [$user, $instance] = kundeMitVorgemerkterKuendigung(null);
+
+ Livewire::actingAs($user)->test(Settings::class, ['tab' => 'contract'])
+ ->assertSee(__('settings.cancel_scheduled_body_open', [
+ 'date' => $instance->service_ends_at->local()->isoFormat('LL'),
+ ]))
+ ->assertDontSee(__('settings.cancel_scheduled_body_yes', [
+ 'date' => $instance->service_ends_at->local()->isoFormat('LL'),
+ ]));
+});
+
+it('zeigt dem Betreiber, wer einen Export bestellt hat und bis wann', function () {
+ // W1: ohne diesen Abschnitt hatte ein „Ja" in der ganzen Konsole keinen
+ // einzigen Leser — ein grep über app/ und die Konsolenansichten fand
+ // ausserhalb dieses Vorhabens keinen Treffer. Beides muss dastehen: wer,
+ // und wann dessen Laufzeit endet.
+ $customer = Customer::factory()->create(['name' => 'Exportkundin GmbH']);
+ $instance = Instance::factory()->create([
+ 'customer_id' => $customer->id,
+ 'status' => 'cancellation_scheduled',
+ 'subdomain' => 'exportkundin',
+ 'service_ends_at' => now()->addDays(9),
+ 'export_wish' => true,
+ ]);
+
+ Livewire::actingAs(admin(), 'operator')->test(AdminInstances::class)
+ ->assertSee(__('admin.export_wish.title'))
+ ->assertSee('Exportkundin GmbH')
+ ->assertSee($instance->service_ends_at->local()->isoFormat('LL'));
+});
+
+it('fuehrt in der Konsole niemanden auf, der keinen Export wollte', function () {
+ // Ein Nein und ein unbeantwortetes Null sind beide keine Bestellung. Der
+ // Abschnitt erscheint gar nicht erst — ein dauerhaft leerer Kasten wird
+ // nach der zweiten Woche nicht mehr gelesen. Geprüft an der Überschrift
+ // und nicht am Kundennamen: der steht ohnehin in der Instanzliste
+ // darunter, und eine Zusicherung darauf wäre aus dem falschen Grund rot.
+ $customer = Customer::factory()->create(['name' => 'Neinkundin GmbH']);
+ Instance::factory()->create([
+ 'customer_id' => $customer->id,
+ 'status' => 'cancellation_scheduled',
+ 'subdomain' => 'neinkundin',
+ 'service_ends_at' => now()->addDays(9),
+ 'export_wish' => false,
+ ]);
+ Instance::factory()->create([
+ 'status' => 'cancellation_scheduled',
+ 'subdomain' => 'niegefragt',
+ 'service_ends_at' => now()->addDays(9),
+ ]);
+
+ Livewire::actingAs(admin(), 'operator')->test(AdminInstances::class)
+ ->assertDontSee(__('admin.export_wish.title'));
+});
diff --git a/tests/Feature/Cancellation/RemindEndingServicesTest.php b/tests/Feature/Cancellation/RemindEndingServicesTest.php
index 2b13624..52c590f 100644
--- a/tests/Feature/Cancellation/RemindEndingServicesTest.php
+++ b/tests/Feature/Cancellation/RemindEndingServicesTest.php
@@ -120,3 +120,20 @@ it('trägt in der Mail das Datum und den Weg zum Selbst-Herunterladen', function
->and($text)->toContain($instance->service_ends_at->local()->isoFormat('LL'))
->and($text)->toContain('https://'.$instance->address(ProvisioningSettings::dnsZone()));
});
+
+it('behauptet keinen Vorgang, den niemand ausfuehrt', function () {
+ // W1: der Satz stand im Präsens und sagte, wir bereiteten bereits einen
+ // Export vor. Es gibt keinen — er hängt an einem Speicherserver, den es
+ // noch nicht gibt. Was stimmt, ist: der Wunsch ist vermerkt, und es
+ // meldet sich jemand.
+ $instance = Instance::factory()->create([
+ 'status' => 'cancellation_scheduled',
+ 'service_ends_at' => now()->addDays(6),
+ 'export_wish' => true,
+ ]);
+
+ $text = (new ServiceEndingSoonMail($instance))->render();
+
+ expect($text)->toContain('vermerkt')
+ ->and($text)->not->toContain('bereiten wir zusätzlich einen für Sie vor');
+});
|