Merge pull request #572 from agrare/fix_ae_state_data_load_error

Fix YAML.load error with serialized StateVarHash

(cherry picked from commit bfa2964)

Author: jrafanie

lib/miq_automation_engine/engine/miq_ae_engine/miq_ae_workspace_runtime.rb

@@ -163,7 +163,7 @@ def instantiate(uri, user, root = nil)
       $miq_ae_logger.info("Instantiating [#{ManageIQ::Password.sanitize_string(uri)}]", :resource_id => miq_request_id) if root.nil?
 
       if (ae_state_data = args.delete('ae_state_data'))
-        @persist_state_hash.merge!(YAML.load(ae_state_data))
+        @persist_state_hash.merge!(YAML.safe_load(ae_state_data, :permitted_classes => [MiqAeEngine::StateVarHash]))
       end
 
       if (ae_state_previous = args.delete('ae_state_previous'))

spec/miq_ae_engine_spec.rb

@@ -924,7 +924,7 @@ def before_ae_starts(_options)
     end
 
     it "fetches value from state_var" do
-      ae_state_data = {:my_id => 45}.to_yaml
+      ae_state_data = MiqAeEngine::StateVarHash.new(:my_id => 45).to_yaml
       workspace = MiqAeEngine.instantiate("/A/C/BARNEY/FRED?ae_state_data=#{ae_state_data}", user)
       expect(workspace.root['field1']).to eq("45")
       expect(workspace.root['field2']).to eq('Bamm Bamm Rubble')