From f82555469a9f4c72f159ca04b6aeb63b8d22953e Mon Sep 17 00:00:00 2001 From: boban Date: Tue, 23 Jun 2026 02:29:59 +0200 Subject: [PATCH] style: pint --test fixes (CI pint step now reachable after the test fix) Co-Authored-By: Claude Opus 4.8 --- app/Support/Confirm/InvalidConfirmToken.php | 4 +--- routes/web.php | 3 ++- tests/Feature/RecoveryCodesModalTest.php | 3 ++- tests/Feature/ReleasePageTest.php | 10 +++++----- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/app/Support/Confirm/InvalidConfirmToken.php b/app/Support/Confirm/InvalidConfirmToken.php index af88f90..c394f2f 100644 --- a/app/Support/Confirm/InvalidConfirmToken.php +++ b/app/Support/Confirm/InvalidConfirmToken.php @@ -8,6 +8,4 @@ use RuntimeException; * Thrown when a confirm token fails to decode/validate or has already been used. * Callers treat it as "reject the confirm" — never leaked to the operator verbatim. */ -class InvalidConfirmToken extends RuntimeException -{ -} +class InvalidConfirmToken extends RuntimeException {} diff --git a/routes/web.php b/routes/web.php index 6db772f..2964507 100644 --- a/routes/web.php +++ b/routes/web.php @@ -15,6 +15,7 @@ use App\Livewire\System; use App\Livewire\Versions; use App\Livewire\Wireguard; use App\Services\DeploymentService; +use App\Services\WgStatus; use Illuminate\Http\Request; use Illuminate\Support\Facades\Auth as AuthFacade; use Illuminate\Support\Facades\Route; @@ -143,7 +144,7 @@ Route::middleware('auth')->group(function () { Route::get('/release', Release\Index::class)->name('release'); } - Route::get('/wg-status.json', fn (\App\Services\WgStatus $wg) => response()->json($wg->read()))->name('wg.status'); + Route::get('/wg-status.json', fn (WgStatus $wg) => response()->json($wg->read()))->name('wg.status'); // Plain Blade view — deliberately NOT a Livewire component so it survives the // container teardown that follows an update request (R1/R2 exception, by design). diff --git a/tests/Feature/RecoveryCodesModalTest.php b/tests/Feature/RecoveryCodesModalTest.php index aafc07e..931ac39 100644 --- a/tests/Feature/RecoveryCodesModalTest.php +++ b/tests/Feature/RecoveryCodesModalTest.php @@ -6,6 +6,7 @@ use App\Livewire\Modals\RecoveryCodes; use App\Models\User; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Support\Facades\Route; +use Livewire\Features\SupportTesting\Testable; use Livewire\Livewire; use Tests\TestCase; @@ -185,7 +186,7 @@ class RecoveryCodesModalTest extends TestCase * serialized snapshot — the signed wire:snapshot blob a client could capture and replay. * * @param array $codes - * @param \Livewire\Features\SupportTesting\Testable $component + * @param Testable $component */ private function assertSnapshotHidesCodes(array $codes, $component): void { diff --git a/tests/Feature/ReleasePageTest.php b/tests/Feature/ReleasePageTest.php index e49fb09..e015098 100644 --- a/tests/Feature/ReleasePageTest.php +++ b/tests/Feature/ReleasePageTest.php @@ -4,8 +4,8 @@ namespace Tests\Feature; use App\Livewire\Release\Index; use App\Models\AuditEvent; +use App\Models\Setting; use App\Models\User; -use App\Services\ReleaseBridge; use Illuminate\Foundation\Testing\RefreshDatabase; use Illuminate\Support\Facades\RateLimiter; use Livewire\Livewire; @@ -116,19 +116,19 @@ class ReleasePageTest extends TestCase public function test_save_test_server_persists_a_valid_url(): void { Livewire::test(Index::class)->call('saveTestServer', 'http://staging.local'); - $this->assertSame('http://staging.local', \App\Models\Setting::get('pipeline_test_server')); + $this->assertSame('http://staging.local', Setting::get('pipeline_test_server')); } public function test_save_test_server_rejects_a_bad_url(): void { Livewire::test(Index::class)->call('saveTestServer', 'not-a-url'); - $this->assertNull(\App\Models\Setting::get('pipeline_test_server')); + $this->assertNull(Setting::get('pipeline_test_server')); } public function test_save_test_server_empty_clears_it(): void { - \App\Models\Setting::put('pipeline_test_server', 'http://staging.local'); + Setting::put('pipeline_test_server', 'http://staging.local'); Livewire::test(Index::class)->call('saveTestServer', ''); - $this->assertSame('', (string) \App\Models\Setting::get('pipeline_test_server', '')); + $this->assertSame('', (string) Setting::get('pipeline_test_server', '')); } }