feat: remember me for login form (#154)

ArkEcosystemArchive · Jul 7, 2021 · 805bf5f · 805bf5f
1 parent 0d939a3
commit 805bf5f
Show file tree

Hide file tree

Showing 3 changed files with 40 additions and 4 deletions.
diff --git a/resources/lang/en/auth.php b/resources/lang/en/auth.php
@@ -15,6 +15,7 @@
     'sign-in' => [
         'forgot_password'  => 'Forgot password?',
         'register_now'     => 'Not a member? <a href=":route" class="font-semibold underline link">Sign up</a>',
+        'remember_me'      => 'Remember Me',
     ],
 
     'register-form' => [

diff --git a/resources/views/auth/login.blade.php b/resources/views/auth/login.blade.php
@@ -73,14 +73,20 @@ class="w-full"
                     />
                 </div>
 
-                <div class="flex flex-col-reverse items-center justify-between space-y-4 md:space-y-0 md:flex-row">
+                <x-ark-checkbox name="remember" :errors="$errors">
+                    @slot('label')
+                        @lang('fortify::auth.sign-in.remember_me')
+                    @endslot
+                </x-ark-checkbox>
+
+                <div class="flex flex-col-reverse items-center justify-between space-y-4 sm:space-y-0 sm:flex-row">
                     @if(Route::has('password.request'))
-                        <div class="flex-1 mt-8 md:mt-0">
-                            <a href="{{ route('password.request') }}" class="link">@lang('fortify::auth.sign-in.forgot_password')</a>
+                        <div class="flex-1 mt-8 sm:mt-0">
+                            <a href="{{ route('password.request') }}" class="link font-semibold">@lang('fortify::auth.sign-in.forgot_password')</a>
                         </div>
                     @endif
 
-                    <button type="submit" class="w-full button-secondary md:w-auto">
+                    <button type="submit" class="w-full button-secondary sm:w-auto">
                         @lang('fortify::actions.sign_in')
                     </button>
                 </div>

diff --git a/tests/Actions/AuthenticateUserTest.php b/tests/Actions/AuthenticateUserTest.php
@@ -24,6 +24,7 @@
 
     $this->assertNotNull($loggedUser);
     $this->assertTrue($user->is($loggedUser));
+    $this->assertFalse($request->filled('remember'));
 });
 
 it('login the user by the email when alt username is set', function () {
@@ -44,6 +45,7 @@
 
     $this->assertNotNull($loggedUser);
     $this->assertTrue($user->is($loggedUser));
+    $this->assertFalse($request->filled('remember'));
 });
 
 it('login the user by the alt username (username)', function () {
@@ -64,6 +66,7 @@
 
     $this->assertNotNull($loggedUser);
     $this->assertTrue($user->is($loggedUser));
+    $this->assertFalse($request->filled('remember'));
 });
 
 it('doesnt login the user by the alt username if not set (username)', function () {
@@ -83,6 +86,7 @@
     $loggedUser = $authenticator->handle();
 
     $this->assertNull($loggedUser);
+    $this->assertFalse($request->filled('remember'));
 });
 
 it('doesnt login the user if password is incorrect', function () {
@@ -102,3 +106,28 @@
 
     $this->assertNull($loggedUser);
 });
+
+it('should handle the remember me checkbox', function () {
+    Config::set('fortify.models.user', \ARKEcosystem\Fortify\Models\User::class);
+    Config::set('fortify.username_alt', 'username');
+
+    $user = User::factory()->withUsername()->create();
+
+    $request = new Request();
+
+    $this->assertFalse($request->filled('remember'));
+
+    $request->replace([
+        'email'    => $user->username,
+        'password' => 'password',
+        'remember' => true,
+    ]);
+
+    $authenticator = new AuthenticateUser($request);
+    $loggedUser = $authenticator->handle();
+
+    $this->assertNotNull($loggedUser);
+    $this->assertTrue($user->is($loggedUser));
+
+    $this->assertTrue($request->filled('remember'));
+});