Add f5_passwordpolicy type/provider #14
Conversation
| Puppet::Type.newtype(:f5_passwordpolicy) do | ||
| @doc = 'Sets the password policy on the BIG-IP system.' | ||
|
|
||
| apply_to_device if Facter.value(:url).nil? |
There was a problem hiding this comment.
The if Facter.value(:url).nil? makes it possible to use puppet resource or puppet apply to set properties. This is really helpful during testing! :)
abrader
left a comment
abrader
left a comment
There was a problem hiding this comment.
Any chance you can provide acceptance tests as well @alexjfisher ? We are updating the README to indicate additional types and providers should come with passing acceptance tests. Thank you.
|
@abrader I would like to add acceptance tests, but I'm not sure I'm able to run them locally. It looks like they're setup to use Puppet Inc's vmpooler testing infrastructure and/or AWS? If they were automatically run by travis that'd be really helpful. |
alexjfisher
force-pushed
the
password_policy
branch
from
219562a to
28adb2c
Compare
|
I've added a really simple acceptance test. It should work, but I've not got an AWS F5 test environment, so I haven't actually run it! |
2 participants
No description provided.