From c5e4b443ffdc1c094450d08d29bd96e43376d6d7 Mon Sep 17 00:00:00 2001 From: Douwe Maan Date: Wed, 13 May 2015 09:45:34 +0200 Subject: Fix GitAccess. --- lib/gitlab/git_access.rb | 5 +++-- spec/lib/gitlab/git_access_spec.rb | 14 +++----------- 2 files changed, 6 insertions(+), 13 deletions(-) diff --git a/lib/gitlab/git_access.rb b/lib/gitlab/git_access.rb index c97249d49e2..c90184d31cf 100644 --- a/lib/gitlab/git_access.rb +++ b/lib/gitlab/git_access.rb @@ -136,7 +136,7 @@ module Gitlab end unless user.can?(action, project) - return + status = case action when :force_push_code_to_protected_branches build_status_object(false, "You are not allowed to force push code to a protected branch on this project.") @@ -148,7 +148,8 @@ module Gitlab build_status_object(false, "You are not allowed to change existing tags on this project.") else # :push_code build_status_object(false, "You are not allowed to push code to this project.") - end + end + return status end build_status_object(true) diff --git a/spec/lib/gitlab/git_access_spec.rb b/spec/lib/gitlab/git_access_spec.rb index 39be9d64644..c7291689e32 100644 --- a/spec/lib/gitlab/git_access_spec.rb +++ b/spec/lib/gitlab/git_access_spec.rb @@ -115,18 +115,10 @@ describe Gitlab::GitAccess do let(:actor) { key } context 'pull code' do - context 'allowed' do - before { key.projects << project } - subject { access.download_access_check } - - it { expect(subject.allowed?).to be_truthy } - end - - context 'denied' do - subject { access.download_access_check } + before { key.projects << project } + subject { access.download_access_check } - it { expect(subject.allowed?).to be_falsey } - end + it { expect(subject.allowed?).to be_truthy } end end end -- cgit v1.2.1