Turn options into indifferent access

ManageIQ · Oct 23, 2017 · efe4ee8 · efe4ee8
1 parent 6458e00
commit efe4ee8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb b/app/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner.rb
@@ -1,7 +1,7 @@
 class ManageIQ::Providers::EmbeddedAnsible::AutomationManager::PlaybookRunner < ::Job
   # options are job table columns, including options column which is the playbook context info
   def self.create_job(options)
-    super(name, options)
+    super(name, options.with_indifferent_access)
   end
 
   def start

diff --git a/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb b/spec/models/manageiq/providers/embedded_ansible/automation_manager/playbook_runner_spec.rb
@@ -36,7 +36,8 @@
     end
 
     context 'other hosts are used' do
-      let(:options) { {:hosts => 'host1,host2'} }
+      # Use string key to also test the indifferent accessibility
+      let(:options) { {'hosts' => 'host1,host2'} }
 
       it 'creates an inventory and moves on to create_job_template' do
         expect(ManageIQ::Providers::EmbeddedAnsible::AutomationManager::Inventory).to receive(:raw_create_inventory).and_return(double(:id => 'inv1'))