puppet 0.25.5 → 2.6.0

data/spec/unit/node_spec.rb

@@ -0,0 +1,210 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Puppet::Node do
+  describe "when managing its environment" do
+    it "should use any set environment" do
+      Puppet::Node.new("foo", :environment => "bar").environment.name.should == :bar
+    end
+
+    it "should support providing an actual environment instance" do
+      Puppet::Node.new("foo", :environment => Puppet::Node::Environment.new(:bar)).environment.name.should == :bar
+    end
+
+    it "should determine its environment from its parameters if no environment is set" do
+      Puppet::Node.new("foo", :parameters => {"environment" => :bar}).environment.name.should == :bar
+    end
+
+    it "should use the default environment if no environment is provided" do
+      Puppet::Node.new("foo").environment.name.should == Puppet::Node::Environment.new.name
+    end
+
+    it "should always return an environment instance rather than a string" do
+      Puppet::Node.new("foo").environment.should be_instance_of(Puppet::Node::Environment)
+    end
+
+    it "should allow the environment to be set after initialization" do
+      node = Puppet::Node.new("foo")
+      node.environment = :bar
+      node.environment.name.should == :bar
+    end
+
+    it "should allow its environment to be set by parameters after initialization" do
+      node = Puppet::Node.new("foo")
+      node.parameters["environment"] = :bar
+      node.environment.name.should == :bar
+    end
+  end
+end
+
+describe Puppet::Node, "when initializing" do
+  before do
+    @node = Puppet::Node.new("testnode")
+  end
+
+  it "should set the node name" do
+    @node.name.should == "testnode"
+  end
+
+  it "should not allow nil node names" do
+    proc { Puppet::Node.new(nil) }.should raise_error(ArgumentError)
+  end
+
+  it "should default to an empty parameter hash" do
+    @node.parameters.should == {}
+  end
+
+  it "should default to an empty class array" do
+    @node.classes.should == []
+  end
+
+  it "should note its creation time" do
+    @node.time.should be_instance_of(Time)
+  end
+
+  it "should accept parameters passed in during initialization" do
+    params = {"a" => "b"}
+    @node = Puppet::Node.new("testing", :parameters => params)
+    @node.parameters.should == params
+  end
+
+  it "should accept classes passed in during initialization" do
+    classes = %w{one two}
+    @node = Puppet::Node.new("testing", :classes => classes)
+    @node.classes.should == classes
+  end
+
+  it "should always return classes as an array" do
+    @node = Puppet::Node.new("testing", :classes => "myclass")
+    @node.classes.should == ["myclass"]
+  end
+end
+
+describe Puppet::Node, "when merging facts" do
+  before do
+    @node = Puppet::Node.new("testnode")
+    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 = Puppet::Node.new("testnode", :parameters => {"one" => "a"})
+    @node.fact_merge
+    @node.parameters["one"].should == "a"
+  end
+
+  it "should add passed parameters to the parameter list" do
+    @node = Puppet::Node.new("testnode", :parameters => {"one" => "a"})
+    @node.fact_merge
+    @node.parameters["two"].should == "b"
+  end
+
+  it "should accept arbitrary parameters to merge into its parameters" do
+    @node = Puppet::Node.new("testnode", :parameters => {"one" => "a"})
+    @node.merge "two" => "three"
+    @node.parameters["two"].should == "three"
+  end
+
+  it "should add the environment to the list of parameters" do
+    Puppet.settings.stubs(:value).with(:environments).returns("one,two")
+    Puppet.settings.stubs(:value).with(:environment).returns("one")
+    @node = Puppet::Node.new("testnode", :environment => "one")
+    @node.merge "two" => "three"
+    @node.parameters["environment"].should == "one"
+  end
+
+  it "should not set the environment if it is already set in the parameters" do
+    Puppet.settings.stubs(:value).with(:environments).returns("one,two")
+    Puppet.settings.stubs(:value).with(:environment).returns("one")
+    @node = Puppet::Node.new("testnode", :environment => "one")
+    @node.merge "environment" => "two"
+    @node.parameters["environment"].should == "two"
+  end
+end
+
+describe Puppet::Node, "when indirecting" do
+  it "should redirect to the indirection" do
+    @indirection = stub 'indirection', :name => :node
+    Puppet::Node.stubs(:indirection).returns(@indirection)
+    @indirection.expects(:find)
+    Puppet::Node.find(:my_node.to_s)
+  end
+
+  it "should default to the 'plain' node terminus" do
+    Puppet::Node.indirection.terminus_class.should == :plain
+  end
+
+  it "should not have a cache class defined" do
+    Puppet::Node.indirection.cache_class.should be_nil
+  end
+
+  after do
+    Puppet::Util::Cacher.expire
+  end
+end
+
+describe Puppet::Node, "when generating the list of names to search through" do
+  before do
+    @node = Puppet::Node.new("foo.domain.com", :parameters => {"hostname" => "yay", "domain" => "domain.com"})
+  end
+
+  it "should return an array of names" do
+    @node.names.should be_instance_of(Array)
+  end
+
+  describe "and the node name is fully qualified" do
+    it "should contain an entry for each part of the node name" do
+      @node.names.should be_include("foo.domain.com")
+      @node.names.should be_include("foo.domain")
+      @node.names.should be_include("foo")
+    end
+  end
+
+  it "should include the node's fqdn" do
+    @node.names.should be_include("yay.domain.com")
+  end
+
+  it "should combine and include the node's hostname and domain if no fqdn is available" do
+    @node.names.should be_include("yay.domain.com")
+  end
+
+  it "should contain an entry for each name available by stripping a segment of the fqdn" do
+    @node.parameters["fqdn"] = "foo.deep.sub.domain.com"
+    @node.names.should be_include("foo.deep.sub.domain")
+    @node.names.should be_include("foo.deep.sub")
+  end
+
+  describe "and :node_name is set to 'cert'" do
+    before do
+      Puppet.settings.stubs(:value).with(:strict_hostname_checking).returns false
+      Puppet.settings.stubs(:value).with(:node_name).returns "cert"
+    end
+
+    it "should use the passed-in key as the first value" do
+      @node.names[0].should == "foo.domain.com"
+    end
+
+    describe "and strict hostname checking is enabled" do
+      it "should only use the passed-in key" do
+        Puppet.settings.expects(:value).with(:strict_hostname_checking).returns true
+        @node.names.should == ["foo.domain.com"]
+      end
+    end
+  end
+
+  describe "and :node_name is set to 'facter'" do
+    before do
+      Puppet.settings.stubs(:value).with(:strict_hostname_checking).returns false
+      Puppet.settings.stubs(:value).with(:node_name).returns "facter"
+    end
+
+    it "should use the node's 'hostname' fact as the first value" do
+      @node.names[0].should == "yay"
+    end
+  end
+end

data/spec/unit/other/selinux_spec.rb

@@ -0,0 +1,94 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+require 'puppet/type/selboolean'
+require 'puppet/type/selmodule'
+
+describe Puppet::Type.type(:file), " when manipulating file contexts" do
+  before :each do
+
+          @file = Puppet::Type::File.new(
+                
+      :name => "/tmp/foo",
+      :ensure => "file",
+      :seluser => "user_u",
+      :selrole => "role_r",
+        
+      :seltype => "type_t" )
+  end
+  it "should use :seluser to get/set an SELinux user file context attribute" do
+    @file.property(:seluser).should == "user_u"
+  end
+  it "should use :selrole to get/set an SELinux role file context attribute" do
+    @file.property(:selrole).should == "role_r"
+  end
+  it "should use :seltype to get/set an SELinux user file context attribute" do
+    @file.property(:seltype).should == "type_t"
+  end
+end
+
+describe Puppet::Type.type(:selboolean), " when manipulating booleans" do
+  before :each do
+    provider_class = Puppet::Type::Selboolean.provider(Puppet::Type::Selboolean.providers[0])
+    Puppet::Type::Selboolean.stubs(:defaultprovider).returns provider_class
+
+
+          @bool = Puppet::Type::Selboolean.new(
+                
+      :name => "foo",
+      :value => "on",
+        
+      :persistent => true )
+  end
+  it "should be able to access :name" do
+    @bool[:name].should == "foo"
+  end
+  it "should be able to access :value" do
+    @bool.property(:value).should == :on
+  end
+  it "should set :value to off" do
+    @bool[:value] = :off
+    @bool.property(:value).should == :off
+  end
+  it "should be able to access :persistent" do
+    @bool[:persistent].should == :true
+  end
+  it "should set :persistent to false" do
+    @bool[:persistent] = false
+    @bool[:persistent].should == :false
+  end
+end
+
+describe Puppet::Type.type(:selmodule), " when checking policy modules" do
+  before :each do
+    provider_class = Puppet::Type::Selmodule.provider(Puppet::Type::Selmodule.providers[0])
+    Puppet::Type::Selmodule.stubs(:defaultprovider).returns provider_class
+
+
+          @module = Puppet::Type::Selmodule.new(
+                
+      :name => "foo",
+      :selmoduledir => "/some/path",
+      :selmodulepath => "/some/path/foo.pp",
+        
+      :syncversion => true)
+  end
+  it "should be able to access :name" do
+    @module[:name].should == "foo"
+  end
+  it "should be able to access :selmoduledir" do
+    @module[:selmoduledir].should == "/some/path"
+  end
+  it "should be able to access :selmodulepath" do
+    @module[:selmodulepath].should == "/some/path/foo.pp"
+  end
+  it "should be able to access :syncversion" do
+    @module.property(:syncversion).should == :true
+  end
+  it "should set the syncversion value to false" do
+    @module[:syncversion] = :false
+    @module.property(:syncversion).should == :false
+  end
+end
+

data/spec/unit/other/transbucket_spec.rb

@@ -0,0 +1,188 @@
+#!/usr/bin/env ruby
+
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe Puppet::TransBucket do
+  before do
+    @bucket = Puppet::TransBucket.new
+  end
+
+  it "should be able to produce a RAL component" do
+    @bucket.name = "luke"
+    @bucket.type = "foo"
+
+    resource = nil
+    proc { resource = @bucket.to_ral }.should_not raise_error
+    resource.should be_instance_of(Puppet::Type::Component)
+    resource.title.should == "Foo[luke]"
+  end
+
+  it "should accept TransObjects into its children list" do
+    object = Puppet::TransObject.new("luke", "user")
+    proc { @bucket.push(object) }.should_not raise_error
+    @bucket.each do |o|
+      o.should equal(object)
+    end
+  end
+
+  it "should accept TransBuckets into its children list" do
+    object = Puppet::TransBucket.new
+    proc { @bucket.push(object) }.should_not raise_error
+    @bucket.each do |o|
+      o.should equal(object)
+    end
+  end
+
+  it "should refuse to accept any children that are not TransObjects or TransBuckets" do
+    proc { @bucket.push "a test" }.should raise_error
+  end
+
+  it "should return use 'node' as the type and the provided name as the title if only a type is provided" do
+    @bucket.type = "mystuff"
+    @bucket.to_ref.should == "Node[mystuff]"
+  end
+
+  it "should return use 'component' as the type and the provided type as the title if only a name is provided" do
+    @bucket.name = "mystuff"
+    @bucket.to_ref.should == "Class[Mystuff]"
+  end
+
+  it "should return nil as its reference when type and name are missing" do
+    @bucket.to_ref.should be_nil
+  end
+
+  it "should return the title as its reference" do
+    @bucket.name = "luke"
+    @bucket.type = "user"
+    @bucket.to_ref.should == "User[luke]"
+  end
+
+  it "should canonize resource references when the type is 'component'" do
+    @bucket.name = 'something'
+    @bucket.type = 'foo::bar'
+
+    @bucket.to_ref.should == "Foo::Bar[something]"
+  end
+end
+
+describe Puppet::TransBucket, " when generating a catalog" do
+  before do
+    @bottom = Puppet::TransBucket.new
+    @bottom.type = "fake"
+    @bottom.name = "bottom"
+    @bottomobj = Puppet::TransObject.new("bottom", "notify")
+    @bottom.push @bottomobj
+
+    @middle = Puppet::TransBucket.new
+    @middle.type = "fake"
+    @middle.name = "middle"
+    @middleobj = Puppet::TransObject.new("middle", "notify")
+    @middle.push(@middleobj)
+    @middle.push(@bottom)
+
+    @top = Puppet::TransBucket.new
+    @top.type = "fake"
+    @top.name = "top"
+    @topobj = Puppet::TransObject.new("top", "notify")
+    @top.push(@topobj)
+    @top.push(@middle)
+
+    @users = %w{top middle bottom}
+    @fakes = %w{Fake[bottom] Fake[middle] Fake[top]}
+  end
+
+  it "should convert all transportable objects to RAL resources" do
+    @catalog = @top.to_catalog
+    @users.each do |name|
+      @catalog.vertices.find { |r| r.class.name == :notify and r.title == name }.should be_instance_of(Puppet::Type.type(:notify))
+    end
+  end
+
+  it "should fail if any transportable resources fail to convert to RAL resources" do
+    @bottomobj.expects(:to_ral).raises ArgumentError
+    lambda { @bottom.to_catalog }.should raise_error(ArgumentError)
+  end
+
+  it "should convert all transportable buckets to RAL components" do
+    @catalog = @top.to_catalog
+    @fakes.each do |name|
+      @catalog.vertices.find { |r| r.class.name == :component and r.title == name }.should be_instance_of(Puppet::Type.type(:component))
+    end
+  end
+
+  it "should add all resources to the graph's resource table" do
+    @catalog = @top.to_catalog
+    @catalog.resource("fake[top]").should equal(@top)
+  end
+
+  it "should finalize all resources" do
+    @catalog = @top.to_catalog
+    @catalog.vertices.each do |vertex| vertex.should be_finalized end
+  end
+
+  it "should only call to_ral on each resource once" do
+    # We just raise exceptions here because we're not interested in
+    # what happens with the result, only that the method only
+    # gets called once.
+    resource = @topobj.to_ral
+    @topobj.expects(:to_ral).once.returns resource
+    @top.to_catalog
+  end
+
+  it "should set each TransObject's catalog before converting to a RAL resource" do
+    @middleobj.expects(:catalog=).with { |c| c.is_a?(Puppet::Resource::Catalog) }
+    @top.to_catalog
+  end
+
+  it "should set each TransBucket's catalog before converting to a RAL resource" do
+    # each bucket is seen twice in the loop, so we have to handle the case where the config
+    # is set twice
+    @bottom.expects(:catalog=).with { |c| c.is_a?(Puppet::Resource::Catalog) }.at_least_once
+    @top.to_catalog
+  end
+end
+
+describe Puppet::TransBucket, " when serializing" do
+  before do
+    @bucket = Puppet::TransBucket.new(%w{one two})
+    @bucket.name = "one"
+    @bucket.type = "two"
+  end
+
+  it "should be able to be dumped to yaml" do
+    proc { YAML.dump(@bucket) }.should_not raise_error
+  end
+
+  it "should dump YAML that produces an equivalent object" do
+    result = YAML.dump(@bucket)
+
+    newobj = YAML.load(result)
+    newobj.name.should == "one"
+    newobj.type.should == "two"
+    children = []
+    newobj.each do |o|
+      children << o
+    end
+    children.should == %w{one two}
+  end
+end
+
+describe Puppet::TransBucket, " when converting to a Puppet::Resource" do
+  before do
+    @trans = Puppet::TransBucket.new
+    @trans.name = "foo"
+    @trans.type = "bar"
+    @trans.param(:noop, true)
+  end
+
+  it "should create a resource with the correct type and title" do
+    result = @trans.to_resource
+    result.type.should == "Bar"
+    result.title.should == "foo"
+  end
+
+  it "should add all of its parameters to the created resource" do
+    @trans.param(:noop, true)
+    @trans.to_resource[:noop].should be_true
+  end
+end