Mod the brokerid by the largest 32bit integer

Prevents kafka from choking on the value when it tries to start up
Add tests for when broker id is > 2**31
Merge branch 'kafka-broker-id-fix'
Add some funky tests to achieve 100% coverage

I think it'd be possible to add some tests for `kafka_download` and
`kafka_install` LWRPs and then get rid of these tests, but right now
I'm not 100% sure.
Add a zookeeper path attribute for the connection string.
Add missing curly bracket to `server.properties` template
Add tests for optional zookeeper path
Don't require path to start with a slash
Merge branch 'kafka-zookeeper-path'
Give props to contributors in README

[ci skip]
Update ChefSpec to v3.4.0
Tidy up unit tests a wee bit
Remove (pointless) `not_if` from `_setup` recipe

Let Chef do its thing.
Prefix name of validate block in `kafka_download` w/ 'kafka'

Global namespaces are global, let's try to keep things isolated to some
extent.
Update `base_url` for downloading archives

Not really 100% satisfied with this solution at all. Probably need to
contact the Kafka team regarding *where* we (people using Chef) are supposed to fetch the archives / packages from. And I'll add a note to
the README that people should keep archives hosted locally as well.
Fix incorrect test description in tests for source recipe
Update tests for `default` recipe to *actually* terminate the Chef run
Set `install_method` to :source in tests for source recipe
Select archive extension based on `version` and `install_method`

I hate myself for this, and I'm pretty sure some kittens died in the
making of this code.
Prefer `%()` over double quotation marks (`""`)