]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
Add system tests for ensure parameter
authorKen Barber <ken@bob.sh>
Wed, 24 Apr 2013 21:13:48 +0000 (22:13 +0100)
committerKen Barber <ken@bob.sh>
Wed, 24 Apr 2013 21:13:48 +0000 (22:13 +0100)
Signed-off-by: Ken Barber <ken@bob.sh>
spec/system/class_spec.rb

index 8ba8415a53af72b71f1e9c5ad7c86e9fba589091..6199c2fe2ed4e615e29b715f88163c2f1a427532 100644 (file)
@@ -1,23 +1,73 @@
 require 'spec_helper_system'
 
 describe "firewall class:" do
-  let(:pp) do
-    pp = <<-EOS
-      class { 'firewall': }
-    EOS
+  context 'no params:' do
+    let(:pp) do
+      pp = <<-EOS.gsub(/^\s{8}/,'')
+        class { 'firewall': }
+      EOS
+    end
+
+    it "should run without event" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should_not eq(1)
+      end
+    end
+
+    it "should be idempotent" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should == 0
+      end
+    end
   end
 
-  it "should run without event" do
-    puppet_apply(pp) do |r|
-      r[:stderr].should == ''
-      r[:exit_code].should_not eq(1)
+  context 'ensure => stopped:' do
+    let(:pp) do
+      pp = <<-EOS.gsub(/^\s{8}/,'')
+        class { 'firewall':
+          ensure => stopped,
+        }
+      EOS
+    end
+
+    it "should run without event" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should_not eq(1)
+      end
+    end
+
+    it "should be idempotent" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should == 0
+      end
     end
   end
 
-  it "should be idempotent" do
-    puppet_apply(pp) do |r|
-      r[:stderr].should == ''
-      r[:exit_code].should == 0
+  context 'ensure => running:' do
+    let(:pp) do
+      pp = <<-EOS.gsub(/^\s{8}/,'')
+        class { 'firewall':
+          ensure => running,
+        }
+      EOS
+    end
+
+    it "should run without event" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should_not eq(1)
+      end
+    end
+
+    it "should be idempotent" do
+      puppet_apply(pp) do |r|
+        r[:stderr].should == ''
+        r[:exit_code].should == 0
+      end
     end
   end
 end