Kevin Adametz
c7bd940f28
Konsolidierter Entwicklungsplan aus den zwei Analyse-Dokumenten plus
Umsetzung der Phase-0-Massnahmen (Korrektheit/Security, je mit Tests):
0.1 BasicAuth: timing-sicherer Vergleich (hash_equals, beide Felder ohne
Short-Circuit); Enabled-Check zur Laufzeit via config('auth.basic.*')
statt env() im Bootstrap-Closure (greift jetzt unter config:cache);
Config-Block + .env.example. Fund: auth.basic.* war nie als Config
definiert, Gate war faktisch unkonfigurierbar.
0.2 Legacy-Invoice-"latest"-Abfrage zentralisiert (LegacyInvoice::
scopeLatestFirst + User::latestLegacyInvoice); beide Consumer N+1-frei
umgestellt; unterschiedliches null-Verhalten bewusst erhalten.
0.3 Payment-Option-Eligibility vereinheitlicht (period-aware, strikt):
UserPaymentOption::scopeActiveNow/grandfatheredNow + User::
hasActivePaymentOption; hasActiveBooking/hasUnlimitedPressReleaseQuota
und ApiAccessEligibilityService darauf umgestellt; $hasGrandfathered-
Mislabel behoben.
0.4 Theme->Portal-Vokabular: totes, projektfremdes ThemeMiddleware
geloescht; Mapping in Portal::fromTheme() zentralisiert; fremden
b2in-Default im ThemeServiceProvider auf Portal-Default korrigiert.
0.5 InvoiceNumberGenerator: Erst-Insert race-sicher via insertOrIgnore
(Unique-Index auf circle existiert bereits). CreditWalletService war
bereits abgesichert (Unique-Index + createOrFirst).
Tests: 4 neue Dateien, 5 bestehende auf gueltiges grandfathered_until
angepasst. Volle Suite gruen (793 passed, 1 skipped).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
31 lines
1.2 KiB
PHP
31 lines
1.2 KiB
PHP
<?php
|
|
|
|
use App\Services\Billing\InvoiceNumberGenerator;
|
|
use Illuminate\Support\Facades\DB;
|
|
|
|
test('numbers are sequential per circle and circles are independent', function () {
|
|
$generator = app(InvoiceNumberGenerator::class);
|
|
|
|
expect($generator->nextStripeNumber())->toBe('STR-00001');
|
|
expect($generator->nextStripeNumber())->toBe('STR-00002');
|
|
expect($generator->nextManualNumber())->toBe('MAN-00001');
|
|
expect($generator->nextStripeNumber())->toBe('STR-00003');
|
|
expect($generator->nextManualNumber())->toBe('MAN-00002');
|
|
});
|
|
|
|
test('an unknown circle is rejected', function () {
|
|
expect(fn () => app(InvoiceNumberGenerator::class)->next('XXX'))
|
|
->toThrow(InvalidArgumentException::class);
|
|
});
|
|
|
|
test('an existing sequence row is not reset by the race-safe ensure-insert', function () {
|
|
DB::table('invoice_number_sequences')->insert([
|
|
'circle' => InvoiceNumberGenerator::CIRCLE_MANUAL,
|
|
'next_number' => 5,
|
|
'created_at' => now(),
|
|
'updated_at' => now(),
|
|
]);
|
|
|
|
expect(app(InvoiceNumberGenerator::class)->nextManualNumber())->toBe('MAN-00005');
|
|
expect(DB::table('invoice_number_sequences')->where('circle', 'MAN')->count())->toBe(1);
|
|
});
|