Fix config[:ssh] to be quoted properly

[zuazo]

With the this small fix, the following example will work as expected:

driver:
  ssh:
    shell: 'sh'

Related to: #131, bf38f94

[fnichol]

Looks good, but the change is causing a test to fail. Think you could update the spec(s) to pass?

[zuazo]

Sorry @fnichol, my fault 😟 Unit tests fixed.

[tam7t]

This change breaks the

---
driver:
  name: vagrant
  require_chef_omnibus: true
  ssh:
    insert_key: false

workflow since the output becomes:

Vagrant.configure("2") do |c|
...
  c.ssh.insert_key = "false"
...
end

instead of

Vagrant.configure("2") do |c|
...
  c.ssh.insert_key = false
...
end

[erikaheidi]

Confirming the problem reported by @tam7t . The generated Vagrantfile is using quotes for "false" which then results in the option not to be correctly handled by Vagrant. Spent a good time trying to figure out what was wrong, only getting the "Authentication failure. Retrying.." from Vagrant. After manually editing the Vagrantfile and removing the quotes, I was able to run kitchen converge.

Test Kitchen version 1.7.3
kitchen-vagrant 0.18.0

[zuazo]

Sorry, guys, I had completely forgotten this issue.

I have proposed a possible solution here: #231