chef-boneyard / delivery-sugar

DEPRECATED: Library cookbook that adds sugar to build cookbooks for Chef Delivery
Apache License 2.0
12 stars 26 forks source link

Refactor TestKitchen class to be shorter #15

Open mattstratton opened 7 years ago

mattstratton commented 7 years ago

Once we merge in #14, the TestKitchen class is longer than 100 lines, which causes a Rubocop failure. In #14 we are working around this by disabling that cop on that particular class, but this seems to be a bit of a band-aid.

Especially as we add more and more drivers to this, the class will start to get really long. We should refactor this class so that perhaps the drivers each subclass the TestKitchen class?

Open to discussion on how to approach this. I'm hoping to add a bunch more drivers shortly, so I'd like to figure this out sooner than later.