Hopefully fixing #685 -- I added a wrapper around the call to getconfig(), so any timeouts will just throw an error and skip the run, rather than failing and killing the daemon. This is not the best approach, since really, each method should be wrapped, but it is sufficient.

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

1 files changed, 12 insertions, 3 deletions

diff --git a/test/network/client/master.rb b/test/network/client/master.rb
index 87eb22b31..89de171db 100755
--- a/test/network/client/master.rb
+++ b/test/network/client/master.rb
@@ -618,9 +618,7 @@ end
             end
         end
 
-        assert_raise(ArgumentError, "did not fail") do
-            master.run
-        end
+        master.run
 
         assert(! master.send(:lockfile).locked?,
             "Master is still locked after failure")
@@ -725,6 +723,17 @@ end
         assert(FileTest.exists?(@createdfile), "File does not exist on disk")
         assert_nil(ftype[@createdfile], "file object was not removed from memory")
     end
+
+    # #685
+    def test_http_failures_do_not_kill_puppetd
+        client = mkclient
+
+        client.meta_def(:getconfig) { raise "A failure" }
+
+        assert_nothing_raised("Failure in getconfig threw an error") do
+            client.run
+        end
+    end
 end
 
 # $Id$