]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
(#10324) Remove extraneous whitespace from iptables rule line in spec tests
authorKen Barber <ken@bob.sh>
Thu, 27 Oct 2011 11:20:36 +0000 (12:20 +0100)
committerKen Barber <ken@bob.sh>
Thu, 27 Oct 2011 11:20:36 +0000 (12:20 +0100)
The extra spaces between port numbers were messing up the parse and
putting elements in the wrong parts of the hash during rule_to_hash.

This wasn't causing a test problem now, but was creating an issue for
other work people were doing on the module.

spec/unit/puppet/provider/iptables_spec.rb

index 4da37ed97ec730a6839b903f6656a8404c26b37a..19e70a4c78ae500422ab057506b2496a95ebbfa9 100644 (file)
@@ -109,7 +109,7 @@ describe 'iptables provider' do
 
   describe 'when converting rules without comments to resources' do
     let(:sample_rule) {
-      '-A INPUT -s 1.1.1.1 -d 1.1.1.1 -p tcp -m multiport --dports 7061,7062 -m multiport --sports 7061, 7062 -j ACCEPT'
+      '-A INPUT -s 1.1.1.1 -d 1.1.1.1 -p tcp -m multiport --dports 7061,7062 -m multiport --sports 7061,7062 -j ACCEPT'
     }
     let(:resource) { provider.rule_to_hash(sample_rule, 'filter', 0) }
     let(:instance) { provider.new(resource) }
@@ -145,7 +145,7 @@ describe 'iptables provider' do
 
   describe 'when deleting resources' do
     let(:sample_rule) {
-      '-A INPUT -s 1.1.1.1 -d 1.1.1.1 -p tcp -m multiport --dports 7061,7062 -m multiport --sports 7061, 7062 -j ACCEPT'
+      '-A INPUT -s 1.1.1.1 -d 1.1.1.1 -p tcp -m multiport --dports 7061,7062 -m multiport --sports 7061,7062 -j ACCEPT'
     }
     let(:resource) { provider.rule_to_hash(sample_rule, 'filter', 0) }
     let(:instance) { provider.new(resource) }