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>
This change is automatically generated by the Cookstyle Bot using the latest version of Cookstyle (6.18.8). Adopting changes suggested by Cookstyle improves cookbook readability, avoids common coding mistakes, and eases upgrades to newer versions of the Chef Infra Client.
### Style/Encoding
- **/recipes/maps.rb:1**: Unnecessary utf-8 encoding comment.
- **/test/integration/sasl_auth_multiple/serverspec/sasl_auth_multiple_spec.rb:1**: Unnecessary utf-8 encoding comment.
- **/test/integration/sasl_auth_one/serverspec/sasl_auth_one_spec.rb:1**: Unnecessary utf-8 encoding comment.
Signed-off-by: Cookstyle <cookbooks@chef.io>
* 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.
