test: fix logout method in AuthTest RedirectTest and SecurityLoggerTest

Author: Miguel Angel

tests/Feature/AuthTest.php

@@ -21,7 +21,7 @@ public function testAuthLoginAndLogout()
         $user = User::factory()->create();
         Auth::login($user);
         $this->assertEquals($user->id, Auth::id());
-        Auth::logout();
+        Auth::logoutCurrentDevice();
         $this->assertNull(Auth::user());
     }
 

tests/Feature/RedirectTest.php

@@ -27,7 +27,7 @@ public function test401RedirectsToLogin()
         $response = $this->get('/requests');
         $response->assertStatus(200);
         $response->assertViewIs('requests.index');
-        Auth::logout();
+        Auth::logoutCurrentDevice();
         $response = $this->get('/requests');
         //302 because we want to make sure they are being redirected
         $response->assertStatus(302);

tests/Feature/SecurityLoggerTest.php

@@ -43,7 +43,11 @@ public function testLogSecurityEvents()
         $this->assertDatabaseHas('security_logs', ['event' => 'login', 'user_id' => $user->id]);
 
         // Attempt to logout
-        Auth::logout();
+        if (in_array(Auth::getDefaultDriver(), config('samlidp.guards'))) {
+            return redirect('saml/logout');
+        } else {
+            Auth::logout();
+        }
         $this->assertDatabaseHas('security_logs', ['event' => 'logout', 'user_id' => $user->id]);
 
         // Disable security logging