diff options
| -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") |