Fix ChefSpec tests to work properly with attributes
Correct ChefSpec tests for execute[shorewall_make] resource

The resource itself does `:nothing`
Later resources (templates) notify the `execute[shorewall_make]` resource
to `:run` delayed.
Ensure shorewall only starts after shorewall_make, add tests

Adds a notify for shorewall_make, which is run delayed from the
template resources for shorewall's configs' creation, to start shorewall's
service after shorewall_make completes.

Avoids race issues and starting the service prematurely.

Add ChefSpec tests to ensure this very behavior.
Fix array to work properly

Unlike `[ ]` style arrays, the `%w()` style treats everything as part of the
string, so putting `%w('string', 'string')` results in an array that looks like:
`["'string',", "'string'"]`

That is, each item outputs, literally, as `'string',` with the apostrophes &
comma intact. Bad juju.