r10k 1.4.2 → 1.5.1

data/spec/integration/git/rugged/bare_repository_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'r10k/git/rugged/bare_repository'
+
+describe R10K::Git::Rugged::BareRepository, :if => R10K::Features.available?(:rugged) do
+  include_context 'Git integration'
+
+  let(:dirname) { 'bare-repo.git' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  it_behaves_like 'a git repository'
+  it_behaves_like 'a git bare repository'
+end

data/spec/integration/git/rugged/thin_repository_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+require 'r10k/git/rugged/thin_repository'
+
+describe R10K::Git::Rugged::ThinRepository, :if => R10K::Features.available?(:rugged) do
+  include_context 'Git integration'
+
+  let(:dirname) { 'working-repo' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  let(:cacherepo) { R10K::Git::Rugged::Cache.generate(remote) }
+
+  it_behaves_like "a git thin repository"
+end

data/spec/integration/git/rugged/working_repository_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'r10k/git/rugged/working_repository'
+
+describe R10K::Git::Rugged::WorkingRepository, :if => R10K::Features.available?(:rugged) do
+  include_context 'Git integration'
+
+  let(:dirname) { 'working-repo' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  it_behaves_like 'a git repository'
+  it_behaves_like 'a git working repository'
+end

data/spec/integration/git/shellgit/bare_repository_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'r10k/git/shellgit/bare_repository'
+
+describe R10K::Git::ShellGit::BareRepository do
+  include_context 'Git integration'
+
+  let(:dirname) { 'bare-repo.git' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  it_behaves_like 'a git repository'
+  it_behaves_like 'a git bare repository'
+end

data/spec/integration/git/shellgit/thin_repository_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+require 'r10k/git/shellgit/thin_repository'
+
+describe R10K::Git::ShellGit::ThinRepository do
+  include_context 'Git integration'
+
+  let(:dirname) { 'working-repo' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  let(:cacherepo) { R10K::Git::ShellGit::Cache.generate(remote) }
+
+  it_behaves_like "a git thin repository"
+end

data/spec/integration/git/shellgit/working_repository_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+require 'r10k/git/shellgit/working_repository'
+
+describe R10K::Git::ShellGit::WorkingRepository do
+  include_context 'Git integration'
+
+  let(:dirname) { 'working-repo' }
+
+  subject { described_class.new(basedir, dirname) }
+
+  it_behaves_like 'a git repository'
+  it_behaves_like 'a git working repository'
+end

data/spec/integration/git/stateful_repository_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+require 'r10k/git'
+require 'r10k/git/stateful_repository'
+
+describe R10K::Git::StatefulRepository do
+  include_context 'Git integration'
+
+  let(:dirname) { 'working-repo' }
+
+  let(:thinrepo) { R10K::Git.thin_repository.new(basedir, dirname) }
+  let(:cacherepo) { R10K::Git.cache.generate(remote) }
+
+  subject { described_class.new('0.9.x', remote, basedir, dirname) }
+
+  describe 'status' do
+    describe "when the directory does not exist" do
+      it "is absent" do
+        expect(subject.status).to eq :absent
+      end
+    end
+
+    describe "when the directory is not a git repository" do
+      it "is mismatched" do
+        thinrepo.path.mkdir
+        expect(subject.status).to eq :mismatched
+      end
+    end
+
+    describe "when the repository doesn't match the desired remote" do
+      it "is mismatched" do
+        thinrepo.clone(remote, {:ref => '1.0.0'})
+        allow(subject.repo).to receive(:origin).and_return('http://some.site/repo.git')
+        expect(subject.status).to eq :mismatched
+      end
+    end
+
+    describe "when the wrong ref is checked out" do
+      it "is outdated" do
+        thinrepo.clone(remote, {:ref => '1.0.0'})
+        expect(subject.status).to eq :outdated
+      end
+    end
+
+    describe "when the ref is a branch and the cache is not synced" do
+      it "is outdated" do
+        thinrepo.clone(remote, {:ref => '0.9.x'})
+        cacherepo.reset!
+        expect(subject.status).to eq :outdated
+      end
+    end
+
+    describe "when the ref can't be resolved" do
+      subject { described_class.new('1.1.x', remote, basedir, dirname) }
+
+      it "is outdated" do
+        thinrepo.clone(remote, {:ref => '0.9.x'})
+        expect(subject.status).to eq :outdated
+      end
+    end
+
+    describe "if the right ref is checked out" do
+      it "is insync" do
+        thinrepo.clone(remote, {:ref => '0.9.x'})
+        expect(subject.status).to eq :insync
+      end
+    end
+  end
+
+  describe "syncing" do
+
+    describe "when the ref is unresolvable" do
+      subject { described_class.new('1.1.x', remote, basedir, dirname) }
+
+      it "raises an error" do
+        expect {
+          subject.sync
+        }.to raise_error(R10K::Git::UnresolvableRefError)
+      end
+    end
+
+    describe "when the repo is absent" do
+      it "creates the repo" do
+        subject.sync
+        expect(subject.status).to eq :insync
+      end
+    end
+
+    describe "when the repo is mismatched" do
+      it "removes and recreates the repo" do
+        thinrepo.path.mkdir
+        subject.sync
+        expect(subject.status).to eq :insync
+      end
+    end
+
+    describe "when the repo is out of date" do
+      it "updates the repository" do
+        thinrepo.clone(remote, {:ref => '1.0.0'})
+        subject.sync
+        expect(subject.status).to eq :insync
+      end
+    end
+  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/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/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,70 @@
+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
+  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
+  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,115 @@
+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
+  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
+      subject.fetch
+      expect(subject.tags).to include('0.9.0')
+    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
+  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
+end