Commit 4401b759 authored by Shinya Maeda's avatar Shinya Maeda

Fix spec

parent 9b7a0ecf
...@@ -413,14 +413,12 @@ describe Ci::CreatePipelineService do ...@@ -413,14 +413,12 @@ describe Ci::CreatePipelineService do
end end
context 'when trigger belongs to a developer' do context 'when trigger belongs to a developer' do
let(:user) { trigger_request.trigger.owner } let(:user) { create(:user) }
let(:trigger) { create(:ci_trigger, owner: user) }
let(:trigger_request) { create(:ci_trigger_request, trigger: trigger) }
let(:trigger_request) do before do
create(:ci_trigger_request).tap do |request|
user = create(:user)
project.add_developer(user) project.add_developer(user)
request.trigger.update(owner: user)
end
end end
it 'does not create a pipeline' do it 'does not create a pipeline' do
...@@ -431,14 +429,12 @@ describe Ci::CreatePipelineService do ...@@ -431,14 +429,12 @@ describe Ci::CreatePipelineService do
end end
context 'when trigger belongs to a master' do context 'when trigger belongs to a master' do
let(:user) { trigger_request.trigger.owner } let(:user) { create(:user) }
let(:trigger) { create(:ci_trigger, owner: user) }
let(:trigger_request) { create(:ci_trigger_request, trigger: trigger) }
let(:trigger_request) do before do
create(:ci_trigger_request).tap do |request|
user = create(:user)
project.add_master(user) project.add_master(user)
request.trigger.update(owner: user)
end
end end
it 'creates a pipeline' do it 'creates a pipeline' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment