diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2016-12-10 03:46:50 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2016-12-10 03:46:50 +0800 |
commit | 6269f523f0f47e98d581d62e543b84185255454a (patch) | |
tree | e4b1f10128e9ae6a105f28e054872a3de0c4939a | |
parent | f97965395198ef1da892dde246edb1e6ef480127 (diff) | |
download | gitlab-ce-6269f523f0f47e98d581d62e543b84185255454a.tar.gz |
Fix tests and also add tests for non-existing repo
-rw-r--r-- | spec/requests/git_http_spec.rb | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/spec/requests/git_http_spec.rb b/spec/requests/git_http_spec.rb index f1728d61def..c701b9a1202 100644 --- a/spec/requests/git_http_spec.rb +++ b/spec/requests/git_http_spec.rb @@ -371,12 +371,26 @@ describe 'Git HTTP requests', lib: true do shared_examples 'can download code only' do it 'downloads get status 200' do - clone_get "#{project.path_with_namespace}.git", user: 'gitlab-ci-token', password: build.token + allow_any_instance_of(Repository). + to receive(:exists?).and_return(true) + + clone_get "#{project.path_with_namespace}.git", + user: 'gitlab-ci-token', password: build.token expect(response).to have_http_status(200) expect(response.content_type.to_s).to eq(Gitlab::Workhorse::INTERNAL_API_CONTENT_TYPE) end + it 'downloads from non-existing repository and gets 403' do + allow_any_instance_of(Repository). + to receive(:exists?).and_return(false) + + clone_get "#{project.path_with_namespace}.git", + user: 'gitlab-ci-token', password: build.token + + expect(response).to have_http_status(403) + end + it 'uploads get status 403' do push_get "#{project.path_with_namespace}.git", user: 'gitlab-ci-token', password: build.token |