use YAML.load instead of YAML.parse

chorankates / rouster

Rouster.is_a?('abstraction layer for controlling Vagrant virtual machines') => true

BSD 3-Clause "New" or "Revised" License

109 stars 14 forks source link

use YAML.load instead of YAML.parse #119

Closed petersenallen closed 6 years ago

petersenallen commented 6 years ago

YAML.load will return a hash, but YAML.parse was returning a Psych::Nodes::Document object. This PR also updates a few usages of .get_output and fixes a couple of spots that were broken after the refactor of .run. The new method does not handle double quotes well if sudo is used and was breaking .get_packages in our tests.