Fixing #540. I modified Puppet::Network::Client::Master so that it disables noop during its run, so that facts and plugins will always be downloaded.

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@2285 980ebf18-57e1-0310-9a29-db15c13687c0

1 files changed, 20 insertions, 0 deletions

diff --git a/test/network/client/master.rb b/test/network/client/master.rb
index ec6c8d5cf..270c3e08c 100755
--- a/test/network/client/master.rb
+++ b/test/network/client/master.rb
@@ -39,6 +39,11 @@ class TestMasterClient < Test::Unit::TestCase
         end
     end
 
+    def setup
+        super
+        @master = Puppet::Network::Client.master
+    end
+
     def mkmaster(file = nil)
         master = nil
 
@@ -525,6 +530,21 @@ end
         assert(Puppet::Type.type(:schedule)["hourly"], "Could not retrieve hourly schedule")
         assert(Puppet::Type.type(:filebucket)["puppet"], "Could not retrieve default bucket")
     end
+
+    # #540 - make sure downloads aren't affected by noop
+    def test_download_in_noop
+        source = tempfile
+        File.open(source, "w") { |f| f.puts "something" }
+        dest = tempfile
+        Puppet[:noop] = true
+        assert_nothing_raised("Could not download in noop") do
+            @master.download(:dest => dest, :source => source, :tag => "yay")
+        end
+
+        assert(FileTest.exists?(dest), "did not download in noop mode")
+
+        assert(Puppet[:noop], "noop got disabled in run")
+    end
 end
 
 # $Id$