Enable setting guestinfo

jasonwbarnett commented 3 years ago

Description

This PR aims to enable setting guestinfo from test-kitchen.

Related Issue

N/A

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

jasonwbarnett commented 3 years ago

@tas50 this is by and large ready to go, but I want to test this against production before giving you the go ahead. I'll also be adding at least a few more unit tests.

jasonwbarnett commented 3 years ago

@tas50 I validated this is all working in production. This is ready for review 👁️

thheinen commented 3 years ago

My 2 ct:

Very glad to see the start of testability in this driver, as it grew and grew... and the fear of breaking something grew as well (for me at least). I will probably steal some of that code for my own projects :-)

Also, adding guestinfo crossed my mind a while ago as well. I think it's a good addition :+1:

chef / kitchen-vcenter