We were previously incorrectly adding double quotes for the alias values.
According to the man page it's not required. However, there is a requirement if
its a command being pipe.
In addition, the man page mentions this for the alias name:
The name is a local address (no domain part). Use double quotes when the
name contains any special characters such as whitespace, `#', `:', or `@'. The
name is folded to lowercase, in order to make database lookups case
insensitive.
This includes logic to support this automatically.
- Add new tests for aliases testing all types
- Convert all serverspec tests to inspec
- Add Github actions
- Update platforms to test
Signed-off-by: Lance Albertson <lance@osuosl.org>
* Wrapper cookbook usually uses default level to override attributes
of the wrapped cookbook. Here, we had a problem because if the
wrapper cookbook was enabling sasl and was setting a relayhost
and a user/password combination using the default level, this was
not working because _attributes resets those attributes.
To fix this, we now use node.default_unless which will set the
attribute only if it's not already set ensuring that wrapper
cookbook overriden attributes using default level are correctly
kept when computing derived attributes.
