Commit 3c36d9dc authored by jnoortheen's avatar jnoortheen

fix: removed signed_out notification

test: replaced signed_out message check with check for sign_in button

fixes #25294
parent 87f3a66c
...@@ -37,6 +37,12 @@ class SessionsController < Devise::SessionsController ...@@ -37,6 +37,12 @@ class SessionsController < Devise::SessionsController
end end
end end
def destroy
super
# hide the signed_out notice
flash[:notice] = nil
end
private private
# Handle an "initial setup" state, where there's only one user, it's an admin, # Handle an "initial setup" state, where there's only one user, it's an admin,
......
---
title: 'fix: removed signed_out notification'
merge_request: 7958
author: jnoortheen
...@@ -75,7 +75,8 @@ module LoginHelpers ...@@ -75,7 +75,8 @@ module LoginHelpers
def logout def logout
find(".header-user-dropdown-toggle").click find(".header-user-dropdown-toggle").click
click_link "Sign out" click_link "Sign out"
expect(page).to have_content('Signed out successfully') # check the sign_in button
expect(page).to have_button('Sign in')
end end
# Logout without JavaScript driver # Logout without JavaScript driver
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment