Update Driver access to Busser methods due to impending upstream change.

doubt72 / kitchen-metal

A metal driver for Test Kitchen.

Other

38 stars 5 forks source link

Update Driver access to Busser methods due to impending upstream change. #1

Closed fnichol closed 10 years ago

fnichol commented 10 years ago

It should be safer long term to ask the busser object directly for its commands so that the base Driver class doesn't have to proxy all of Busser's API.

After looking around as much as possible for example in the wild, I'm going to remove the following methods from Kitchen::Driver::Base:

#busser_setup_cmd
#busser_sync_cmd
#busser_run_cmd

The fix here is a drop-in replacement. Thanks!

fnichol commented 10 years ago

Note methods removed as of https://github.com/test-kitchen/test-kitchen/commit/032ada6a36532c07a5ed2190ca9449afd58ca728

doubt72 commented 10 years ago

These changes happened to the driver independently (as we discussed last week).

(Otherwise would have just merged it! :+1: )