mirror of
https://github.com/maybe-finance/maybe.git
synced 2025-07-19 13:19:39 +02:00
fix(authentication): Fix issues when users try to logout (#603)
Run set last login information action only when the user exist. Thus preventing running it after the logout method, which was causing an error
This commit is contained in:
parent
d29d465a3c
commit
00a42be5c0
2 changed files with 5 additions and 2 deletions
|
@ -3,7 +3,7 @@ module Authentication
|
|||
|
||||
included do
|
||||
before_action :authenticate_user!
|
||||
after_action :set_last_login_at
|
||||
after_action :set_last_login_at, if: -> { Current.user }
|
||||
end
|
||||
|
||||
class_methods do
|
||||
|
|
|
@ -12,6 +12,9 @@ class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
|
|||
fill_in "Password", with: "password"
|
||||
click_button "Log in"
|
||||
end
|
||||
assert_text "Dashboard"
|
||||
assert_text "Dashboard", wait: 5
|
||||
find('[data-controller="menu"]').click
|
||||
click_button "Logout"
|
||||
assert_text "Sign in to your account"
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue