Enhance unit test suite
puppet-lint: disable variable scope check

Our Puppet style tends to "create" such top-level variables for our
class parameters, and we don't want to use '$::varname' for those.
Add tests for $zookeeper_connect
Minor code reformatting