diff options
Diffstat (limited to 'spec/unit')
| -rw-r--r-- | spec/unit/parser/loaded_code.rb | 61 | ||||
| -rwxr-xr-x | spec/unit/parser/resource_type.rb | 6 | ||||
| -rwxr-xr-x | spec/unit/util/rdoc/parser.rb | 4 |
3 files changed, 40 insertions, 31 deletions
diff --git a/spec/unit/parser/loaded_code.rb b/spec/unit/parser/loaded_code.rb index 9b72dab0d..3a230f988 100644 --- a/spec/unit/parser/loaded_code.rb +++ b/spec/unit/parser/loaded_code.rb @@ -8,22 +8,31 @@ require 'puppet/parser/resource_type' describe Puppet::Parser::LoadedCode do before do @instance = Puppet::Parser::ResourceType.new(:hostclass, "foo") - @code = Puppet::Parser::LoadedCode.new + @code = Puppet::Parser::LoadedCode.new("env") + end + + it "should require an environment at initialization" do + Puppet::Parser::LoadedCode.new("foo").environment.should == "foo" + end + + it "should store itself as the environment-specific code collection in its class" do + code = Puppet::Parser::LoadedCode.new("foo") + Puppet::Parser::LoadedCode["foo"].should equal(code) end it "should be able to add a resource type" do - Puppet::Parser::LoadedCode.new.should respond_to(:add) + Puppet::Parser::LoadedCode.new("env").should respond_to(:add) end it "should consider '<<' to be an alias to 'add' but should return self" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.expects(:add).with "foo" loader.expects(:add).with "bar" loader << "foo" << "bar" end it "should set itself as the code collection for added resource types" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") node = Puppet::Parser::ResourceType.new(:node, "foo") @@ -56,48 +65,48 @@ describe Puppet::Parser::LoadedCode do %w{hostclass node definition}.each do |data| it "should have a method for adding a #{data}" do - Puppet::Parser::LoadedCode.new.should respond_to("add_" + data) + Puppet::Parser::LoadedCode.new("env").should respond_to("add_" + data) end it "should use the name of the instance to add it" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.send("add_#{data}", @instance) loader.send(data, @instance.name).should equal(@instance) end it "should fail to add a #{data} when one already exists" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.add @instance lambda { loader.add(@instance) }.should raise_error(Puppet::ParseError) end it "should return the added #{data}" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.add(@instance).should equal(@instance) end it "should be able to retrieve #{data} by name" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(data, "bar") loader.add instance loader.send(data, "bar").should equal(instance) end it "should retrieve #{data} insensitive to case" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(data, "Bar") loader.add instance loader.send(data, "bAr").should equal(instance) end it "should return nil when asked for a #{data} that has not been added" do - Puppet::Parser::LoadedCode.new.send(data, "foo").should be_nil + Puppet::Parser::LoadedCode.new("env").send(data, "foo").should be_nil end it "should be able to retrieve all #{data}s" do plurals = { "hostclass" => "hostclasses", "node" => "nodes", "definition" => "definitions" } - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(data, "foo") loader.add instance loader.send(plurals[data]).should == { "foo" => instance } @@ -106,54 +115,54 @@ describe Puppet::Parser::LoadedCode do describe "when finding a qualified instance" do it "should return any found instance if the instance name is fully qualified" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar") loader.add instance loader.find("namespace", "::foo::bar", :hostclass).should equal(instance) end it "should return nil if the instance name is fully qualified and no such instance exists" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.find("namespace", "::foo::bar", :hostclass).should be_nil end it "should return the partially qualified object if it exists in the provided namespace" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar::baz") loader.add instance loader.find("foo", "bar::baz", :hostclass).should equal(instance) end it "should return the unqualified object if it exists in the provided namespace" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar") loader.add instance loader.find("foo", "bar", :hostclass).should equal(instance) end it "should return the unqualified object if it exists in the parent namespace" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar") loader.add instance loader.find("foo::bar::baz", "bar", :hostclass).should equal(instance) end it "should should return the partially qualified object if it exists in the parent namespace" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar::baz") loader.add instance loader.find("foo::bar", "bar::baz", :hostclass).should equal(instance) end it "should return the qualified object if it exists in the root namespace" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar::baz") loader.add instance loader.find("foo::bar", "foo::bar::baz", :hostclass).should equal(instance) end it "should return nil if the object cannot be found" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") instance = Puppet::Parser::ResourceType.new(:hostclass, "foo::bar::baz") loader.add instance loader.find("foo::bar", "eh", :hostclass).should be_nil @@ -161,36 +170,36 @@ describe Puppet::Parser::LoadedCode do end it "should use the generic 'find' method with an empty namespace to find nodes" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.expects(:find).with("", "bar", :node) loader.find_node("bar") end it "should use the generic 'find' method to find hostclasses" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.expects(:find).with("foo", "bar", :hostclass) loader.find_hostclass("foo", "bar") end it "should use the generic 'find' method to find definitions" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.expects(:find).with("foo", "bar", :definition) loader.find_definition("foo", "bar") end it "should indicate whether any nodes are defined" do - loader = Puppet::Parser::LoadedCode.new + loader = Puppet::Parser::LoadedCode.new("env") loader.add_node(Puppet::Parser::ResourceType.new(:node, "foo")) loader.should be_nodes end it "should indicate whether no nodes are defined" do - Puppet::Parser::LoadedCode.new.should_not be_nodes + Puppet::Parser::LoadedCode.new("env").should_not be_nodes end describe "when finding nodes" do before :each do - @loader = Puppet::Parser::LoadedCode.new + @loader = Puppet::Parser::LoadedCode.new("env") end it "should return any node whose name exactly matches the provided node name" do diff --git a/spec/unit/parser/resource_type.rb b/spec/unit/parser/resource_type.rb index 3afab69a5..08f7e2af0 100755 --- a/spec/unit/parser/resource_type.rb +++ b/spec/unit/parser/resource_type.rb @@ -275,7 +275,7 @@ describe Puppet::Parser::ResourceType do describe "when describing and managing parent classes" do before do - @code = Puppet::Parser::LoadedCode.new + @code = Puppet::Parser::ResourceTypeCollection.new("env") @parent = Puppet::Parser::ResourceType.new(:hostclass, "bar") @code.add @parent @@ -369,7 +369,7 @@ describe Puppet::Parser::ResourceType do @top = Puppet::Parser::ResourceType.new :hostclass, "top" @middle = Puppet::Parser::ResourceType.new :hostclass, "middle", :parent => "top" - @code = Puppet::Parser::LoadedCode.new + @code = Puppet::Parser::ResourceTypeCollection.new("env") @code.add @top @code.add @middle end @@ -460,7 +460,7 @@ describe Puppet::Parser::ResourceType do end it "should fail if both classes have different parent classes" do - code = Puppet::Parser::LoadedCode.new + code = Puppet::Parser::ResourceTypeCollection.new("env") {"a" => "b", "c" => "d"}.each do |parent, child| code.add Puppet::Parser::ResourceType.new(:hostclass, parent) code.add Puppet::Parser::ResourceType.new(:hostclass, child, :parent => parent) diff --git a/spec/unit/util/rdoc/parser.rb b/spec/unit/util/rdoc/parser.rb index ce776da9b..c1b55a53d 100755 --- a/spec/unit/util/rdoc/parser.rb +++ b/spec/unit/util/rdoc/parser.rb @@ -141,8 +141,8 @@ describe RDoc::Parser do @definition = stub_everything 'definition', :file => "module/manifests/init.pp", :type => :definition, :name => "mydef" @node = stub_everything 'node', :file => "module/manifests/init.pp", :type => :node, :name => "mynode" - @loadedcode = Puppet::Parser::LoadedCode.new - @parser.ast = @loadedcode + @resource_type_collection = Puppet::Parser::ResourceTypeCollection.new("env") + @parser.ast = @resource_type_collection @container = stub_everything 'container' end |
