Add seperated 'describe' block for build trace specs

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