Failure to find node facts is now a failure.

It was previously just a warning, but the node
is essentially non-functional without the facts,
so it makes more sense for it to be a warning.

Signed-off-by: Luke Kanies <luke@madstop.com>

1 files changed, 5 insertions, 0 deletions

diff --git a/spec/unit/node.rb b/spec/unit/node.rb
index 25dad8c67..99c53e008 100755
--- a/spec/unit/node.rb
+++ b/spec/unit/node.rb
@@ -76,6 +76,11 @@ describe Puppet::Node, "when merging facts" do
         Puppet::Node::Facts.stubs(:find).with(@node.name).returns(Puppet::Node::Facts.new(@node.name, "one" => "c", "two" => "b"))
     end
 
+    it "should fail intelligently if it cannot find facts" do
+        Puppet::Node::Facts.expects(:find).with(@node.name).raises "foo"
+        lambda { @node.fact_merge }.should raise_error(Puppet::Error)
+    end
+
     it "should prefer parameters already set on the node over facts from the node" do
         @node.parameters = {"one" => "a"}
         @node.fact_merge