[Bug] Added server id to start speedtest action (alexjustesen#1805)

* added server id to start speedtest action

* replaced deprecated modal button label

* code quality

Author: alexjustesen

app/Actions/Ookla/StartSpeedtest.php

@@ -13,15 +13,17 @@ class StartSpeedtest
 {
     use AsAction;
 
-    public function handle(bool $scheduled = false): void
+    public function handle(bool $scheduled = false, ?int $serverId = null): void
     {
         $result = Result::create([
             'service' => ResultService::Ookla,
             'status' => ResultStatus::Started,
             'scheduled' => $scheduled,
         ]);
 
-        $serverId = SelectSpeedtestServer::run();
+        if (blank($serverId)) {
+            $serverId = SelectSpeedtestServer::run();
+        }
 
         if (! blank($serverId)) {
             $result->update([

app/Filament/Pages/Dashboard.php

@@ -50,27 +50,31 @@ protected function getHeaderActions(): array
                 ->color('gray')
                 ->hidden(fn (): bool => ! config('speedtest.public_dashboard'))
                 ->url(shouldOpenInNewTab: true, url: '/'),
-            Action::make('ookla_speedtest')
+
+            Action::make('speedtest')
                 ->form([
                     Forms\Components\Select::make('server_id')
                         ->label('Select Server')
-                        ->helperText('Leave blank to run the speedtest without specifying a server.')
-                        ->options(fn (callable $get) => app(GetOoklaSpeedtestServers::class)->handle($get('server_search')))
+                        ->helperText('Leave empty to run the speedtest without specifying a server.')
+                        ->options(fn (): array => GetOoklaSpeedtestServers::run())
                         ->searchable(),
                 ])
                 ->action(function (array $data) {
                     $serverId = $data['server_id'] ?? null;
 
-                    StartSpeedtest::run(serverId: $serverId);
+                    StartSpeedtest::run(
+                        scheduled: false,
+                        serverId: $serverId,
+                    );
 
                     Notification::make()
                         ->title('Speedtest started')
                         ->success()
                         ->send();
                 })
                 ->modalHeading('Run Speedtest')
-                ->modalButton('Run Speedtest')
                 ->modalWidth('lg')
+                ->modalSubmitActionLabel('Start')
                 ->button()
                 ->color('primary')
                 ->label('Run Speedtest')