puppet 0.25.5 → 2.6.0

data/spec/unit/daemon.rb

@@ -1,287 +0,0 @@
-#!/usr/bin/env ruby"
-
-require File.dirname(__FILE__) + '/../spec_helper'
-require 'puppet/daemon'
-
-describe Puppet::Daemon do
-    before do
-        @daemon = Puppet::Daemon.new
-    end
-
-    it "should be able to manage an agent" do
-        @daemon.should respond_to(:agent)
-    end
-
-    it "should be able to manage a network server" do
-        @daemon.should respond_to(:server)
-    end
-
-    it "should reopen the Log logs when told to reopen logs" do
-        Puppet::Util::Log.expects(:reopen)
-        @daemon.reopen_logs
-    end
-
-    describe "when setting signal traps" do
-        before do
-            @daemon.stubs(:trap)
-        end
-
-        {:INT => :stop, :TERM => :stop, :HUP => :restart, :USR1 => :reload, :USR2 => :reopen_logs}.each do |signal, method|
-            it "should log and call #{method} when it receives #{signal}" do
-                @daemon.expects(:trap).with(signal).yields
-
-                Puppet.expects(:notice)
-
-                @daemon.expects(method)
-
-                @daemon.set_signal_traps
-            end
-        end
-    end
-
-    describe "when starting" do
-        before do
-            @daemon.stubs(:create_pidfile)
-            @daemon.stubs(:set_signal_traps)
-            EventLoop.current.stubs(:run)
-        end
-
-        it "should fail if it has neither agent nor server" do
-            lambda { @daemon.start }.should raise_error(Puppet::DevError)
-        end
-
-        it "should create its pidfile" do
-            @daemon.stubs(:agent).returns stub('agent', :start => nil)
-
-            @daemon.expects(:create_pidfile)
-            @daemon.start
-        end
-
-        it "should start the agent if the agent is configured" do
-            agent = mock 'agent'
-            agent.expects(:start)
-            @daemon.stubs(:agent).returns agent
-
-            @daemon.start
-        end
-
-        it "should start its server if one is configured" do
-            server = mock 'server'
-            server.expects(:start)
-            @daemon.stubs(:server).returns server
-
-            @daemon.start
-        end
-
-        it "should let the current EventLoop run" do
-            @daemon.stubs(:agent).returns stub('agent', :start => nil)
-            EventLoop.current.expects(:run)
-
-            @daemon.start
-        end
-    end
-
-    describe "when stopping" do
-        before do
-            @daemon.stubs(:remove_pidfile)
-            @daemon.stubs(:exit)
-            Puppet::Util::Log.stubs(:close_all)
-        end
-
-        it "should stop its server if one is configured" do
-            server = mock 'server'
-            server.expects(:stop)
-            @daemon.stubs(:server).returns server
-
-            @daemon.stop
-        end
-
-        it "should stop its agent if one is configured" do
-            agent = mock 'agent'
-            agent.expects(:stop)
-            @daemon.stubs(:agent).returns agent
-
-            @daemon.stop
-        end
-
-        it "should remove its pidfile" do
-            @daemon.expects(:remove_pidfile)
-
-            @daemon.stop
-        end
-
-        it "should close all logs" do
-            Puppet::Util::Log.expects(:close_all)
-
-            @daemon.stop
-        end
-
-        it "should exit unless called with ':exit => false'" do
-            @daemon.expects(:exit)
-            @daemon.stop
-        end
-
-        it "should not exit if called with ':exit => false'" do
-            @daemon.expects(:exit).never
-            @daemon.stop :exit => false
-        end
-    end
-
-    describe "when creating its pidfile" do
-        it "should use an exclusive mutex" do
-            Puppet.settings.expects(:value).with(:name).returns "me"
-
-            sync = mock 'sync'
-            Puppet::Util.expects(:sync).with("me").returns sync
-
-            sync.expects(:synchronize).with(Sync::EX)
-            @daemon.create_pidfile
-        end
-
-        it "should lock the pidfile using the Pidlock class" do
-            pidfile = mock 'pidfile'
-
-            Puppet.settings.stubs(:value).with(:name).returns "eh"
-            Puppet.settings.expects(:value).with(:pidfile).returns "/my/file"
-
-            Puppet::Util::Pidlock.expects(:new).with("/my/file").returns pidfile
-
-            pidfile.expects(:lock).returns true
-            @daemon.create_pidfile
-        end
-
-        it "should fail if it cannot lock" do
-            pidfile = mock 'pidfile'
-
-            Puppet.settings.stubs(:value).with(:name).returns "eh"
-            Puppet.settings.stubs(:value).with(:pidfile).returns "/my/file"
-
-            Puppet::Util::Pidlock.expects(:new).with("/my/file").returns pidfile
-
-            pidfile.expects(:lock).returns false
-
-            lambda { @daemon.create_pidfile }.should raise_error
-        end
-    end
-
-    describe "when removing its pidfile" do
-        it "should use an exclusive mutex" do
-            Puppet.settings.expects(:value).with(:name).returns "me"
-
-            sync = mock 'sync'
-            Puppet::Util.expects(:sync).with("me").returns sync
-
-            sync.expects(:synchronize).with(Sync::EX)
-            @daemon.remove_pidfile
-        end
-
-        it "should do nothing if the pidfile is not present" do
-            pidfile = mock 'pidfile', :locked? => false
-            Puppet::Util::Pidlock.expects(:new).with("/my/file").returns pidfile
-
-            Puppet.settings.stubs(:value).with(:name).returns "eh"
-            Puppet.settings.stubs(:value).with(:pidfile).returns "/my/file"
-
-            pidfile.expects(:unlock).never
-            @daemon.remove_pidfile
-        end
-
-        it "should unlock the pidfile using the Pidlock class" do
-            pidfile = mock 'pidfile', :locked? => true
-            Puppet::Util::Pidlock.expects(:new).with("/my/file").returns pidfile
-            pidfile.expects(:unlock).returns true
-
-            Puppet.settings.stubs(:value).with(:name).returns "eh"
-            Puppet.settings.stubs(:value).with(:pidfile).returns "/my/file"
-
-            @daemon.remove_pidfile
-        end
-
-        it "should warn if it cannot remove the pidfile" do
-            pidfile = mock 'pidfile', :locked? => true
-            Puppet::Util::Pidlock.expects(:new).with("/my/file").returns pidfile
-            pidfile.expects(:unlock).returns false
-
-            Puppet.settings.stubs(:value).with(:name).returns "eh"
-            Puppet.settings.stubs(:value).with(:pidfile).returns "/my/file"
-
-            Puppet.expects :err
-            @daemon.remove_pidfile
-        end
-    end
-
-    describe "when reloading" do
-        it "should do nothing if no agent is configured" do
-            @daemon.reload
-        end
-
-        it "should do nothing if the agent is running" do
-            agent = mock 'agent'
-            agent.expects(:running?).returns true
-
-            @daemon.stubs(:agent).returns agent
-
-            @daemon.reload
-        end
-
-        it "should run the agent if one is available and it is not running" do
-            agent = mock 'agent'
-            agent.expects(:running?).returns false
-            agent.expects :run
-
-            @daemon.stubs(:agent).returns agent
-
-            @daemon.reload
-        end
-    end
-
-    describe "when restarting" do
-        it "should reexec itself if no agent is available" do
-            @daemon.expects(:reexec)
-
-            @daemon.restart
-        end
-
-        it "should reexec itself if the agent is not running" do
-            agent = mock 'agent'
-            agent.expects(:running?).returns false
-            @daemon.stubs(:agent).returns agent
-            @daemon.expects(:reexec)
-
-            @daemon.restart
-        end
-
-        it "should configure the agent for later restart if the agent is running" do
-            agent = mock 'agent'
-            agent.expects(:running?).returns true
-            @daemon.stubs(:agent).returns agent
-            @daemon.expects(:reexec).never
-
-            agent.expects(:configure_delayed_restart)
-
-            @daemon.restart
-        end
-    end
-
-    describe "when reexecing it self" do
-        it "should fail if no argv values are available" do
-            @daemon.expects(:argv).returns nil
-            lambda { @daemon.reexec }.should raise_error(Puppet::DevError)
-        end
-
-        it "should shut down without exiting" do
-            @daemon.argv = %w{foo}
-            @daemon.expects(:stop).with(:exit => false)
-
-            @daemon.stubs(:exec)
-            @daemon.reexec
-        end
-
-        it "should call 'exec' with the original executable and arguments" do
-            @daemon.argv = %w{foo}
-            @daemon.expects(:exec).with($0 + " " + "foo")
-
-            @daemon.reexec
-        end
-    end
-end

data/spec/unit/file_collection.rb

@@ -1,53 +0,0 @@
-#!/usr/bin/env ruby
-
-require File.dirname(__FILE__) + '/../spec_helper'
-
-require 'puppet/file_collection'
-
-describe Puppet::FileCollection do
-    before do
-        @collection = Puppet::FileCollection.new
-    end
-
-    it "should be able to convert a file name into an index" do
-        @collection.index("/my/file").should be_instance_of(Fixnum)
-    end
-
-    it "should be able to convert an index into a file name" do
-        index = @collection.index("/path/to/file")
-        @collection.path(index).should == "/path/to/file"
-    end
-
-    it "should always give the same file name for a given index" do
-        index = @collection.index("/path/to/file")
-        @collection.path(index).should == @collection.path(index)
-    end
-
-    it "should always give the same index for a given file name" do
-        @collection.index("/my/file").should == @collection.index("/my/file")
-    end
-
-    it "should always correctly relate a file name and its index even when multiple files are in the collection" do
-        indexes = %w{a b c d e f}.inject({}) do |hash, letter|
-            hash[letter] = @collection.index("/path/to/file/%s" % letter)
-            hash
-        end
-
-        indexes.each do |letter, index|
-            @collection.index("/path/to/file/%s" % letter).should == indexes[letter]
-            @collection.path(index).should == @collection.path(index)
-        end
-    end
-
-    it "should return nil as the file name when an unknown index is provided" do
-        @collection.path(50).should be_nil
-    end
-
-    it "should provide a global collection" do
-        Puppet::FileCollection.collection.should be_instance_of(Puppet::FileCollection)
-    end
-
-    it "should reuse the global collection" do
-        Puppet::FileCollection.collection.should equal(Puppet::FileCollection.collection)
-    end
-end

data/spec/unit/file_collection/lookup.rb

@@ -1,46 +0,0 @@
-#!/usr/bin/env ruby
-
-require File.dirname(__FILE__) + '/../../spec_helper'
-require 'puppet/file_collection/lookup'
-
-class LookupTester
-    include Puppet::FileCollection::Lookup
-end
-
-describe Puppet::FileCollection::Lookup do
-    before do
-        @tester = LookupTester.new
-
-        @file_collection = mock 'file_collection'
-        Puppet::FileCollection.stubs(:collection).returns @file_collection
-    end
-
-    it "should use the file collection to determine the index of the file name" do
-        @file_collection.expects(:index).with("/my/file").returns 50
-
-        @tester.file = "/my/file"
-        @tester.file_index.should == 50
-    end
-
-    it "should return nil as the file name if no index is set" do
-        @tester.file.should be_nil
-    end
-
-    it "should use the file collection to convert the index to a file name" do
-        @file_collection.expects(:path).with(25).returns "/path/to/file"
-
-        @tester.file_index = 25
-
-        @tester.file.should == "/path/to/file"
-    end
-
-    it "should support a line attribute" do
-        @tester.line = 50
-        @tester.line.should == 50
-    end
-
-    it "should default to the global file collection" do
-        Puppet::FileCollection.expects(:collection).returns "collection"
-        @tester.file_collection.should == "collection"
-    end
-end

data/spec/unit/file_serving/base.rb

@@ -1,132 +0,0 @@
-#!/usr/bin/env ruby
-
-require File.dirname(__FILE__) + '/../../spec_helper'
-
-require 'puppet/file_serving/base'
-
-describe Puppet::FileServing::Base do
-    it "should accept a path" do
-        Puppet::FileServing::Base.new("/module/dir/file").path.should == "/module/dir/file"
-    end
-
-    it "should require that paths be fully qualified" do
-        lambda { Puppet::FileServing::Base.new("module/dir/file") }.should raise_error(ArgumentError)
-    end
-
-    it "should allow specification of whether links should be managed" do
-        Puppet::FileServing::Base.new("/module/dir/file", :links => :manage).links.should == :manage
-    end
-
-    it "should have a :source attribute" do
-        file = Puppet::FileServing::Base.new("/module/dir/file")
-        file.should respond_to(:source)
-        file.should respond_to(:source=)
-    end
-
-    it "should consider :ignore links equivalent to :manage links" do
-        Puppet::FileServing::Base.new("/module/dir/file", :links => :ignore).links.should == :manage
-    end
-
-    it "should fail if :links is set to anything other than :manage, :follow, or :ignore" do
-        proc { Puppet::FileServing::Base.new("/module/dir/file", :links => :else) }.should raise_error(ArgumentError)
-    end
-
-    it "should allow links values to be set as strings" do
-        Puppet::FileServing::Base.new("/module/dir/file", :links => "follow").links.should == :follow
-    end
-
-    it "should default to :manage for :links" do
-        Puppet::FileServing::Base.new("/module/dir/file").links.should == :manage
-    end
-
-    it "should allow specification of a path" do
-        FileTest.stubs(:exists?).returns(true)
-        Puppet::FileServing::Base.new("/module/dir/file", :path => "/my/file").path.should == "/my/file"
-    end
-
-    it "should allow specification of a relative path" do
-        FileTest.stubs(:exists?).returns(true)
-        Puppet::FileServing::Base.new("/module/dir/file", :relative_path => "my/file").relative_path.should == "my/file"
-    end
-
-    it "should have a means of determining if the file exists" do
-        Puppet::FileServing::Base.new("/blah").should respond_to(:exist?)
-    end
-
-    it "should correctly indicate if the file is present" do
-        File.expects(:lstat).with("/my/file").returns(mock("stat"))
-        Puppet::FileServing::Base.new("/my/file").exist?.should be_true
-    end
-
-    it "should correctly indicate if the file is absent" do
-        File.expects(:lstat).with("/my/file").raises RuntimeError
-        Puppet::FileServing::Base.new("/my/file").exist?.should be_false
-    end
-
-    describe "when setting the relative path" do
-        it "should require that the relative path be unqualified" do
-            @file = Puppet::FileServing::Base.new("/module/dir/file")
-            FileTest.stubs(:exists?).returns(true)
-            proc { @file.relative_path = "/qualified/file" }.should raise_error(ArgumentError)
-        end
-    end
-
-    describe "when determining the full file path" do
-        before do
-            @file = Puppet::FileServing::Base.new("/this/file")
-        end
-
-        it "should return the path if there is no relative path" do
-            @file.full_path.should == "/this/file"
-        end
-
-        it "should return the path if the relative_path is set to ''" do
-            @file.relative_path = ""
-            @file.full_path.should == "/this/file"
-        end
-
-        it "should return the path if the relative_path is set to '.'" do
-            @file.relative_path = "."
-            @file.full_path.should == "/this/file"
-        end
-
-        it "should return the path joined with the relative path if there is a relative path and it is not set to '/' or ''" do
-            @file.relative_path = "not/qualified"
-            @file.full_path.should == "/this/file/not/qualified"
-        end
-
-        it "should strip extra slashes" do
-            file = Puppet::FileServing::Base.new("//this//file")
-            file.full_path.should == "/this/file"
-        end
-    end
-
-    describe "when stat'ing files" do
-        before do
-            @file = Puppet::FileServing::Base.new("/this/file")
-        end
-
-        it "should stat the file's full path" do
-            @file.stubs(:full_path).returns("/this/file")
-            File.expects(:lstat).with("/this/file").returns stub("stat", :ftype => "file")
-            @file.stat
-        end
-
-        it "should fail if the file does not exist" do
-            @file.stubs(:full_path).returns("/this/file")
-            File.expects(:lstat).with("/this/file").raises(Errno::ENOENT)
-            proc { @file.stat }.should raise_error(Errno::ENOENT)
-        end
-
-        it "should use :lstat if :links is set to :manage" do
-            File.expects(:lstat).with("/this/file").returns stub("stat", :ftype => "file")
-            @file.stat
-        end
-
-        it "should use :stat if :links is set to :follow" do
-            File.expects(:stat).with("/this/file").returns stub("stat", :ftype => "file")
-            @file.links = :follow
-            @file.stat
-        end
-    end
-end