refactor: update settings navigation and remove unused components

routes/navigation.php

@@ -14,7 +14,7 @@
 */
 
 // User menu - Settings
-Navigation::add('Settings', fn () => route('settings.index'), function (Section $section) {
+Navigation::add('Settings', fn () => route('settings.profile'), function (Section $section) {
     $section->attributes([
         'group' => 'user',
         'slug' => 'settings',
@@ -23,36 +23,12 @@
     ]);
 });
 
-// Settings sidebar - General
-Navigation::add('General', fn () => route('settings.index'), function (Section $section) {
-    $section->attributes([
-        'group' => 'settings',
-        'slug' => 'settings',
-        'icon' => 'settings',
-        'order' => 10,
-    ]);
-});
-
 // Settings sidebar - Profile
 Navigation::add('Profile', fn () => route('settings.profile'), function (Section $section) {
     $section->attributes([
         'group' => 'settings',
         'slug' => 'profile',
         'icon' => 'profile',
-        'order' => 20,
-    ]);
-});
-
-// Secondary navigation - Settings
-Navigation::add('Settings', fn () => route('settings.index'), function (Section $section) {
-    $section->attributes([
-        'group' => 'secondary',
-        'slug' => 'settings',
-        'icon' => 'settings',
         'order' => 10,
-        'badge' => [
-            'content' => '1',
-            'variant' => 'destructive',
-        ],
     ]);
 });

routes/web.php

@@ -3,7 +3,6 @@
 use Illuminate\Support\Facades\Route;
 use Modules\Settings\Http\Controllers\PasswordController;
 use Modules\Settings\Http\Controllers\ProfileController;
-use Modules\Settings\Http\Controllers\SettingsController;
 
 Route::middleware('web')->group(function (): void {
     Route::group(['middleware' => [
@@ -12,7 +11,7 @@
         'role:admin|user',
     ]], function (): void {
         Route::prefix('settings')->group(function (): void {
-            Route::get('/', [SettingsController::class, 'index'])
+            Route::redirect('/', '/settings/profile')
                 ->name('settings.index');
 
             Route::get('profile', [ProfileController::class, 'show'])

tests/Feature/SettingsNavigationTest.php

@@ -0,0 +1,42 @@
+<?php
+
+namespace Modules\Settings\Tests\Feature;
+
+use App\Filament\Admin\Pages\GeneralSettings;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Illuminate\Support\Facades\Route;
+use Tests\TestCase;
+
+class SettingsNavigationTest extends TestCase
+{
+    use RefreshDatabase;
+
+    public function test_settings_index_redirects_authenticated_user_to_profile(): void
+    {
+        $user = $this->createUser();
+
+        $response = $this->actingAs($user)->get(route('settings.index'));
+
+        $response->assertRedirectToRoute('settings.profile');
+    }
+
+    public function test_settings_navigation_only_contains_profile(): void
+    {
+        $user = $this->createUser();
+
+        $response = $this->actingAs($user)->get(route('settings.profile'));
+
+        $this->assertSame(
+            ['Profile'],
+            array_column($response->inertiaProps('navigation.settings'), 'title'),
+        );
+    }
+
+    public function test_admin_general_settings_route_uses_core_settings_page(): void
+    {
+        $route = Route::getRoutes()->getByName('filament.admin.pages.general-settings');
+
+        $this->assertNotNull($route);
+        $this->assertSame(GeneralSettings::class, $route->getActionName());
+    }
+}