diff options
author | Luke Kanies <luke@puppetlabs.com> | 2010-04-09 16:18:12 -0700 |
---|---|---|
committer | Luke Kanies <luke@puppetlabs.com> | 2010-04-09 16:18:12 -0700 |
commit | 84d68924c9e5538d7d40a9ff5cdd19a3f32874ad (patch) | |
tree | 8d9809bc5a24ddf0ee7eaeb8ad0cd936634ced41 /spec | |
parent | b643413da520fabb7ee81e8c710e30f43bc1475b (diff) | |
download | puppet-84d68924c9e5538d7d40a9ff5cdd19a3f32874ad.tar.gz puppet-84d68924c9e5538d7d40a9ff5cdd19a3f32874ad.tar.xz puppet-84d68924c9e5538d7d40a9ff5cdd19a3f32874ad.zip |
Fixing all 'require' function tests
This involved making some tests better, but mostly
just involved fixing calls to use new APIs and such.
Signed-off-by: Luke Kanies <luke@puppetlabs.com>
Diffstat (limited to 'spec')
-rwxr-xr-x | spec/integration/parser/functions/require.rb | 8 | ||||
-rwxr-xr-x | spec/unit/parser/functions/require.rb | 19 |
2 files changed, 16 insertions, 11 deletions
diff --git a/spec/integration/parser/functions/require.rb b/spec/integration/parser/functions/require.rb index fa1c1bcbb..3d65a50f0 100755 --- a/spec/integration/parser/functions/require.rb +++ b/spec/integration/parser/functions/require.rb @@ -26,8 +26,8 @@ describe "The require function" do end it "should queue relationships between the 'required' class and our classes" do - @parser.newclass("requiredclass1") - @parser.newclass("requiredclass2") + @compiler.known_resource_types.add Puppet::Resource::Type.new(:hostclass, "requiredclass1") + @compiler.known_resource_types.add Puppet::Resource::Type.new(:hostclass, "requiredclass2") @scope.function_require("requiredclass1") @scope.function_require("requiredclass2") @@ -36,9 +36,9 @@ describe "The require function" do (ref1,ref2) = @scope.resource["require"] ref1.type.should == "Class" - ref1.title.should == "requiredclass1" + ref1.title.should == "Requiredclass1" ref2.type.should == "Class" - ref2.title.should == "requiredclass2" + ref2.title.should == "Requiredclass2" end end diff --git a/spec/unit/parser/functions/require.rb b/spec/unit/parser/functions/require.rb index 1d9ce931c..45769ccbe 100755 --- a/spec/unit/parser/functions/require.rb +++ b/spec/unit/parser/functions/require.rb @@ -8,13 +8,15 @@ describe "the require function" do @catalog = stub 'catalog' @compiler = stub 'compiler', :catalog => @catalog - @resource = stub 'resource', :set_parameter => nil, :metaparam_compatibility_mode? => false, :[] => nil @scope = Puppet::Parser::Scope.new() - @scope.stubs(:resource).returns @resource @scope.stubs(:findresource) @scope.stubs(:compiler).returns(@compiler) @klass = stub 'class', :name => "myclass" @scope.stubs(:find_hostclass).returns(@klass) + + @resource = Puppet::Parser::Resource.new(:file, "/my/file", :scope => @scope, :source => "source") + @resource.stubs(:metaparam_compatibility_mode?).returns false + @scope.stubs(:resource).returns @resource end it "should exist" do @@ -28,9 +30,11 @@ describe "the require function" do end it "should set the 'require' prarameter on the resource to a resource reference" do - @resource.expects(:set_parameter).with { |name, value| name == :require and value[0].is_a?(Puppet::Resource) } @scope.stubs(:function_include) @scope.function_require("myclass") + + @resource["require"].should be_instance_of(Array) + @resource["require"][0].should be_instance_of(Puppet::Resource) end it "should verify the 'include' function is loaded" do @@ -59,11 +63,12 @@ describe "the require function" do it "should append the required class to the require parameter" do @scope.stubs(:function_include) - Puppet::Parser::Resource::Reference.stubs(:new).returns(:require2) - - @resource.expects(:[]).with(:require).returns(:require1) - @resource.expects(:set_parameter).with(:require, [:require1, :require2]) + one = Puppet::Resource.new(:file, "/one") + @resource[:require] = one @scope.function_require("myclass") + + @resource[:require].should be_include(one) + @resource[:require].detect { |r| r.to_s == "Class[Myclass]" }.should be_instance_of(Puppet::Resource) end end |