Working #3139 - Catalogs default to host_config

The whole host_config concept is a bit outdated now that Configurer
exists, I think, (since any catalog it uses should be a host_config).
However, fixing that is outside of the scope of this series.

In the meantime, this does a better job of making sure every
catalog except the Settings catalogs are host_configs.

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

2 files changed, 15 insertions, 21 deletions

diff --git a/spec/unit/configurer.rb b/spec/unit/configurer.rb
index 2bdb63d53..377ac74b4 100755
--- a/spec/unit/configurer.rb
+++ b/spec/unit/configurer.rb
@@ -147,28 +147,25 @@ describe Puppet::Configurer, "when executing a catalog run" do
     end
 
     it "should apply the catalog with all options to :run" do
-        catalog = stub 'catalog', :retrieval_duration= => nil
-        @agent.expects(:retrieve_catalog).returns catalog
+        @agent.expects(:retrieve_catalog).returns @catalog
 
-        catalog.expects(:apply).with { |args| args[:one] == true }
+        @catalog.expects(:apply).with { |args| args[:one] == true }
         @agent.run :one => true
     end
 
     it "should accept a catalog and use it instead of retrieving a different one" do
-        catalog = stub 'catalog', :retrieval_duration= => nil
         @agent.expects(:retrieve_catalog).never
 
-        catalog.expects(:apply)
-        @agent.run :one => true, :catalog => catalog
+        @catalog.expects(:apply)
+        @agent.run :one => true, :catalog => @catalog
     end
 
     it "should benchmark how long it takes to apply the catalog" do
         @agent.expects(:benchmark).with(:notice, "Finished catalog run")
 
-        catalog = stub 'catalog', :retrieval_duration= => nil
-        @agent.expects(:retrieve_catalog).returns catalog
+        @agent.expects(:retrieve_catalog).returns @catalog
 
-        catalog.expects(:apply).never # because we're not yielding
+        @catalog.expects(:apply).never # because we're not yielding
         @agent.run
     end
 
@@ -190,14 +187,12 @@ describe Puppet::Configurer, "when executing a catalog run" do
         report = stub 'report'
         @agent.expects(:initialize_report).returns report
 
-        catalog = stub 'catalog', :retrieval_duration= => nil
-
         trans = stub 'transaction'
-        catalog.expects(:apply).returns trans
+        @catalog.expects(:apply).returns trans
 
         @agent.expects(:send_report).with { |r, t| t == trans }
 
-        @agent.run :catalog => catalog
+        @agent.run :catalog => @catalog
     end
 
     it "should send the transaction report even if the catalog could not be retrieved" do
@@ -461,12 +456,6 @@ describe Puppet::Configurer, "when converting the catalog" do
 
         @agent.convert_catalog(@oldcatalog, 10)
     end
-
-    it "should mark the RAL catalog as a host catalog" do
-        @catalog.expects(:host_config=).with true
-
-        @agent.convert_catalog(@oldcatalog, 10)
-    end
 end
 
 describe Puppet::Configurer, "when preparing for a run" do
diff --git a/spec/unit/resource/catalog.rb b/spec/unit/resource/catalog.rb
index bd241fd17..e633b131c 100755
--- a/spec/unit/resource/catalog.rb
+++ b/spec/unit/resource/catalog.rb
@@ -628,8 +628,13 @@ describe Puppet::Resource::Catalog, "when compiling" do
             end
         end
 
-        it "should default to not being a host catalog" do
-            @catalog.host_config.should be_nil
+        it "should default to being a host catalog" do
+            @catalog.host_config.should be_true
+        end
+
+        it "should be able to be set to a non-host_config" do
+            @catalog.host_config = false
+            @catalog.host_config.should be_false
         end
 
         it "should pass supplied tags on to the transaction" do