]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commit
Fix puppet resource acceptance tests
authorColleen Murphy <colleen@puppetlabs.com>
Tue, 30 Dec 2014 20:46:55 +0000 (12:46 -0800)
committerColleen Murphy <colleen@puppetlabs.com>
Tue, 30 Dec 2014 20:54:29 +0000 (12:54 -0800)
commitd0f4c84879d6d305303fd232ff96bd2a7580e8ad
treee43247a2edd2117858d4a27c6092c183d0a58002
parent8b5a8ab60ccd2d4a051dbf685bba6685662a693b
Fix puppet resource acceptance tests

The dport/sport test needs iptables-persistent pre-installed in order
for the iptables_persistent_version fact to work.

This commit also changes the test to create the rule with a comment and
with the default protocol so that the puppet resource command is
checking the existence of a resource and not creating a new one or
altering an existing one.

After those changes, checking stderr still fails because of puppet's
deprecation warnings. This commit fixes puppet.conf to avoid the
warnings so that we can actually check stderr. With that accomplished I
added checks for stderr to the other puppet resource tests since they
should now be reliable.
spec/acceptance/resource_cmd_spec.rb