Fix wizard store method, and add test coverage

tighten · Jun 4, 2024 · f19089f · f19089f
1 parent 15446f2
commit f19089f
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 4 deletions.
diff --git a/app/Http/Controllers/Auth/WizardController.php b/app/Http/Controllers/Auth/WizardController.php
@@ -21,12 +21,10 @@ public function index(): View
 
     public function store(): RedirectResponse
     {
-        $valid = request()->validate(request(), [
+        $valid = request()->validate([
             'os' => ['required', 'string', Rule::in(OperatingSystem::ALL)],
             'track' => ['required', 'int', 'exists:tracks,id'],
             'locale' => ['required', 'string', Rule::in((new Locale)->slugs())],
-        ], [], [
-            'os' => 'OS',
         ]);
 
         auth()->user()->update(['track_id' => $valid['track']]);

diff --git a/database/seeders/ContentSeeder.php b/database/seeders/ContentSeeder.php
@@ -13,7 +13,7 @@ class ContentSeeder extends Seeder
     public function run(): void
     {
         User::factory()->create([
-            'email' => 'matt@tighten.co',
+            'email' => 'testuser@tighten.co',
             'password' => bcrypt('password'),
             'role' => 'admin',
         ]);

diff --git a/tests/Feature/WizardTest.php b/tests/Feature/WizardTest.php
@@ -2,7 +2,10 @@
 
 namespace Tests\Feature;
 
+use App\Facades\Preferences;
+use App\Models\Track;
 use App\Models\User;
+use App\OperatingSystem;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;
 
@@ -19,4 +22,23 @@ public function it_loads(): void
 
         $response->assertStatus(200);
     }
+
+    /** @test */
+    public function it_can_be_submitted(): void
+    {
+        $this->be($user = User::factory()->create());
+        $response = $this->post(route('wizard.store', ['locale' => 'en']),
+            [
+            'os' => OperatingSystem::MACOS,
+            'track' => $track_id = Track::factory()->create()->id,
+            'locale' => 'en',
+            ]
+        );
+
+        $response->assertStatus(302);
+        $user->refresh();
+        $this->assertEquals($track_id, $user->track_id);
+        $this->assertEquals(OperatingSystem::MACOS, Preferences::get('operating-system'));
+        $this->assertEquals('en', Preferences::get('locale'));
+    }
 }