puppet 2.7.9 → 2.7.11

data/lib/puppet/application/module.rb

@@ -1,3 +0,0 @@
-require 'puppet/application/face_base'
-
-class Puppet::Application::Module < Puppet::Application::FaceBase; end

data/lib/puppet/face/module.rb

@@ -1,12 +0,0 @@
-require 'puppet/face'
-require 'puppet/module_tool'
-
-Puppet::Face.define(:module, '1.0.0') do
-  copyright "Puppet Labs", 2011
-  license   "Apache 2 license; see COPYING"
-
-  summary "Creates, installs and searches for modules on the Puppet Forge."
-  description <<-EOT
-    Creates, installs and searches for modules on the Puppet Forge.
-  EOT
-end

data/spec/unit/face/module/build_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-
-describe "puppet module build" do
-  subject { Puppet::Face[:module, :current] }
-
-  describe "option validation" do
-    context "without any options" do
-      it "should require a path" do
-        pattern = /wrong number of arguments/
-        expect { subject.build }.to raise_error ArgumentError, pattern
-      end
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :build }
-
-    its(:summary)     { should =~ /build.*module/im }
-    its(:description) { should =~ /build.*module/im }
-    its(:returns)     { should =~ /pathname/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/spec/unit/face/module/changes_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-
-describe "puppet module changes" do
-  subject { Puppet::Face[:module, :current] }
-
-  describe "option validation" do
-    context "without any options" do
-      it "should require a path" do
-        pattern = /wrong number of arguments/
-        expect { subject.changes }.to raise_error ArgumentError, pattern
-      end
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :changes }
-
-    its(:summary)     { should =~ /modified.*module/im }
-    its(:description) { should =~ /modified.*module/im }
-    its(:returns)     { should =~ /array/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/spec/unit/face/module/clean_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-
-describe "puppet module clean" do
-  subject { Puppet::Face[:module, :current] }
-
-  describe "option validation" do
-    context "without any options" do
-      it "should not require any arguments" do
-        Puppet::Module::Tool::Applications::Cleaner.expects(:run).once
-        subject.clean
-      end
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :clean }
-
-    its(:summary)     { should =~ /clean.*module/im }
-    its(:description) { should =~ /clean.*module/im }
-    its(:returns)     { should =~ /hash/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/spec/unit/face/module/generate_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-
-describe "puppet module generate" do
-  subject { Puppet::Face[:module, :current] }
-
-  describe "option validation" do
-    context "without any options" do
-      it "should require name" do
-        pattern = /wrong number of arguments/
-        expect { subject.generate }.to raise_error ArgumentError, pattern
-      end
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :generate }
-
-    its(:summary)     { should =~ /generate.*module/im }
-    its(:description) { should =~ /generate.*module/im }
-    its(:returns)     { should =~ /array/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/spec/unit/face/module/install_spec.rb

@@ -1,75 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-require 'puppet/module_tool'
-
-describe "puppet module install" do
-  subject { Puppet::Face[:module, :current] }
-
-  let(:options) do
-    {}
-  end
-
-  describe "option validation" do
-    let(:expected_options) do
-      {
-        :install_dir => File.expand_path("/dev/null/modules"),
-        :module_repository => "http://forge.puppetlabs.com",
-      }
-    end
-
-    context "without any options" do
-      it "should require a name" do
-        pattern = /wrong number of arguments/
-        expect { subject.install }.to raise_error ArgumentError, pattern
-      end
-
-      it "should not require any options" do
-        Puppet::Module::Tool::Applications::Installer.expects(:run).with("puppetlabs-apache", expected_options).once
-        subject.install("puppetlabs-apache")
-      end
-    end
-
-    it "should accept the --force option" do
-      options[:force] = true
-      expected_options.merge!(options)
-      Puppet::Module::Tool::Applications::Installer.expects(:run).with("puppetlabs-apache", expected_options).once
-      subject.install("puppetlabs-apache", options)
-    end
-
-    it "should accept the --install-dir option" do
-      options[:install_dir] = "/foo/puppet/modules"
-      expected_options.merge!(options)
-      Puppet::Module::Tool::Applications::Installer.expects(:run).with("puppetlabs-apache", expected_options).once
-      subject.install("puppetlabs-apache", options)
-    end
-
-    it "should accept the --module-repository option" do
-      options[:module_repository] = "http://forge.example.com"
-      expected_options.merge!(options)
-      Puppet::Module::Tool::Applications::Installer.expects(:run).with("puppetlabs-apache", expected_options).once
-      subject.install("puppetlabs-apache", options)
-    end
-
-    it "should accept the --version option" do
-      options[:version] = "0.0.1"
-      expected_options.merge!(options)
-      Puppet::Module::Tool::Applications::Installer.expects(:run).with("puppetlabs-apache", expected_options).once
-      subject.install("puppetlabs-apache", options)
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :install }
-
-    its(:summary)     { should =~ /install.*module/im }
-    its(:description) { should =~ /install.*module/im }
-    its(:returns)     { should =~ /pathname/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/spec/unit/face/module/search_spec.rb

@@ -1,40 +0,0 @@
-require 'spec_helper'
-require 'puppet/face'
-
-describe "puppet module search" do
-  subject { Puppet::Face[:module, :current] }
-
-  let(:options) do
-    {}
-  end
-
-  describe "option validation" do
-    context "without any options" do
-      it "should require a search term" do
-        pattern = /wrong number of arguments/
-        expect { subject.search }.to raise_error ArgumentError, pattern
-      end
-    end
-
-    it "should accept the --module-repository option" do
-      options[:module_repository] = "http://forge.example.com"
-      Puppet::Module::Tool::Applications::Searcher.expects(:run).with("puppetlabs-apache", options).once
-      subject.search("puppetlabs-apache", options)
-    end
-  end
-
-  describe "inline documentation" do
-    subject { Puppet::Face[:module, :current].get_action :search }
-
-    its(:summary)     { should =~ /search.*module/im }
-    its(:description) { should =~ /search.*module/im }
-    its(:returns)     { should =~ /array/i }
-    its(:examples)    { should_not be_empty }
-
-    %w{ license copyright summary description returns examples }.each do |doc|
-      context "of the" do
-        its(doc.to_sym) { should_not =~ /(FIXME|REVISIT|TODO)/ }
-      end
-    end
-  end
-end

data/test/util/pidlock.rb

@@ -1,126 +0,0 @@
-require File.expand_path(File.dirname(__FILE__) + '/../lib/puppettest')
-
-require 'puppet/util/pidlock'
-require 'fileutils'
-
-# This is *fucked* *up*
-Puppet.debug = false
-
-class TestPuppetUtilPidlock < Test::Unit::TestCase
-  include PuppetTest
-
-  def setup
-    super
-    @workdir = tstdir
-  end
-
-  def teardown
-    super
-    FileUtils.rm_rf(@workdir)
-  end
-
-  def test_00_basic_create
-    l = nil
-    assert_nothing_raised { l = Puppet::Util::Pidlock.new(@workdir + '/nothingmuch') }
-
-    assert_equal Puppet::Util::Pidlock, l.class
-
-    assert_equal @workdir + '/nothingmuch', l.lockfile
-  end
-
-  def test_10_uncontended_lock
-    l = Puppet::Util::Pidlock.new(@workdir + '/test_lock')
-
-    assert !l.locked?
-    assert !l.mine?
-    assert l.lock
-    assert l.locked?
-    assert l.mine?
-    assert !l.anonymous?
-    # It's OK to call lock multiple times
-    assert l.lock
-    assert l.unlock
-    assert !l.locked?
-    assert !l.mine?
-  end
-
-  def test_20_someone_elses_lock
-    childpid = nil
-    l = Puppet::Util::Pidlock.new(@workdir + '/someone_elses_lock')
-
-    # First, we need a PID that's guaranteed to be (a) used, (b) someone
-    # else's, and (c) around for the life of this test.
-    childpid = fork { loop do; sleep 10; end }
-
-    File.open(l.lockfile, 'w') { |fd| fd.write(childpid) }
-
-    assert l.locked?
-    assert !l.mine?
-    assert !l.lock
-    assert l.locked?
-    assert !l.mine?
-    assert !l.unlock
-    assert l.locked?
-    assert !l.mine?
-  ensure
-    Process.kill("KILL", childpid) unless childpid.nil?
-  end
-
-  def test_30_stale_lock
-    # This is a bit hard to guarantee, but we need a PID that is definitely
-    # unused, and will stay so for the the life of this test.  Our best
-    # bet is to create a process, get it's PID, let it die, and *then*
-    # lock on it.
-    childpid = fork { exit }
-
-    # Now we can't continue until we're sure that the PID is dead
-    Process.wait(childpid)
-
-    l = Puppet::Util::Pidlock.new(@workdir + '/stale_lock')
-
-    # locked? should clear the lockfile
-    File.open(l.lockfile, 'w') { |fd| fd.write(childpid) }
-    assert File.exists?(l.lockfile)
-    assert !l.locked?
-    assert !File.exists?(l.lockfile)
-
-    # lock should replace the lockfile with our own
-    File.open(l.lockfile, 'w') { |fd| fd.write(childpid) }
-    assert File.exists?(l.lockfile)
-    assert l.lock
-    assert l.locked?
-    assert l.mine?
-
-    # unlock should fail, and should *not* molest the existing lockfile,
-    # despite it being stale
-    File.open(l.lockfile, 'w') { |fd| fd.write(childpid) }
-    assert File.exists?(l.lockfile)
-    assert !l.unlock
-    assert File.exists?(l.lockfile)
-  end
-
-  def test_40_not_locked_at_all
-    l = Puppet::Util::Pidlock.new(@workdir + '/not_locked')
-
-    assert !l.locked?
-    # We can't unlock if we don't hold the lock
-    assert !l.unlock
-  end
-
-  def test_50_anonymous_lock
-    l = Puppet::Util::Pidlock.new(@workdir + '/anonymous_lock')
-
-    assert !l.locked?
-    assert l.lock(:anonymous => true)
-    assert l.locked?
-    assert l.anonymous?
-    assert !l.mine?
-    assert "", File.read(l.lockfile)
-    assert !l.unlock
-    assert l.locked?
-    assert l.anonymous?
-    assert l.unlock(:anonymous => true)
-    assert !File.exists?(l.lockfile)
-  end
-end
-