diff options
author | Tomasz Maczukin <tomasz@maczukin.pl> | 2016-01-13 18:30:49 +0100 |
---|---|---|
committer | Tomasz Maczukin <tomasz@maczukin.pl> | 2016-01-13 18:30:49 +0100 |
commit | 13032b713d0943c2b7e2f2a3b886ef06be8e88ef (patch) | |
tree | d35a855ca66f35027e7a89806d185ee04d602175 | |
parent | 990bd06c04bebe6319968aa619990bf4cb60483c (diff) | |
download | gitlab-ce-13032b713d0943c2b7e2f2a3b886ef06be8e88ef.tar.gz |
Add seperated 'describe' block for build trace specs
-rw-r--r-- | spec/requests/api/builds_spec.rb | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/spec/requests/api/builds_spec.rb b/spec/requests/api/builds_spec.rb index e5567d42500..8c9f5a382b7 100644 --- a/spec/requests/api/builds_spec.rb +++ b/spec/requests/api/builds_spec.rb @@ -73,7 +73,7 @@ describe API::API, api: true do end end - describe 'GET /projects/:id/builds/:build_id(/trace)?' do + describe 'GET /projects/:id/builds/:build_id' do context 'authorized user' do it 'should return specific build data' do get api("/projects/#{project.id}/builds/#{build.id}", user) @@ -81,7 +81,19 @@ describe API::API, api: true do expect(response.status).to eq(200) expect(json_response['name']).to eq('test') end + end + + context 'unauthorized user' do + it 'should not return specific build data' do + get api("/projects/#{project.id}/builds/#{build.id}") + expect(response.status).to eq(401) + end + end + end + + describe 'GET /projects/:id/builds/:build_id/trace' do + context 'authorized user' do it 'should return specific build trace' do get api("/projects/#{project.id}/builds/#{build_with_trace.id}/trace", user) @@ -91,12 +103,6 @@ describe API::API, api: true do end context 'unauthorized user' do - it 'should not return specific build data' do - get api("/projects/#{project.id}/builds/#{build.id}") - - expect(response.status).to eq(401) - end - it 'should not return specific build trace' do get api("/projects/#{project.id}/builds/#{build_with_trace.id}/trace") |