From 6f7ccff8bb764dffd1d41d5391dd79f7bd4a387c Mon Sep 17 00:00:00 2001
From: Luke Kanies <luke@madstop.com>
Date: Tue, 28 Oct 2008 17:50:20 -0500
Subject: Fixing #1641 - file recursion now only passes original parameters to
 child resources.

Previously, parameter values provided by remote sources or default values
were all passed to children, which provided strange state maintenance.

Signed-off-by: Luke Kanies <luke@madstop.com>
---
 spec/unit/type/file.rb | 45 +++++++++++++++++++++++++--------------------
 1 file changed, 25 insertions(+), 20 deletions(-)

(limited to 'spec')

diff --git a/spec/unit/type/file.rb b/spec/unit/type/file.rb
index a476d3cd5..533f19556 100755
--- a/spec/unit/type/file.rb
+++ b/spec/unit/type/file.rb
@@ -564,38 +564,43 @@ describe Puppet::Type.type(:file) do
 
         describe "and making a new child resource" do
             it "should create an implicit resource using the provided relative path joined with the file's path" do
-                path = File.join(@file[:path], "my/path")
-                Puppet::Type.type(:file).expects(:create).with { |options| options[:implicit] == true and options[:path] == path }
-                @file.newchild("my/path")
-            end
-
-            it "should copy most of the parent resource's 'should' values to the new resource" do
-                @file.expects(:to_hash).returns :foo => "bar", :fee => "fum"
-                Puppet::Type.type(:file).expects(:create).with { |options| options[:foo] == "bar" and options[:fee] == "fum" }
-                @file.newchild("my/path")
+                @file.newchild("my/path").should be_implicit
             end
 
             it "should not copy the parent resource's parent" do
-                @file.expects(:to_hash).returns :parent => "foo"
                 Puppet::Type.type(:file).expects(:create).with { |options| ! options.include?(:parent) }
                 @file.newchild("my/path")
             end
 
-            it "should not copy the parent resource's recurse value" do
-                @file.expects(:to_hash).returns :recurse => true
-                Puppet::Type.type(:file).expects(:create).with { |options| ! options.include?(:recurse) }
-                @file.newchild("my/path")
+            {:recurse => true, :target => "/foo/bar", :ensure => :present}.each do |param, value|
+                it "should not pass on #{param} to the sub resource" do
+                    @file[param] = value
+
+                    @file.class.expects(:create).with { |params| params[param].nil? }
+
+                    @file.newchild("sub/file")
+                end
+            end
+
+            it "should copy all of the parent resource's 'should' values that were set at initialization" do
+                file = @file.class.create(:path => "/foo/bar", :owner => "root", :group => "wheel")
+                file.class.expects(:create).with { |options| options[:owner] == "root" and options[:group] == "wheel" }
+                file.newchild("my/path")
             end
 
-            it "should not copy the parent resource's target value" do
-                @file.expects(:to_hash).returns :target => "foo"
-                Puppet::Type.type(:file).expects(:create).with { |options| ! options.include?(:target) }
+            it "should not copy default values to the new child" do
+                @file.class.expects(:create).with { |params| params[:backup].nil? }
                 @file.newchild("my/path")
             end
 
-            it "should not copy any nil values from the parent" do
-                @file.expects(:to_hash).returns :ensure => nil
-                Puppet::Type.type(:file).expects(:create).with { |options| ! options.include?(:ensure) }
+            it "should not copy values to the child which were set by the source" do
+                @file[:source] = "/foo/bar"
+                metadata = stub 'metadata', :owner => "root", :group => "root", :mode => 0755, :ftype => "file", :checksum => "{md5}whatever"
+                @file.property(:source).stubs(:metadata).returns metadata
+
+                @file.property(:source).copy_source_values
+
+                @file.class.expects(:create).with { |params| params[:group].nil? }
                 @file.newchild("my/path")
             end
         end
-- 
cgit