fix: if order by date is used, the test fails if there are logins with the same time (second).

Author: kenjis

src/Models/LoginModel.php

@@ -65,7 +65,7 @@ public function lastLogin(User $user): ?Login
     {
         return $this->where('success', 1)
             ->where('user_id', $user->id)
-            ->orderBy('date', 'desc')
+            ->orderBy('id', 'desc')
             ->first();
     }
 

tests/Unit/UserTest.php

@@ -76,19 +76,23 @@ public function testLastLogin()
         // No logins found.
         $this->assertNull($this->user->lastLogin());
 
-        $login = fake(
+        $login1 = fake(
             LoginModel::class,
             ['identifier' => $this->user->email, 'user_id' => $this->user->id]
         );
-        fake(
+        $login2 = fake(
+            LoginModel::class,
+            ['identifier' => $this->user->email, 'user_id' => $this->user->id]
+        );
+        $login3 = fake(
             LoginModel::class,
             ['identifier' => $this->user->email, 'user_id' => $this->user->id, 'success' => false]
         );
 
         $last = $this->user->lastLogin();
 
         $this->assertInstanceOf(Login::class, $last); // @phpstan-ignore-line
-        $this->assertSame($login->id, $last->id);
+        $this->assertSame($login2->id, $last->id);
         $this->assertInstanceOf(Time::class, $last->date);
     }