Skip to content

Commit

Permalink
Add flag for init of defaults in fields
Browse files Browse the repository at this point in the history
  • Loading branch information
d-m-u committed Oct 8, 2018
1 parent 41f44af commit 148893f
Show file tree
Hide file tree
Showing 5 changed files with 37 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/models/resource_action_workflow.rb
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ def load_dialog(resource_action, values, options)
dialog.target_resource = @target
if options[:display_view_only]
dialog.init_fields_with_values_for_request(values)
elsif options[:provision_workflow]
elsif options[:provision_workflow] || options[:init_defaults]
dialog.initialize_value_context(values)
dialog.load_values_into_fields(values, false)
elsif options[:refresh] || options[:submit_workflow]
Expand Down
3 changes: 2 additions & 1 deletion app/models/service_template.rb
Original file line number Diff line number Diff line change
Expand Up @@ -427,7 +427,8 @@ def provision_workflow(user, dialog_options = nil, request_options = {})
:target => self,
:initiator => request_options[:initiator],
:submit_workflow => request_options[:submit_workflow],
:provision_workflow => request_options[:provision_workflow]
:provision_workflow => request_options[:provision_workflow],
:init_defaults => request_options[:init_defaults]
}

ResourceActionWorkflow.new(dialog_options, user, provision_action, ra_options).tap do |wf|
Expand Down
1 change: 1 addition & 0 deletions config/settings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -979,6 +979,7 @@
:name:
:maindb: ExtManagementSystem
:container_deployment_wizard: false
:run_automate_methods_on_service_catalog_api_submit: false
:prototype:
:queue_type: miq_queue
:artemis:
Expand Down
10 changes: 10 additions & 0 deletions spec/models/resource_action_workflow_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,16 @@
end
end

context "when the options set init_defaults to true" do
let(:options) { {:init_defaults => true} }

it "calls init_fields_with_values_for_request" do
expect(dialog).to receive(:initialize_value_context).with(values).ordered
expect(dialog).to receive(:load_values_into_fields).with(values, false).ordered
ResourceActionWorkflow.new(values, nil, resource_action, options)
end
end

context "when the options are set to a refresh request" do
let(:options) { {:refresh => true} }

Expand Down
23 changes: 23 additions & 0 deletions spec/models/service_template_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -882,6 +882,29 @@
context "#provision_request" do
let(:arg1) { {'ordered_by' => 'fred'} }

context "with init_defaults" do
let(:arg2) { {:initiator => 'control', :init_defaults => true} }

it "provisions a service template without errors" do
expect(resource_action_workflow).to receive(:validate_dialog).and_return([])
expect(resource_action_workflow).to receive(:make_request).and_return(miq_request)
expect(resource_action_workflow).to receive(:request_options=).with(
:initiator => 'control', :init_defaults => true, :provision_workflow => true
)

expect(service_template.provision_request(user, arg1, arg2)).to eq(miq_request)
end

it "provisions a service template with errors" do
expect(resource_action_workflow).to receive(:validate_dialog).and_return(%w(Error1 Error2))
expect(resource_action_workflow).to receive(:request_options=).with(
:initiator => 'control', :init_defaults => true, :provision_workflow => true
)

expect { service_template.provision_request(user, arg1, arg2) }.to raise_error(RuntimeError)
end
end

context "with submit_workflow" do
let(:arg2) { {:initiator => 'control', :submit_workflow => true} }

Expand Down

0 comments on commit 148893f

Please sign in to comment.