r10k 3.5.2

data/spec/matchers/exit_with.rb

@@ -0,0 +1,28 @@
+RSpec::Matchers.define :exit_with do |expected|
+
+  supports_block_expectations
+
+  match do |block|
+    actual = nil
+    begin
+      block.call
+    rescue SystemExit => e
+      actual = e.status
+    end
+    actual and actual == expected
+  end
+
+  failure_message do |actual|
+    "expected exit with code #{expected} but " +
+      (actual.nil? ? " exit was not called" : "we exited with #{actual} instead")
+  end
+
+  failure_message_when_negated do |actual|
+    "expected that exit would not be called with #{expected}"
+  end
+
+  description do
+    "expect exit with #{expected}"
+  end
+end
+

data/spec/matchers/match_realpath.rb

@@ -0,0 +1,18 @@
+RSpec::Matchers.define :match_realpath do |expected|
+
+  match do |actual|
+    actual == expected || realpath(actual) == realpath(expected)
+  end
+
+  failure_message do |actual|
+    "expected that #{actual} would have a real path of #{expected}"
+  end
+
+  failure_message_when_negated do |actual|
+    "expected that #{actual} would not have a real path of #{expected}"
+  end
+
+  def realpath(path)
+    Pathname.new(path).realpath.to_s
+  end
+end

data/spec/r10k-mocks.rb

@@ -0,0 +1,3 @@
+require 'r10k-mocks/mock_env'
+require 'r10k-mocks/mock_source'
+require 'r10k-mocks/mock_config'

data/spec/r10k-mocks/mock_config.rb

@@ -0,0 +1,33 @@
+require 'r10k/deployment/config'
+
+module R10K
+  class Deployment
+    class MockConfig
+
+      attr_accessor :hash
+
+      def initialize(hash)
+        @hash = hash.merge(deploy: {})
+      end
+
+      def configfile
+        "/some/nonexistent/config_file"
+      end
+
+      # Perform a scan for key and check for both string and symbol keys
+      def setting(key)
+        @hash[key]
+      end
+
+      alias [] setting
+
+      def settings
+        @hash
+      end
+
+      def merge(other)
+        hash.merge(other)
+      end
+    end
+  end
+end

data/spec/r10k-mocks/mock_env.rb

@@ -0,0 +1,15 @@
+require 'r10k/environment'
+
+class R10K::Environment::Mock < R10K::Environment::Base
+  def sync
+    "synced"
+  end
+
+  def status
+    :insync
+  end
+
+  def signature
+    "mock"
+  end
+end

data/spec/r10k-mocks/mock_source.rb

@@ -0,0 +1,13 @@
+require 'r10k/source'
+require 'r10k-mocks/mock_env'
+
+class R10K::Source::Mock < R10K::Source::Base
+  R10K::Source.register(:mock, self)
+
+  def environments
+    corrected_environment_names = @options[:environments].map do |env|
+      R10K::Environment::Name.new(env, :prefix => @prefix, :invalid => 'correct_and_warn')
+    end
+    corrected_environment_names.map { |env| R10K::Environment::Mock.new(env.name, @basedir, env.dirname) }
+  end
+end

data/spec/shared-contexts/git-fixtures.rb

@@ -0,0 +1,55 @@
+require 'archive/tar/minitar'
+require 'tmpdir'
+
+shared_context "Git integration" do
+
+  # Use tmpdir for cached git repositories
+
+  before(:all) do
+    @old_cache_root = R10K::Git::Cache.settings[:cache_root]
+    R10K::Git::Cache.settings[:cache_root] = Dir.mktmpdir
+  end
+
+  after(:all) do
+    FileUtils.remove_entry_secure(R10K::Git::Cache.settings[:cache_root])
+    R10K::Git::Cache.settings[:cache_root] = @old_cache_root
+  end
+
+  # Create a mutable path for remote repositories
+
+  def fixture_path
+    File.join(PROJECT_ROOT, 'spec', 'fixtures', 'integration', 'git')
+  end
+
+  def remote_path
+    @remote_path
+  end
+
+  def populate_remote_path
+    Archive::Tar::Minitar.unpack(File.join(fixture_path, 'puppet-boolean-bare.tar'), remote_path)
+  end
+
+  def clear_remote_path
+    FileUtils.remove_entry_secure(remote_path)
+  end
+
+  before(:all) do
+    @remote_path = Dir.mktmpdir
+    populate_remote_path
+  end
+
+  after(:all) do
+    clear_remote_path
+    @remote_path = nil
+  end
+
+  let(:remote) { Pathname.new(File.join(remote_path, 'puppet-boolean.git')).realpath.to_s }
+
+  # Create a temp path for the git basedir and clean it up when finished
+
+  let(:basedir) { Dir.mktmpdir }
+
+  after do
+    FileUtils.remove_entry_secure(basedir)
+  end
+end

data/spec/shared-examples/deploy-actions.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+require 'r10k/deployment'
+
+shared_examples_for "a deploy action that requires a config file" do
+  let(:config) do
+    R10K::Deployment::MockConfig.new(
+      :sources => {
+        :control => {
+          :type => :mock,
+          :basedir => '/some/nonexistent/path/control',
+          :environments => %w[first second third],
+        },
+        :hiera => {
+          :type => :mock,
+          :basedir => '/some/nonexistent/path/hiera',
+          :environments => %w[fourth fifth sixth],
+        }
+      }
+    )
+  end
+
+  let(:deployment) { R10K::Deployment.new(config) }
+
+  before do
+    allow(R10K::Deployment).to receive(:new).and_return(deployment)
+  end
+
+  it "exits when no config file was provided or found" do
+    subject.instance_variable_set(:@config, nil)
+    expect {
+      subject.call
+    }.to exit_with(8)
+  end
+end
+
+shared_examples_for "a deploy action that can be write locked" do
+
+  let(:config) do
+    R10K::Deployment::MockConfig.new(
+      :sources => {
+        :control => {
+          :type => :mock,
+          :basedir => '/some/nonexistent/path/control',
+          :environments => %w[first second third],
+        },
+        :hiera => {
+          :type => :mock,
+          :basedir => '/some/nonexistent/path/hiera',
+          :environments => %w[fourth fifth sixth],
+        }
+      }
+    )
+  end
+
+  let(:deployment) { R10K::Deployment.new(config) }
+
+  before do
+    allow(R10K::Deployment).to receive(:new).and_return(deployment)
+    subject.settings = {deploy: {write_lock: "Disabled, yo"}}
+  end
+
+  it "exits without running when the write lock is set" do
+    expect(subject).to_not receive(:visit_deployment)
+    expect {
+      subject.call
+    }.to exit_with(16)
+  end
+end

data/spec/shared-examples/git-repository.rb

@@ -0,0 +1,38 @@
+shared_examples_for "a git repository" do
+
+  it "does not exist if the repo is not present" do
+    expect(subject.exist?).to be_falsey
+  end
+
+  describe "listing tags" do
+    before do
+      subject.clone(remote)
+    end
+
+    it "lists all tags in alphabetical order" do
+      expect(subject.tags).to eq(%w[0.9.0 0.9.0-rc1 1.0.0 1.0.1])
+    end
+  end
+
+  describe "resolving refs" do
+    before do
+      subject.clone(remote)
+    end
+
+    it "can resolve branches" do
+      expect(subject.resolve('master')).to eq '157011a4eaa27f1202a9d94335ee4876b26d377e'
+    end
+
+    it "can resolve tags" do
+      expect(subject.resolve('1.0.0')).to eq '14cbb45ae3a5f764320b7e63f1a54a25a1ef6c9c'
+    end
+
+    it "can resolve commits" do
+      expect(subject.resolve('3084373e8d181cf2fea5b4ade2690ba22872bd67')).to eq '3084373e8d181cf2fea5b4ade2690ba22872bd67'
+    end
+
+    it "returns nil when the object cannot be resolved" do
+      expect(subject.resolve('1.2.3')).to be_nil
+    end
+  end
+end

data/spec/shared-examples/git/bare_repository.rb

@@ -0,0 +1,132 @@
+RSpec.shared_examples "a git bare repository" do
+
+  describe "checking for the presence of the repo" do
+    it "exists if the repo is present" do
+      subject.clone(remote)
+      expect(subject.exist?).to be_truthy
+    end
+
+    it "doesn't exist if the repo is not present" do
+      expect(subject.exist?).to be_falsey
+    end
+  end
+
+  describe "cloning the repo" do
+    it "creates the repo at the expected location" do
+      subject.clone(remote)
+      config = File.read(File.join(basedir, dirname, 'config'))
+      expect(config).to match(remote)
+    end
+
+    context "without a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return(nil)
+      end
+
+      it 'does not change proxy ENV' do
+        expect(ENV).to_not receive(:[]=)
+        expect(ENV).to_not receive(:update)
+
+        subject.clone(remote)
+      end
+    end
+
+    context "with a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return('http://proxy.example.com:3128')
+      end
+
+      it "manages proxy-related ENV vars" do
+        # Sets proxy settings.
+        ['HTTPS_PROXY', 'https_proxy', 'HTTP_PROXY', 'http_proxy'].each do |var|
+          expect(ENV).to receive(:[]=).with(var, 'http://proxy.example.com:3128')
+        end
+
+        # Resets proxy settings when done.
+        expect(ENV).to receive(:update).with(hash_including('HTTPS_PROXY' => nil))
+
+        subject.clone(remote)
+      end
+    end
+  end
+
+  describe "updating the repo" do
+    let(:tag_090) { subject.git_dir + 'refs' + 'tags' + '0.9.0' }
+    let(:packed_refs) { subject.git_dir + 'packed-refs' }
+
+    before do
+      subject.clone(remote)
+      tag_090.delete if tag_090.exist?
+      packed_refs.delete if packed_refs.exist?
+    end
+
+    it "fetches objects from the remote" do
+      expect(subject.tags).to_not include('0.9.0')
+      subject.fetch
+      expect(subject.tags).to include('0.9.0')
+    end
+
+    context "without a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return(nil)
+      end
+
+      it 'does not change proxy ENV' do
+        expect(ENV).to_not receive(:[]=)
+        expect(ENV).to_not receive(:update)
+
+        subject.fetch
+      end
+    end
+
+    context "with a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return('http://proxy.example.com:3128')
+      end
+
+      it "manages proxy-related ENV vars" do
+        # Sets proxy settings.
+        ['HTTPS_PROXY', 'https_proxy', 'HTTP_PROXY', 'http_proxy'].each do |var|
+          expect(ENV).to receive(:[]=).with(var, 'http://proxy.example.com:3128')
+        end
+
+        # Resets proxy settings when done.
+        expect(ENV).to receive(:update).with(hash_including('HTTPS_PROXY' => nil))
+
+        subject.fetch
+      end
+    end
+  end
+
+  describe "listing branches" do
+    before do
+      subject.clone(remote)
+    end
+
+    it "lists all branches in alphabetical order" do
+      expect(subject.branches).to eq(%w[0.9.x master])
+    end
+  end
+
+  describe "determining ref type" do
+    before do
+      subject.clone(remote)
+    end
+
+    it "can infer the type of a branch ref" do
+      expect(subject.ref_type('master')).to eq :branch
+    end
+
+    it "can infer the type of a tag ref" do
+      expect(subject.ref_type('1.0.0')).to eq :tag
+    end
+
+    it "can infer the type of a commit" do
+      expect(subject.ref_type('3084373e8d181cf2fea5b4ade2690ba22872bd67')).to eq :commit
+    end
+
+    it "returns :unknown when the type cannot be inferred" do
+      expect(subject.ref_type('1.2.3')).to eq :unknown
+    end
+  end
+end

data/spec/shared-examples/git/thin_repository.rb

@@ -0,0 +1,26 @@
+RSpec.shared_examples "a git thin repository" do
+  describe "cloning" do
+    it "creates a working copy of the repo" do
+      subject.clone(remote)
+      expect(subject.exist?).to be_truthy
+    end
+
+    it "sets the remote origin url to the provided url" do
+      subject.clone(remote)
+      expect(subject.origin).to eq remote
+    end
+
+    it "sets the remote cache url to the path to the cache repo" do
+      subject.clone(remote)
+      expect(subject.cache).to eq cacherepo.git_dir.to_s
+    end
+
+    it "adds the cache repo to the alternates file" do
+      subject.clone(remote)
+      objectpath = cacherepo.git_dir + 'objects'
+      alternates = subject.alternates.to_a
+      expect(alternates.size).to eq 1
+      expect(alternates[0]).to match_realpath objectpath
+    end
+  end
+end

data/spec/shared-examples/git/working_repository.rb

@@ -0,0 +1,207 @@
+RSpec.shared_examples "a git working repository" do
+  describe "cloning" do
+    it "creates a working copy of the repo" do
+      subject.clone(remote)
+      expect(subject.exist?).to be_truthy
+    end
+
+    describe "using the default branch" do
+      describe "and the remote HEAD is 'master'" do
+        it "checks out the default branch" do
+          subject.clone(remote)
+          expect(subject.head).to eq "157011a4eaa27f1202a9d94335ee4876b26d377e"
+        end
+      end
+
+      describe "and the remote HEAD is '0.9.x'" do
+        before do
+          File.open(File.join(remote, 'HEAD'), 'w') do |fh|
+            fh.write('ref: refs/heads/0.9.x')
+          end
+        end
+
+        after do
+          clear_remote_path
+          populate_remote_path
+        end
+
+        it "checks out the default branch" do
+          subject.clone(remote)
+          expect(subject.head).to eq "3084373e8d181cf2fea5b4ade2690ba22872bd67"
+        end
+      end
+    end
+
+    describe "using an explicit ref" do
+      it "can check out tags" do
+        subject.clone(remote, {:ref => '1.0.0'})
+        expect(subject.head).to eq "14cbb45ae3a5f764320b7e63f1a54a25a1ef6c9c"
+      end
+
+      it "can check out remote branches" do
+        subject.clone(remote, {:ref => 'origin/0.9.x'})
+        expect(subject.head).to eq "3084373e8d181cf2fea5b4ade2690ba22872bd67"
+      end
+
+      it "can check out commits" do
+        subject.clone(remote, {:ref => '14cbb45ae3a5f764320b7e63f1a54a25a1ef6c9c'})
+        expect(subject.head).to eq "14cbb45ae3a5f764320b7e63f1a54a25a1ef6c9c"
+      end
+    end
+
+    describe "with a reference repository" do
+      it "adds the reference repository to the alternates directory" do
+        subject.clone(remote, {:reference => remote})
+        alternates = subject.alternates.to_a
+        expect(alternates.size).to eq 1
+        expect(alternates[0]).to match_realpath File.join(remote, 'objects')
+      end
+    end
+
+    context "without a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return(nil)
+      end
+
+      it 'does not change proxy ENV' do
+        expect(ENV).to_not receive(:[]=)
+        expect(ENV).to_not receive(:update)
+
+        subject.clone(remote)
+      end
+    end
+
+    context "with a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return('http://proxy.example.com:3128')
+      end
+
+      it "manages proxy-related ENV vars" do
+        # Sets proxy settings.
+        ['HTTPS_PROXY', 'https_proxy', 'HTTP_PROXY', 'http_proxy'].each do |var|
+          expect(ENV).to receive(:[]=).with(var, 'http://proxy.example.com:3128')
+        end
+
+        # Resets proxy settings when done.
+        expect(ENV).to receive(:update).with(hash_including('HTTPS_PROXY' => nil))
+
+        subject.clone(remote)
+      end
+    end
+  end
+
+  describe "updating the repo" do
+    let(:tag_090) { subject.git_dir + 'refs' + 'tags' + '0.9.0' }
+    let(:packed_refs) { subject.git_dir + 'packed-refs' }
+
+    before do
+      subject.clone(remote)
+      tag_090.delete if tag_090.exist?
+      packed_refs.delete if packed_refs.exist?
+    end
+
+    it "fetches objects from the remote" do
+      expect(subject.tags).to_not include('0.9.0')
+      subject.fetch
+      expect(subject.tags).to include('0.9.0')
+    end
+
+    context "without a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return(nil)
+      end
+
+      it 'does not change proxy ENV' do
+        expect(ENV).to_not receive(:[]=)
+        expect(ENV).to_not receive(:update)
+
+        subject.fetch
+      end
+    end
+
+    context "with a proxy" do
+      before(:each) do
+        allow(R10K::Git).to receive(:get_proxy_for_remote).with(remote).and_return('http://proxy.example.com:3128')
+      end
+
+      it "manages proxy-related ENV vars" do
+        # Sets proxy settings.
+        ['HTTPS_PROXY', 'https_proxy', 'HTTP_PROXY', 'http_proxy'].each do |var|
+          expect(ENV).to receive(:[]=).with(var, 'http://proxy.example.com:3128')
+        end
+
+        # Resets proxy settings when done.
+        expect(ENV).to receive(:update).with(hash_including('HTTPS_PROXY' => nil))
+
+        subject.fetch
+      end
+    end
+  end
+
+  describe "listing branches" do
+    before do
+      subject.clone(remote)
+    end
+
+    it "lists the local branches" do
+      expect(subject.branches).to eq(%w[master])
+    end
+  end
+
+  describe "listing the origin" do
+    it "is nil if the remote is not set" do
+      expect(subject.origin).to be_nil
+    end
+
+    it "is the remote URL when set" do
+      subject.clone(remote)
+      expect(subject.origin).to eq remote
+    end
+  end
+
+  describe "checking out ref" do
+    before(:each) do
+      subject.clone(remote)
+      File.open(File.join(subject.path, 'README.markdown'), 'a') { |f| f.write('local modifications!') }
+    end
+
+    context "with force = true" do
+      it "should revert changes in managed files" do
+        subject.checkout(subject.head, {:force => true})
+        expect(File.read(File.join(subject.path, 'README.markdown')).include?('local modifications!')).to eq false
+      end
+    end
+
+    context "with force = false" do
+      it "should not revert changes in managed files" do
+        subject.checkout(subject.head, {:force => false})
+        expect(File.read(File.join(subject.path, 'README.markdown')).include?('local modifications!')).to eq true
+      end
+    end
+  end
+
+  describe "checking if worktree is dirty" do
+    before do
+      subject.clone(remote)
+    end
+
+    context "with no local changes" do
+      it "reports worktree as not dirty" do
+        expect(subject.dirty?).to be false
+      end
+    end
+
+    context "with local changes" do
+      before(:each) do
+        File.open(File.join(subject.path, 'README.markdown'), 'a') { |f| f.write('local modifications!') }
+      end
+
+      it "logs and reports worktree as dirty" do
+        expect(subject.logger).to receive(:debug).with(/found local modifications in.*README\.markdown/i)
+        expect(subject.logger).to receive(:debug1)
+
+        expect(subject.dirty?).to be true
+      end
+    end
+  end
+end