diff options
author | Matija Čupić <matteeyah@gmail.com> | 2018-01-30 00:05:48 +0100 |
---|---|---|
committer | Matija Čupić <matteeyah@gmail.com> | 2018-01-30 00:05:48 +0100 |
commit | 977996d2b396c1892a0bcdb386d63e4f78675239 (patch) | |
tree | ceaa55c046134fcdf26838d5b3a59563c1a09f65 | |
parent | 8be4f3ecf480b64284bd0d2ed31f59f332df1bae (diff) | |
download | gitlab-ce-977996d2b396c1892a0bcdb386d63e4f78675239.tar.gz |
Add check for guest user
-rw-r--r-- | app/helpers/callouts_helper.rb | 2 | ||||
-rw-r--r-- | spec/helpers/callouts_helper_spec.rb | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/app/helpers/callouts_helper.rb b/app/helpers/callouts_helper.rb index e435879ee43..a27533f30c6 100644 --- a/app/helpers/callouts_helper.rb +++ b/app/helpers/callouts_helper.rb @@ -1,6 +1,6 @@ module CalloutsHelper def show_gke_cluster_integration_callout?(kube_feature_name, project) - !user_dismissed?(kube_feature_name) && project.team.master?(current_user) + current_user && !user_dismissed?(kube_feature_name) && project.team.master?(current_user) end private diff --git a/spec/helpers/callouts_helper_spec.rb b/spec/helpers/callouts_helper_spec.rb index feabbf33ef6..f160aafbd7b 100644 --- a/spec/helpers/callouts_helper_spec.rb +++ b/spec/helpers/callouts_helper_spec.rb @@ -41,5 +41,13 @@ describe CalloutsHelper do it { is_expected.to be false } end + + context 'when the user is not logged in' do + before do + allow(helper).to receive(:current_user).and_return(nil) + end + + it { is_expected.to be_falsey } + end end end |