diff options
author | Eric Eastwood <contact@ericeastwood.com> | 2017-11-29 16:04:41 -0600 |
---|---|---|
committer | Eric Eastwood <contact@ericeastwood.com> | 2017-11-29 16:04:41 -0600 |
commit | ed3fcdd09bed143b0397d8dd9aaf7b9350597920 (patch) | |
tree | c112cd6f28b76d27ddce2d121be24375200b98d8 /spec | |
parent | fbe8dfb3008a0502fbc6234a169851a70707b7b6 (diff) | |
download | gitlab-ce-40671-fix-empty-blank-flash-messages.tar.gz |
Remove blank flash messages caused by nil40671-fix-empty-blank-flash-messages
Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/40671
See
https://gitlab.com/gitlab-org/gitlab-ce/blob/f7254a4060b30e3134c6cf932eaba0fc8e249e9a/app/controllers/sessions_controller.rb#L42
for an example of where we set `flash[:notice] = nil`
Diffstat (limited to 'spec')
-rw-r--r-- | spec/features/logout_spec.rb | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/spec/features/logout_spec.rb b/spec/features/logout_spec.rb new file mode 100644 index 00000000000..635729efa53 --- /dev/null +++ b/spec/features/logout_spec.rb @@ -0,0 +1,22 @@ +require 'spec_helper' + +describe 'Logout/Sign out', :js do + let(:user) { create(:user) } + + before do + sign_in(user) + visit root_path + end + + it 'sign out redirects to sign in page' do + gitlab_sign_out + + expect(current_path).to eq new_user_session_path + end + + it 'sign out does not show signed out flash notice' do + gitlab_sign_out + + expect(page).not_to have_selector('.flash-notice') + end +end |