Commit 6f3d23f3 authored by Matija Čupić's avatar Matija Čupić

Add variables option to Ci::CreatePipelineService

parent c5902d08
......@@ -27,6 +27,7 @@ module Ci
ignore_skip_ci: ignore_skip_ci,
save_incompleted: save_on_errors,
seeds_block: block,
variables: params[:variables_attributes],
project: project,
current_user: current_user,
......
......@@ -17,11 +17,13 @@ describe Ci::CreatePipelineService do
after: project.commit.id,
message: 'Message',
ref: ref_name,
trigger_request: nil)
trigger_request: nil,
variables: nil)
params = { ref: ref,
before: '00000000',
after: after,
commits: [{ message: message }] }
commits: [{ message: message }],
variables_attributes: variables }
described_class.new(project, user, params).execute(
source, trigger_request: trigger_request)
......@@ -545,5 +547,20 @@ describe Ci::CreatePipelineService do
expect(pipeline.tag?).to be true
end
end
context 'when pipeline variables are specified' do
let(:variables) do
[{ key: 'first', secret_value: 'world' },
{ key: 'second', secret_value: 'second_world' }]
end
subject { execute_service(variables: variables) }
it 'creates a pipeline with specified variables' do
expect(subject.variables.count).to eq(variables.count)
expect(subject.variables.first.key).to eq(variables.first[:key])
expect(subject.variables.last.secret_value).to eq(variables.last[:secret_value])
end
end
end
end
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