Merge pull request #7 from kerwin-wildsea/fix/notification-translation

Added logic to LanguageController for showing "Language changed" notification in the newly set language

src/Http/Controllers/LanguageController.php

@@ -41,7 +41,7 @@ public function index(Request $request): RedirectResponse
         }
 
         Notification::make()
-            ->title(trans('filament-language-switcher::translation.notification'))
+            ->title(trans('filament-language-switcher::translation.notification', locale: $request->get('lang')))
             ->icon('heroicon-o-check-circle')
             ->iconColor('success')
             ->send();

tests/src/LanguageSwitcherTest.php

@@ -1,9 +1,9 @@
 <?php
 
-use Illuminate\Database\Eloquent\Relations\Relation;
+
+use Filament\Notifications\Notification;
 use Illuminate\Support\Str;
 use TomatoPHP\FilamentLanguageSwitcher\Tests\Models\User;
-
 use function Pest\Laravel\actingAs;
 use function Pest\Laravel\get;
 
@@ -62,20 +62,21 @@
     expect($currentLang)->toBeTrue();
 });
 
-it('can switch language while using morphMap', function () {
-    // Register Relation MorphMap
-    Relation::morphMap(['user' => get_class(auth()->user())]);
 
+it('sends a notification in the newly selected language', function (string $locale) {
     $response = get(route('languages.switcher', [
-        'model' => get_class(auth()->user()),
+        'model' => get_class(auth('web')->user()),
         'model_id' => auth()->user()->id,
-        'lang' => 'ar',
+        'lang' => $locale,
     ]));
 
     // Ensure the response status is OK (200)
     $response->assertStatus(302);
 
-    $currentLang = auth('web')->user()->lang == 'ar';
-
-    expect($currentLang)->toBeTrue();
-});
+    Notification::assertNotified(
+        Notification::make()
+            ->title(trans('filament-language-switcher::translation.notification', locale: $locale))
+            ->icon('heroicon-o-check-circle')
+            ->iconColor('success')
+    );
+})->with(['locale' => ['en', 'nl', 'ar', 'de']]);