refactoring

app/Actions/SourceControl/ConnectSourceControl.php

@@ -3,33 +3,48 @@
 namespace App\Actions\SourceControl;
 
 use App\Models\SourceControl;
+use Illuminate\Support\Facades\Validator;
+use Illuminate\Validation\Rule;
 use Illuminate\Validation\ValidationException;
 
 class ConnectSourceControl
 {
-    public function connect(string $provider, array $input): void
+    public function connect(array $input): void
     {
-        $sourceControl = SourceControl::query()
-            ->where('provider', $provider)
-            ->first();
-        if (! $sourceControl) {
-            $sourceControl = new SourceControl([
-                'provider' => $provider,
-            ]);
-        }
+        $this->validate($input);
+        $sourceControl = new SourceControl([
+            'provider' => $input['provider'],
+            'profile' => $input['name'],
+            'access_token' => $input['token']
+        ]);
 
-        if (! $input['token']) {
-            $sourceControl->delete();
-
-            return;
-        }
-
-        $sourceControl->access_token = $input['token'];
         if (! $sourceControl->provider()->connect()) {
             throw ValidationException::withMessages([
-                'token' => __('Cannot connect to :provider or invalid token!', ['provider' => $provider]),
+                'token' => __('Cannot connect to :provider or invalid token!', ['provider' => $sourceControl->provider]
+                ),
             ]);
         }
+        
         $sourceControl->save();
     }
+
+    /**
+     * @throws ValidationException
+     */
+    private function validate(array $input): void
+    {
+        $rules = [
+            'provider' => [
+                'required',
+                Rule::in(\App\Enums\SourceControl::getValues())
+            ],
+            'name' => [
+                'required',
+            ],
+            'token' => [
+                'required'
+            ]
+        ];
+        Validator::make($input, $rules)->validate();
+    }
 }