cookbook-omnifetch 0.7.0 → 0.10.0

data/spec/unit/base_spec.rb

@@ -1,87 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/base"
-
-module CookbookOmnifetch
-  describe BaseLocation do
-    let(:constraint) { double("constraint") }
-    let(:dependency) { double("dependency", name: "cookbook", version_constraint: constraint) }
-
-    subject { described_class.new(dependency) }
-
-    describe "#installed?" do
-      it "is an abstract function" do
-        expect { subject.installed? }.to raise_error(AbstractFunction)
-      end
-    end
-
-    describe "#install" do
-      it "is an abstract function" do
-        expect { subject.install }.to raise_error(AbstractFunction)
-      end
-    end
-
-    describe "#cached_cookbook" do
-      it "is an abstract function" do
-        expect { subject.cached_cookbook }.to raise_error(AbstractFunction)
-      end
-    end
-
-    describe "#to_lock" do
-      it "is an abstract function" do
-        expect { subject.to_lock }.to raise_error(AbstractFunction)
-      end
-    end
-
-    describe "#lock_data" do
-      it "is an abstract function" do
-        expect { subject.lock_data }.to raise_error(AbstractFunction)
-      end
-    end
-
-    describe "#validate_cached!" do
-      context "when the path is not a cookbook" do
-        before { CookbookOmnifetch.stub(:cookbook?).and_return(false) }
-
-        it "raises an error" do
-          expect do
-            subject.validate_cached!("/foo/bar")
-          end.to raise_error(NotACookbook)
-        end
-      end
-
-      context "when the path is a cookbook" do
-        let(:cookbook) do
-          double("cookbook",
-            cookbook_name: "cookbook",
-            version: "0.1.0"
-          )
-        end
-
-        before do
-          CookbookOmnifetch.stub(:cookbook?).and_return(true)
-          MockCachedCookbook.stub(:from_path).and_return(cookbook)
-        end
-
-        it "raises an error if the constraint does not satisfy" do
-          constraint.stub(:satisfies?).with("0.1.0").and_return(false)
-          expect do
-            subject.validate_cached!(cookbook)
-          end.to raise_error(CookbookValidationFailure)
-        end
-
-        it "raises an error if the names do not match" do
-          constraint.stub(:satisfies?).with("0.1.0").and_return(true)
-          cookbook.stub(:cookbook_name).and_return("different_name")
-          expect do
-            subject.validate_cached!(cookbook)
-          end.to raise_error(MismatchedCookbookName)
-        end
-
-        it "returns true when the validation succeeds" do
-          constraint.stub(:satisfies?).with("0.1.0").and_return(true)
-          expect(subject.validate_cached!(cookbook)).to be true
-        end
-      end
-    end
-  end
-end

data/spec/unit/chef_server_artifact_spec.rb

@@ -1,74 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/chef_server.rb"
-
-RSpec.describe CookbookOmnifetch::ChefServerArtifactLocation do
-
-  let(:http_client) { double("Http Client") }
-
-  let(:test_root) { "/some/fake/path" }
-
-  let(:storage_path) { File.join(test_root, "storage") }
-
-  let(:dependency) { double("Dependency", name: cookbook_name) }
-
-  let(:cookbook_name) { "example" }
-
-  let(:cookbook_identifier) { "467dc855408ce8b74f991c5dc2fd72a6aa369b60" }
-
-  let(:url) { "https://chef.example.com/organizations/example" }
-
-  let(:options) { { chef_server_artifact: url, identifier: cookbook_identifier, http_client: http_client } }
-
-  let(:expected_cache_key) { "example-467dc855408ce8b74f991c5dc2fd72a6aa369b60" }
-
-  subject(:chef_server_artifact_location) { described_class.new(dependency, options) }
-
-  before do
-    allow(CookbookOmnifetch).to receive(:storage_path).and_return(Pathname.new(storage_path))
-  end
-
-  it "has a URI" do
-    expect(chef_server_artifact_location.uri).to eq(url)
-  end
-
-  it "has an HTTP client" do
-    expect(chef_server_artifact_location.http_client).to eq(http_client)
-  end
-
-  it "has a metadata_based_installer" do
-    installer = chef_server_artifact_location.installer
-    expect(installer).to be_a(CookbookOmnifetch::MetadataBasedInstaller)
-    expect(installer.http_client).to eq(http_client)
-    expect(installer.url_path).to eq("/cookbook_artifacts/example/467dc855408ce8b74f991c5dc2fd72a6aa369b60")
-    expect(installer.install_path.to_s).to eq(File.join(storage_path, expected_cache_key))
-  end
-
-  it "has a cache key containing the site URI and version" do
-    expect(chef_server_artifact_location.cache_key).to eq(expected_cache_key)
-  end
-
-  it "has an identifier" do
-    expect(chef_server_artifact_location.cookbook_identifier).to eq(cookbook_identifier)
-  end
-
-  it "provides lock data as a Hash" do
-    expected_data = {
-      "chef_server" => url,
-      "server_identifier" => cookbook_identifier,
-    }
-    expect(chef_server_artifact_location.lock_data).to eq(expected_data)
-  end
-
-  describe "when installing" do
-
-    let(:installer) { instance_double("CookbookOmnifetch::MetadataBasedInstaller") }
-
-    it "delegates to the MetadataBasedInstaller" do
-      allow(chef_server_artifact_location).to receive(:installer).and_return(installer)
-      expect(installer).to receive(:install)
-      chef_server_artifact_location.install
-    end
-
-  end
-
-end

data/spec/unit/chef_server_spec.rb

@@ -1,72 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/chef_server.rb"
-
-RSpec.describe CookbookOmnifetch::ChefServerLocation do
-
-  let(:http_client) { double("Http Client") }
-
-  let(:test_root) { "/some/fake/path" }
-
-  let(:storage_path) { File.join(test_root, "storage") }
-
-  let(:dependency) { double("Dependency", name: cookbook_name) }
-
-  let(:cookbook_name) { "example" }
-
-  let(:cookbook_version) { "0.5.0" }
-
-  let(:url) { "https://chef.example.com/organizations/example" }
-
-  let(:options) { { chef_server: url, version: cookbook_version, http_client: http_client } }
-
-  subject(:chef_server_location) { described_class.new(dependency, options) }
-
-  before do
-    allow(CookbookOmnifetch).to receive(:storage_path).and_return(Pathname.new(storage_path))
-  end
-
-  it "has a URI" do
-    expect(chef_server_location.uri).to eq(url)
-  end
-
-  it "has an HTTP client" do
-    expect(chef_server_location.http_client).to eq(http_client)
-  end
-
-  it "has a metadata_based_installer" do
-    installer = chef_server_location.installer
-    expect(installer).to be_a(CookbookOmnifetch::MetadataBasedInstaller)
-    expect(installer.http_client).to eq(http_client)
-    expect(installer.url_path).to eq("/cookbooks/example/0.5.0")
-    expect(installer.install_path.to_s).to eq(File.join(storage_path, "example-0.5.0"))
-  end
-
-  it "has a cache key containing the site URI and version" do
-    expect(chef_server_location.cache_key).to eq("example-0.5.0")
-  end
-
-  it "has an exact version" do
-    expect(chef_server_location.cookbook_version).to eq("0.5.0")
-  end
-
-  it "provides lock data as a Hash" do
-    expected_data = {
-      "chef_server" => url,
-      "version" => "0.5.0",
-    }
-    expect(chef_server_location.lock_data).to eq(expected_data)
-  end
-
-  describe "when installing" do
-
-    let(:installer) { instance_double("CookbookOmnifetch::MetadataBasedInstaller") }
-
-    it "delegates to the MetadataBasedInstaller" do
-      allow(chef_server_location).to receive(:installer).and_return(installer)
-      expect(installer).to receive(:install)
-      chef_server_location.install
-    end
-
-  end
-
-end

data/spec/unit/exceptions_spec.rb

@@ -1,87 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/exceptions"
-
-module CookbookOmnifetch
-  describe "Exceptions" do
-
-    describe NotACookbook do
-
-      subject(:exception) do
-        described_class.new("/path/to/cookbook")
-      end
-
-      let(:message) do
-        "The resource at '/path/to/cookbook' does not appear to be a valid cookbook. Does it have a metadata.rb?"
-      end
-
-      it "creates an informative error in #message" do
-        expect(exception.message).to eq(message)
-      end
-
-      it "creates an informative error in #to_s" do
-        expect(exception.to_s).to eq(message)
-      end
-
-    end
-
-    describe CookbookValidationFailure do
-
-      let(:dependency) { instance_double("ChefDK::Policyfile::CookbookLocationSpecification", to_s: "apt ~> 1.2.3") }
-
-      # The exception class itself doesn't use this
-      let(:cached_cookbook) { Object.new }
-
-      subject(:exception) do
-        described_class.new(dependency, cached_cookbook)
-      end
-
-      let(:message) do
-        "The cookbook downloaded for apt ~> 1.2.3 did not satisfy the constraint."
-      end
-
-      it "creates an informative error in #message" do
-        expect(exception.message).to eq(message)
-      end
-
-      it "creates an informative error in #to_s" do
-        expect(exception.to_s).to eq(message)
-      end
-
-    end
-
-    describe MismatchedCookbookName do
-
-      let(:dependency) { instance_double("ChefDK::Policyfile::CookbookLocationSpecification", name: "apt") }
-
-      let(:cached_cookbook) { instance_double("Chef::Cookbook", cookbook_name: "apt") }
-
-      subject(:exception) do
-        described_class.new(dependency, cached_cookbook)
-      end
-
-      let(:message) do
-        <<-EOM
-In your Berksfile, you have:
-
-  cookbook 'apt'
-
-But that cookbook is actually named 'apt'
-
-This can cause potentially unwanted side-effects in the future.
-
-NOTE: If you do not explicitly set the 'name' attribute in the metadata, the name of the directory will be used instead. This is often a cause of confusion for dependency solving.
-EOM
-      end
-
-      it "creates an informative error in #message" do
-        expect(exception.message).to eq(message)
-      end
-
-      it "creates an informative error in #to_s" do
-        expect(exception.to_s).to eq(message)
-      end
-
-    end
-
-  end
-end

data/spec/unit/git_spec.rb

@@ -1,290 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/git"
-
-module CookbookOmnifetch
-  describe GitLocation do
-    let(:dependency) { double(name: "bacon") }
-
-    subject do
-      described_class.new(dependency, git: "https://repo.com", branch: "ham",
-                                      tag: "v1.2.3", ref: "abc123", revision: "defjkl123456", rel: "hi")
-    end
-
-    describe ".initialize" do
-      it "sets the uri" do
-        instance = described_class.new(dependency, git: "https://repo.com")
-        expect(instance.uri).to eq("https://repo.com")
-      end
-
-      it "sets the branch" do
-        instance = described_class.new(dependency,
-          git: "https://repo.com", branch: "magic_new_feature")
-        expect(instance.branch).to eq("magic_new_feature")
-      end
-
-      it "sets the tag" do
-        instance = described_class.new(dependency,
-          git: "https://repo.com", tag: "v1.2.3")
-        expect(instance.tag).to eq("v1.2.3")
-      end
-
-      it "adds the ref" do
-        instance = described_class.new(dependency,
-          git: "https://repo.com", ref: "abc123")
-        expect(instance.ref).to eq("abc123")
-      end
-
-      it "sets the revision" do
-        instance = described_class.new(dependency,
-          git: "https://repo.com", revision: "abcde12345")
-        expect(instance.revision).to eq("abcde12345")
-      end
-
-      it "sets the rel" do
-        instance = described_class.new(dependency,
-          git: "https://repo.com", rel: "internal/path")
-        expect(instance.rel).to eq("internal/path")
-      end
-
-      context "rev_parse" do
-        def rev_parse(instance)
-          instance.instance_variable_get(:@rev_parse)
-        end
-
-        it "uses the :ref option with priority" do
-          instance = described_class.new(dependency,
-            git: "https://repo.com", ref: "abc123", branch: "magic_new_feature")
-          expect(rev_parse(instance)).to eq("abc123")
-        end
-
-        it "uses the :branch option with priority" do
-          instance = described_class.new(dependency,
-            git: "https://repo.com", branch: "magic_new_feature", tag: "v1.2.3")
-          expect(rev_parse(instance)).to eq("magic_new_feature")
-        end
-
-        it "uses the :tag option" do
-          instance = described_class.new(dependency,
-            git: "https://repo.com", tag: "v1.2.3")
-          expect(rev_parse(instance)).to eq("v1.2.3")
-        end
-
-        it 'uses "master" when none is given' do
-          instance = described_class.new(dependency, git: "https://repo.com")
-          expect(rev_parse(instance)).to eq("master")
-        end
-      end
-    end
-
-    describe "#cache_key" do
-
-      it "concatenates the name and revision" do
-        subject.stub(:revision).and_return("abcdef123456")
-        expect(subject.cache_key).to eq("bacon-abcdef123456")
-      end
-    end
-
-    describe "#installed?" do
-      it "returns false when there is no revision" do
-        subject.stub(:revision).and_return(nil)
-        expect(subject.installed?).to be false
-      end
-
-      it "returns false when the install_path does not exist" do
-        subject.stub(:revision).and_return("abcd1234")
-        subject.stub(:install_path).and_return(double(exist?: false))
-        expect(subject.installed?).to be false
-      end
-
-      it "returns true when the location is installed" do
-        subject.stub(:revision).and_return("abcd1234")
-        subject.stub(:install_path).and_return(double(exist?: true))
-        expect(subject.installed?).to be true
-      end
-    end
-
-    describe "#install" do
-      before do
-        File.stub(:chmod)
-        FileUtils.stub(:cp_r)
-        subject.stub(:validate_cached!)
-        subject.stub(:git)
-      end
-
-      context "when the repository is cached" do
-        it "pulls a new version" do
-          Dir.stub(:chdir) { |args, &b| b.call } # Force eval the chdir block
-
-          subject.stub(:cached?).and_return(true)
-          expect(subject).to receive(:git).with(
-            'fetch --force --tags https://repo.com "refs/heads/*:refs/heads/*"'
-          )
-          subject.install
-        end
-      end
-
-      context "when the revision is not cached" do
-        it "clones the repository" do
-          Dir.stub(:chdir) { |args, &b| b.call } # Force eval the chdir block
-
-          cache_path = subject.send(:cache_path)
-          subject.stub(:cached?).and_return(false)
-          expect(subject).to receive(:git).with(
-            %{clone https://repo.com "#{cache_path}" --bare --no-hardlinks}
-          )
-          subject.install
-        end
-      end
-    end
-
-    describe "#cached_cookbook" do
-      it "returns nil if the cookbook is not installed" do
-        subject.stub(:installed?).and_return(false)
-        expect(subject.cached_cookbook).to be_nil
-      end
-
-      it "returns the cookbook at the install_path" do
-        subject.stub(:installed?).and_return(true)
-        MockCachedCookbook.stub(:from_path)
-
-        expect(MockCachedCookbook).to receive(:from_path).once
-        subject.cached_cookbook
-      end
-    end
-
-    describe "#==" do
-      let(:other) { subject.dup }
-
-      it "returns true when everything matches" do
-        expect(subject).to eq(other)
-      end
-
-      it "returns false when the other location is not an GitLocation" do
-        other.stub(:is_a?).and_return(false)
-        expect(subject).to_not eq(other)
-      end
-
-      it "returns false when the uri is different" do
-        other.stub(:uri).and_return("different")
-        expect(subject).to_not eq(other)
-      end
-
-      it "returns false when the branch is different" do
-        other.stub(:branch).and_return("different")
-        expect(subject).to_not eq(other)
-      end
-
-      it "returns false when the tag is different" do
-        other.stub(:tag).and_return("different")
-        expect(subject).to_not eq(other)
-      end
-
-      it "returns false when the ref is different" do
-        other.stub(:ref).and_return("different")
-        expect(subject).to_not eq(other)
-      end
-
-      it "returns false when the rel is different" do
-        other.stub(:rel).and_return("different")
-        expect(subject).to_not eq(other)
-      end
-    end
-
-    describe "#to_s" do
-      it "prefers the tag" do
-        expect(subject.to_s).to eq("https://repo.com (at v1.2.3/hi)")
-      end
-
-      it "prefers the branch" do
-        subject.stub(:tag).and_return(nil)
-        expect(subject.to_s).to eq("https://repo.com (at ham/hi)")
-      end
-
-      it "falls back to the ref" do
-        subject.stub(:tag).and_return(nil)
-        subject.stub(:branch).and_return(nil)
-        expect(subject.to_s).to eq("https://repo.com (at abc123/hi)")
-      end
-
-      it "does not use the rel if missing" do
-        subject.stub(:rel).and_return(nil)
-        expect(subject.to_s).to eq("https://repo.com (at v1.2.3)")
-      end
-    end
-
-    describe "#to_lock" do
-      it "includes all the information" do
-        expect(subject.to_lock).to eq <<-EOH.gsub(/^ {8}/, "")
-            git: https://repo.com
-            revision: defjkl123456
-            ref: abc123
-            branch: ham
-            tag: v1.2.3
-            rel: hi
-        EOH
-      end
-
-      it "does not include the branch if missing" do
-        subject.stub(:branch).and_return(nil)
-        expect(subject.to_lock).to_not include("branch")
-      end
-
-      it "does not include the tag if missing" do
-        subject.stub(:tag).and_return(nil)
-        expect(subject.to_lock).to_not include("tag")
-      end
-
-      it "does not include the rel if missing" do
-        subject.stub(:rel).and_return(nil)
-        expect(subject.to_lock).to_not include("rel")
-      end
-    end
-
-    describe "#lock_data" do
-      let(:full_lock_data) do
-        {
-          "git" => "https://repo.com",
-          "revision" => "defjkl123456",
-          "ref" => "abc123",
-          "branch" => "ham",
-          "tag" => "v1.2.3",
-          "rel" => "hi",
-        }
-      end
-
-      it "includes all the information" do
-        expect(subject.lock_data).to eq(full_lock_data)
-      end
-
-      it "does not include the branch if missing" do
-        subject.stub(:branch).and_return(nil)
-        expect(subject.lock_data).to_not have_key("branch")
-      end
-
-      it "does not include the tag if missing" do
-        subject.stub(:tag).and_return(nil)
-        expect(subject.lock_data).to_not have_key("tag")
-      end
-
-      it "does not include the rel if missing" do
-        subject.stub(:rel).and_return(nil)
-        expect(subject.lock_data).to_not have_key("rel")
-      end
-    end
-
-    describe "#git" do
-      before { described_class.send(:public, :git) }
-
-      it "raises an error if Git is not installed" do
-        CookbookOmnifetch.stub(:which).and_return(false)
-        expect { subject.git("foo") }.to raise_error(GitNotInstalled)
-      end
-
-      it "raises an error if the command fails" do
-        shell_out = double("shell_out", success?: false, stderr: nil)
-        MockShellOut.stub(:shell_out).and_return(shell_out)
-        expect { subject.git("foo") }.to raise_error(GitCommandError)
-      end
-    end
-  end
-end