Closed bbaugher closed 9 years ago
I'd been meaning to add some tests but hadn't had time, so yay!
Hey @bbaugher, thanks a lot for the PR!
I have one comment. Otherwise, this PR seems legit. Thanks for taking a look at zookeeper_node
; that is some old functionality that hasn't gotten the love it deserves, so it's super rad to see it get a look. Thanks for the tests too, love it.
Removed the include_recipes from the service recipe.
Thanks!
Originally started this to fix a namespace issue with the zookeeper_node provider referencing the zookeeper provider instead of the zookeeper client.
The other fixes were surprises I found when setting up testing for this