diff options
Diffstat (limited to 'spec/requests/api/pipelines_spec.rb')
| -rw-r--r-- | spec/requests/api/pipelines_spec.rb | 358 |
1 files changed, 179 insertions, 179 deletions
diff --git a/spec/requests/api/pipelines_spec.rb b/spec/requests/api/pipelines_spec.rb index 52599db9a9e..21ca3624ada 100644 --- a/spec/requests/api/pipelines_spec.rb +++ b/spec/requests/api/pipelines_spec.rb @@ -1,4 +1,4 @@ -require 'spec_helper' +require "spec_helper" describe API::Pipelines do let(:user) { create(:user) } @@ -14,85 +14,85 @@ describe API::Pipelines do project.add_maintainer(user) end - describe 'GET /projects/:id/pipelines ' do - context 'authorized user' do - it 'returns project pipelines' do + describe "GET /projects/:id/pipelines " do + context "authorized user" do + it "returns project pipelines" do get api("/projects/#{project.id}/pipelines", user) expect(response).to have_gitlab_http_status(200) expect(response).to include_pagination_headers expect(json_response).to be_an Array - expect(json_response.first['sha']).to match /\A\h{40}\z/ - expect(json_response.first['id']).to eq pipeline.id - expect(json_response.first['web_url']).to be_present - expect(json_response.first.keys).to contain_exactly(*%w[id sha ref status web_url]) + expect(json_response.first["sha"]).to match /\A\h{40}\z/ + expect(json_response.first["id"]).to eq pipeline.id + expect(json_response.first["web_url"]).to be_present + expect(json_response.first.keys).to contain_exactly("id", "sha", "ref", "status", "web_url") end - context 'when parameter is passed' do + context "when parameter is passed" do %w[running pending].each do |target| context "when scope is #{target}" do before do create(:ci_pipeline, project: project, status: target) end - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { scope: target } + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {scope: target} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - json_response.each { |r| expect(r['status']).to eq(target) } + json_response.each { |r| expect(r["status"]).to eq(target) } end end end - context 'when scope is finished' do + context "when scope is finished" do before do - create(:ci_pipeline, project: project, status: 'success') - create(:ci_pipeline, project: project, status: 'failed') - create(:ci_pipeline, project: project, status: 'canceled') + create(:ci_pipeline, project: project, status: "success") + create(:ci_pipeline, project: project, status: "failed") + create(:ci_pipeline, project: project, status: "canceled") end - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { scope: 'finished' } + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {scope: "finished"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - json_response.each { |r| expect(r['status']).to be_in(%w[success failed canceled]) } + json_response.each { |r| expect(r["status"]).to be_in(%w[success failed canceled]) } end end - context 'when scope is branches or tags' do + context "when scope is branches or tags" do let!(:pipeline_branch) { create(:ci_pipeline, project: project) } - let!(:pipeline_tag) { create(:ci_pipeline, project: project, ref: 'v1.0.0', tag: true) } + let!(:pipeline_tag) { create(:ci_pipeline, project: project, ref: "v1.0.0", tag: true) } - context 'when scope is branches' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { scope: 'branches' } + context "when scope is branches" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {scope: "branches"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - expect(json_response.last['id']).to eq(pipeline_branch.id) + expect(json_response.last["id"]).to eq(pipeline_branch.id) end end - context 'when scope is tags' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { scope: 'tags' } + context "when scope is tags" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {scope: "tags"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - expect(json_response.last['id']).to eq(pipeline_tag.id) + expect(json_response.last["id"]).to eq(pipeline_tag.id) end end end - context 'when scope is invalid' do - it 'returns bad_request' do - get api("/projects/#{project.id}/pipelines", user), params: { scope: 'invalid-scope' } + context "when scope is invalid" do + it "returns bad_request" do + get api("/projects/#{project.id}/pipelines", user), params: {scope: "invalid-scope"} expect(response).to have_gitlab_http_status(:bad_request) end @@ -106,44 +106,44 @@ describe API::Pipelines do create(:ci_pipeline, project: project, status: exception_status.sample) end - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { status: target } + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {status: target} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - json_response.each { |r| expect(r['status']).to eq(target) } + json_response.each { |r| expect(r["status"]).to eq(target) } end end end - context 'when status is invalid' do - it 'returns bad_request' do - get api("/projects/#{project.id}/pipelines", user), params: { status: 'invalid-status' } + context "when status is invalid" do + it "returns bad_request" do + get api("/projects/#{project.id}/pipelines", user), params: {status: "invalid-status"} expect(response).to have_gitlab_http_status(:bad_request) end end - context 'when ref is specified' do + context "when ref is specified" do before do create(:ci_pipeline, project: project) end - context 'when ref exists' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { ref: 'master' } + context "when ref exists" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {ref: "master"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty - json_response.each { |r| expect(r['ref']).to eq('master') } + json_response.each { |r| expect(r["ref"]).to eq("master") } end end - context 'when ref does not exist' do - it 'returns empty' do - get api("/projects/#{project.id}/pipelines", user), params: { ref: 'invalid-ref' } + context "when ref does not exist" do + it "returns empty" do + get api("/projects/#{project.id}/pipelines", user), params: {ref: "invalid-ref"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers @@ -152,22 +152,22 @@ describe API::Pipelines do end end - context 'when name is specified' do + context "when name is specified" do let!(:pipeline) { create(:ci_pipeline, project: project, user: user) } - context 'when name exists' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { name: user.name } + context "when name exists" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {name: user.name} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers - expect(json_response.first['id']).to eq(pipeline.id) + expect(json_response.first["id"]).to eq(pipeline.id) end end - context 'when name does not exist' do - it 'returns empty' do - get api("/projects/#{project.id}/pipelines", user), params: { name: 'invalid-name' } + context "when name does not exist" do + it "returns empty" do + get api("/projects/#{project.id}/pipelines", user), params: {name: "invalid-name"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers @@ -176,22 +176,22 @@ describe API::Pipelines do end end - context 'when username is specified' do + context "when username is specified" do let!(:pipeline) { create(:ci_pipeline, project: project, user: user) } - context 'when username exists' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { username: user.username } + context "when username exists" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {username: user.username} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers - expect(json_response.first['id']).to eq(pipeline.id) + expect(json_response.first["id"]).to eq(pipeline.id) end end - context 'when username does not exist' do - it 'returns empty' do - get api("/projects/#{project.id}/pipelines", user), params: { username: 'invalid-username' } + context "when username does not exist" do + it "returns empty" do + get api("/projects/#{project.id}/pipelines", user), params: {username: "invalid-username"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers @@ -200,72 +200,72 @@ describe API::Pipelines do end end - context 'when yaml_errors is specified' do - let!(:pipeline1) { create(:ci_pipeline, project: project, yaml_errors: 'Syntax error') } + context "when yaml_errors is specified" do + let!(:pipeline1) { create(:ci_pipeline, project: project, yaml_errors: "Syntax error") } let!(:pipeline2) { create(:ci_pipeline, project: project) } - context 'when yaml_errors is true' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { yaml_errors: true } + context "when yaml_errors is true" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {yaml_errors: true} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers - expect(json_response.first['id']).to eq(pipeline1.id) + expect(json_response.first["id"]).to eq(pipeline1.id) end end - context 'when yaml_errors is false' do - it 'returns matched pipelines' do - get api("/projects/#{project.id}/pipelines", user), params: { yaml_errors: false } + context "when yaml_errors is false" do + it "returns matched pipelines" do + get api("/projects/#{project.id}/pipelines", user), params: {yaml_errors: false} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers - expect(json_response.first['id']).to eq(pipeline2.id) + expect(json_response.first["id"]).to eq(pipeline2.id) end end - context 'when yaml_errors is invalid' do - it 'returns bad_request' do - get api("/projects/#{project.id}/pipelines", user), params: { yaml_errors: 'invalid-yaml_errors' } + context "when yaml_errors is invalid" do + it "returns bad_request" do + get api("/projects/#{project.id}/pipelines", user), params: {yaml_errors: "invalid-yaml_errors"} expect(response).to have_gitlab_http_status(:bad_request) end end end - context 'when order_by and sort are specified' do - context 'when order_by user_id' do + context "when order_by and sort are specified" do + context "when order_by user_id" do before do 3.times do create(:ci_pipeline, project: project, user: create(:user)) end end - context 'when sort parameter is valid' do - it 'sorts as user_id: :desc' do - get api("/projects/#{project.id}/pipelines", user), params: { order_by: 'user_id', sort: 'desc' } + context "when sort parameter is valid" do + it "sorts as user_id: :desc" do + get api("/projects/#{project.id}/pipelines", user), params: {order_by: "user_id", sort: "desc"} expect(response).to have_gitlab_http_status(:ok) expect(response).to include_pagination_headers expect(json_response).not_to be_empty pipeline_ids = Ci::Pipeline.all.order(user_id: :desc).pluck(:id) - expect(json_response.map { |r| r['id'] }).to eq(pipeline_ids) + expect(json_response.map { |r| r["id"] }).to eq(pipeline_ids) end end - context 'when sort parameter is invalid' do - it 'returns bad_request' do - get api("/projects/#{project.id}/pipelines", user), params: { order_by: 'user_id', sort: 'invalid_sort' } + context "when sort parameter is invalid" do + it "returns bad_request" do + get api("/projects/#{project.id}/pipelines", user), params: {order_by: "user_id", sort: "invalid_sort"} expect(response).to have_gitlab_http_status(:bad_request) end end end - context 'when order_by is invalid' do - it 'returns bad_request' do - get api("/projects/#{project.id}/pipelines", user), params: { order_by: 'lock_version', sort: 'asc' } + context "when order_by is invalid" do + it "returns bad_request" do + get api("/projects/#{project.id}/pipelines", user), params: {order_by: "lock_version", sort: "asc"} expect(response).to have_gitlab_http_status(:bad_request) end @@ -274,152 +274,152 @@ describe API::Pipelines do end end - context 'unauthorized user' do - it 'does not return project pipelines' do + context "unauthorized user" do + it "does not return project pipelines" do get api("/projects/#{project.id}/pipelines", non_member) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Project Not Found' + expect(json_response["message"]).to eq "404 Project Not Found" expect(json_response).not_to be_an Array end end end - describe 'POST /projects/:id/pipeline ' do + describe "POST /projects/:id/pipeline " do def expect_variables(variables, expected_variables) variables.each_with_index do |variable, index| expected_variable = expected_variables[index] - expect(variable.key).to eq(expected_variable['key']) - expect(variable.value).to eq(expected_variable['value']) + expect(variable.key).to eq(expected_variable["key"]) + expect(variable.value).to eq(expected_variable["value"]) end end - context 'authorized user' do - context 'with gitlab-ci.yml' do + context "authorized user" do + context "with gitlab-ci.yml" do before do stub_ci_pipeline_to_return_yaml_file end - it 'creates and returns a new pipeline' do - expect do - post api("/projects/#{project.id}/pipeline", user), params: { ref: project.default_branch } - end.to change { project.ci_pipelines.count }.by(1) + it "creates and returns a new pipeline" do + expect { + post api("/projects/#{project.id}/pipeline", user), params: {ref: project.default_branch} + }.to change { project.ci_pipelines.count }.by(1) expect(response).to have_gitlab_http_status(201) expect(json_response).to be_a Hash - expect(json_response['sha']).to eq project.commit.id + expect(json_response["sha"]).to eq project.commit.id end - context 'variables given' do - let(:variables) { [{ 'key' => 'UPLOAD_TO_S3', 'value' => 'true' }] } + context "variables given" do + let(:variables) { [{"key" => "UPLOAD_TO_S3", "value" => "true"}] } - it 'creates and returns a new pipeline using the given variables' do - expect do - post api("/projects/#{project.id}/pipeline", user), params: { ref: project.default_branch, variables: variables } - end.to change { project.ci_pipelines.count }.by(1) + it "creates and returns a new pipeline using the given variables" do + expect { + post api("/projects/#{project.id}/pipeline", user), params: {ref: project.default_branch, variables: variables} + }.to change { project.ci_pipelines.count }.by(1) expect_variables(project.ci_pipelines.last.variables, variables) expect(response).to have_gitlab_http_status(201) expect(json_response).to be_a Hash - expect(json_response['sha']).to eq project.commit.id - expect(json_response).not_to have_key('variables') + expect(json_response["sha"]).to eq project.commit.id + expect(json_response).not_to have_key("variables") end end - describe 'using variables conditions' do - let(:variables) { [{ 'key' => 'STAGING', 'value' => 'true' }] } + describe "using variables conditions" do + let(:variables) { [{"key" => "STAGING", "value" => "true"}] } before do - config = YAML.dump(test: { script: 'test', only: { variables: ['$STAGING'] } }) + config = YAML.dump(test: {script: "test", only: {variables: ["$STAGING"]}}) stub_ci_pipeline_yaml_file(config) end - it 'creates and returns a new pipeline using the given variables' do - expect do - post api("/projects/#{project.id}/pipeline", user), params: { ref: project.default_branch, variables: variables } - end.to change { project.ci_pipelines.count }.by(1) + it "creates and returns a new pipeline using the given variables" do + expect { + post api("/projects/#{project.id}/pipeline", user), params: {ref: project.default_branch, variables: variables} + }.to change { project.ci_pipelines.count }.by(1) expect_variables(project.ci_pipelines.last.variables, variables) expect(response).to have_gitlab_http_status(201) expect(json_response).to be_a Hash - expect(json_response['sha']).to eq project.commit.id - expect(json_response).not_to have_key('variables') + expect(json_response["sha"]).to eq project.commit.id + expect(json_response).not_to have_key("variables") end - context 'condition unmatch' do - let(:variables) { [{ 'key' => 'STAGING', 'value' => 'false' }] } + context "condition unmatch" do + let(:variables) { [{"key" => "STAGING", "value" => "false"}] } it "doesn't create a job" do - expect do - post api("/projects/#{project.id}/pipeline", user), params: { ref: project.default_branch } - end.not_to change { project.ci_pipelines.count } + expect { + post api("/projects/#{project.id}/pipeline", user), params: {ref: project.default_branch} + }.not_to change { project.ci_pipelines.count } expect(response).to have_gitlab_http_status(400) end end end - it 'fails when using an invalid ref' do - post api("/projects/#{project.id}/pipeline", user), params: { ref: 'invalid_ref' } + it "fails when using an invalid ref" do + post api("/projects/#{project.id}/pipeline", user), params: {ref: "invalid_ref"} expect(response).to have_gitlab_http_status(400) - expect(json_response['message']['base'].first).to eq 'Reference not found' + expect(json_response["message"]["base"].first).to eq "Reference not found" expect(json_response).not_to be_an Array end end - context 'without gitlab-ci.yml' do - context 'without auto devops enabled' do + context "without gitlab-ci.yml" do + context "without auto devops enabled" do before do - project.update!(auto_devops_attributes: { enabled: false }) + project.update!(auto_devops_attributes: {enabled: false}) end - it 'fails to create pipeline' do - post api("/projects/#{project.id}/pipeline", user), params: { ref: project.default_branch } + it "fails to create pipeline" do + post api("/projects/#{project.id}/pipeline", user), params: {ref: project.default_branch} expect(response).to have_gitlab_http_status(400) - expect(json_response['message']['base'].first).to eq 'Missing .gitlab-ci.yml file' + expect(json_response["message"]["base"].first).to eq "Missing .gitlab-ci.yml file" expect(json_response).not_to be_an Array end end end end - context 'unauthorized user' do - it 'does not create pipeline' do - post api("/projects/#{project.id}/pipeline", non_member), params: { ref: project.default_branch } + context "unauthorized user" do + it "does not create pipeline" do + post api("/projects/#{project.id}/pipeline", non_member), params: {ref: project.default_branch} expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Project Not Found' + expect(json_response["message"]).to eq "404 Project Not Found" expect(json_response).not_to be_an Array end end end - describe 'GET /projects/:id/pipelines/:pipeline_id' do - context 'authorized user' do - it 'returns project pipelines' do + describe "GET /projects/:id/pipelines/:pipeline_id" do + context "authorized user" do + it "returns project pipelines" do get api("/projects/#{project.id}/pipelines/#{pipeline.id}", user) expect(response).to have_gitlab_http_status(200) - expect(json_response['sha']).to match /\A\h{40}\z/ + expect(json_response["sha"]).to match /\A\h{40}\z/ end - it 'returns 404 when it does not exist' do + it "returns 404 when it does not exist" do get api("/projects/#{project.id}/pipelines/123456", user) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Not found' - expect(json_response['id']).to be nil + expect(json_response["message"]).to eq "404 Not found" + expect(json_response["id"]).to be nil end - context 'with coverage' do + context "with coverage" do before do create(:ci_build, coverage: 30, pipeline: pipeline) end - it 'exposes the coverage' do + it "exposes the coverage" do get api("/projects/#{project.id}/pipelines/#{pipeline.id}", user) expect(json_response["coverage"].to_i).to eq(30) @@ -427,43 +427,43 @@ describe API::Pipelines do end end - context 'unauthorized user' do - it 'should not return a project pipeline' do + context "unauthorized user" do + it "should not return a project pipeline" do get api("/projects/#{project.id}/pipelines/#{pipeline.id}", non_member) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Project Not Found' - expect(json_response['id']).to be nil + expect(json_response["message"]).to eq "404 Project Not Found" + expect(json_response["id"]).to be nil end end end - describe 'DELETE /projects/:id/pipelines/:pipeline_id' do - context 'authorized user' do + describe "DELETE /projects/:id/pipelines/:pipeline_id" do + context "authorized user" do let(:owner) { project.owner } - it 'destroys the pipeline' do + it "destroys the pipeline" do delete api("/projects/#{project.id}/pipelines/#{pipeline.id}", owner) expect(response).to have_gitlab_http_status(204) expect { pipeline.reload }.to raise_error(ActiveRecord::RecordNotFound) end - it 'returns 404 when it does not exist' do + it "returns 404 when it does not exist" do delete api("/projects/#{project.id}/pipelines/123456", owner) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Not found' + expect(json_response["message"]).to eq "404 Not found" end - it 'does not log an audit event' do + it "does not log an audit event" do expect { delete api("/projects/#{project.id}/pipelines/#{pipeline.id}", owner) }.not_to change { SecurityEvent.count } end - context 'when the pipeline has jobs' do + context "when the pipeline has jobs" do let!(:build) { create(:ci_build, project: project, pipeline: pipeline) } - it 'destroys associated jobs' do + it "destroys associated jobs" do delete api("/projects/#{project.id}/pipelines/#{pipeline.id}", owner) expect(response).to have_gitlab_http_status(204) @@ -472,35 +472,35 @@ describe API::Pipelines do end end - context 'unauthorized user' do - context 'when user is not member' do - it 'should return a 404' do + context "unauthorized user" do + context "when user is not member" do + it "should return a 404" do delete api("/projects/#{project.id}/pipelines/#{pipeline.id}", non_member) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Project Not Found' + expect(json_response["message"]).to eq "404 Project Not Found" end end - context 'when user is developer' do + context "when user is developer" do let(:developer) { create(:user) } before do project.add_developer(developer) end - it 'should return a 403' do + it "should return a 403" do delete api("/projects/#{project.id}/pipelines/#{pipeline.id}", developer) expect(response).to have_gitlab_http_status(403) - expect(json_response['message']).to eq '403 Forbidden' + expect(json_response["message"]).to eq "403 Forbidden" end end end end - describe 'POST /projects/:id/pipelines/:pipeline_id/retry' do - context 'authorized user' do + describe "POST /projects/:id/pipelines/:pipeline_id/retry" do + context "authorized user" do let!(:pipeline) do create(:ci_pipeline, project: project, sha: project.commit.id, ref: project.default_branch) @@ -508,28 +508,28 @@ describe API::Pipelines do let!(:build) { create(:ci_build, :failed, pipeline: pipeline) } - it 'retries failed builds' do - expect do + it "retries failed builds" do + expect { post api("/projects/#{project.id}/pipelines/#{pipeline.id}/retry", user) - end.to change { pipeline.builds.count }.from(1).to(2) + }.to change { pipeline.builds.count }.from(1).to(2) expect(response).to have_gitlab_http_status(201) expect(build.reload.retried?).to be true end end - context 'unauthorized user' do - it 'should not return a project pipeline' do + context "unauthorized user" do + it "should not return a project pipeline" do post api("/projects/#{project.id}/pipelines/#{pipeline.id}/retry", non_member) expect(response).to have_gitlab_http_status(404) - expect(json_response['message']).to eq '404 Project Not Found' - expect(json_response['id']).to be nil + expect(json_response["message"]).to eq "404 Project Not Found" + expect(json_response["id"]).to be nil end end end - describe 'POST /projects/:id/pipelines/:pipeline_id/cancel' do + describe "POST /projects/:id/pipelines/:pipeline_id/cancel" do let!(:pipeline) do create(:ci_empty_pipeline, project: project, sha: project.commit.id, ref: project.default_branch) @@ -537,27 +537,27 @@ describe API::Pipelines do let!(:build) { create(:ci_build, :running, pipeline: pipeline) } - context 'authorized user' do - it 'retries failed builds' do + context "authorized user" do + it "retries failed builds" do post api("/projects/#{project.id}/pipelines/#{pipeline.id}/cancel", user) expect(response).to have_gitlab_http_status(200) - expect(json_response['status']).to eq('canceled') + expect(json_response["status"]).to eq("canceled") end end - context 'user without proper access rights' do + context "user without proper access rights" do let!(:reporter) { create(:user) } before do project.add_reporter(reporter) end - it 'rejects the action' do + it "rejects the action" do post api("/projects/#{project.id}/pipelines/#{pipeline.id}/cancel", reporter) expect(response).to have_gitlab_http_status(403) - expect(pipeline.reload.status).to eq('pending') + expect(pipeline.reload.status).to eq("pending") end end end |