cookbook-omnifetch 0.8.0 → 0.10.1

data/spec/unit/metadata_based_installer_spec.rb

@@ -1,137 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/metadata_based_installer.rb"
-
-RSpec.shared_context "sample_metadata" do
-
-  let(:raw_metadata) do
-    {
-      "recipes" => [
-        { "name" => "default.rb", "path" => "recipes/default.rb", "checksum" => "a6be794cdd2eb44d38fdf17f792a0d0d", "specificity" => "default", "url" => "https://example.com/recipes/default.rb" },
-      ],
-      "root_files" => [
-        { "name" => "metadata.rb", "path" => "metadata.rb", "checksum" => "5b346119e5e41ab99500608decac8dca", "specificity" => "default", "url" => "https://example.com/metadata.rb" },
-      ],
-    }
-
-  end
-end
-
-RSpec.describe CookbookOmnifetch::MetadataBasedInstaller::CookbookMetadata do
-
-  include_context "sample_metadata"
-
-  subject(:cb_metadata) { described_class.new(raw_metadata) }
-
-  it "yields a set of paths and urls" do
-    expect { |b| cb_metadata.files(&b) }.to yield_successive_args(["https://example.com/recipes/default.rb", "recipes/default.rb"], ["https://example.com/metadata.rb", "metadata.rb"])
-  end
-end
-
-RSpec.describe CookbookOmnifetch::MetadataBasedInstaller do
-
-  include_context "sample_metadata"
-
-  let(:url_path) { "/cookbooks/example/0.5.0" }
-
-  let(:http_client) do
-    double("Http Client")
-  end
-
-  let(:recipe_url) do
-    raw_metadata["recipes"][0]["url"]
-  end
-
-  let(:recipe_path) do
-    raw_metadata["recipes"][0]["path"]
-  end
-
-  let(:recipe_filehandle) do
-    File.open(File.join(remote_path, recipe_path))
-  end
-
-  let(:root_file_url) do
-    raw_metadata["root_files"][0]["url"]
-  end
-
-  let(:root_file_path) do
-    raw_metadata["root_files"][0]["path"]
-  end
-
-  let(:root_file_filehandle) do
-    File.open(File.join(remote_path, root_file_path))
-  end
-
-  let(:cookbook_fixture_path) { fixtures_path.join("cookbooks/example_cookbook") }
-
-  let(:test_root) { Dir.mktmpdir(nil) }
-
-  let(:cache_path) { File.join(test_root, "cache") }
-
-  let(:remote_path) { File.join(test_root, "remote") }
-
-  let(:install_path) { File.join(test_root, "install_path") }
-
-  let(:cookbook_files) { %w{metadata.rb recipes recipes/default.rb} }
-
-  let(:expected_installed_files) do
-    cookbook_files.map do |file|
-      File.join(install_path, file)
-    end
-  end
-
-  subject(:installer) do
-    described_class.new(http_client: http_client,
-                        url_path: url_path,
-                        install_path: install_path)
-  end
-
-  before do
-    FileUtils.cp_r(cookbook_fixture_path, remote_path)
-
-    allow(CookbookOmnifetch).to receive(:cache_path).and_return(cache_path)
-  end
-
-  after do
-    FileUtils.rm_r(test_root)
-  end
-
-  it "stages the download to a randomized location" do
-    Kernel.srand(0)
-    expected_path = Pathname.new(cache_path).join(".cache_tmp/metadata-installer/_cookbooks_example_0_5_0_209652396")
-
-    expect(installer.staging_path).to eq(expected_path)
-
-    next_installer = described_class.new(http_client: http_client,
-                                         url_path: url_path,
-                                         install_path: install_path)
-
-    next_expected_path = Pathname.new(cache_path).join(".cache_tmp/metadata-installer/_cookbooks_example_0_5_0_398764591")
-    expect(next_installer.staging_path).to eq(next_expected_path)
-  end
-
-  describe "installing the cookbook" do
-
-    before do
-      expect(http_client).to receive(:get).
-        with(url_path).
-        and_return(raw_metadata)
-      expect(http_client).to receive(:streaming_request).
-        with(recipe_url).
-        and_yield(recipe_filehandle)
-      expect(http_client).to receive(:streaming_request).
-        with(root_file_url).
-        and_yield(root_file_filehandle)
-    end
-
-    it "installs the cookbook to the desired install path" do
-      expect(Dir).to_not exist(install_path)
-
-      installer.install
-
-      expect(Dir).to exist(install_path)
-      expect(Dir.glob("#{install_path}/**/*")).to match_array(expected_installed_files)
-    end
-
-  end
-
-end

data/spec/unit/path_spec.rb

@@ -1,119 +0,0 @@
-require "spec_helper"
-require "cookbook-omnifetch/path"
-
-module CookbookOmnifetch
-  describe PathLocation do
-    let(:relative_paths_root) { File.dirname(__FILE__) }
-    let(:constraint) { double("constraint", satisfies?: true) }
-    let(:dependency) do
-      double("dependency",
-        name: "nginx",
-        version_constraint: constraint,
-        relative_paths_root: relative_paths_root
-      )
-    end
-    let(:path) { fixtures_path.join("cookbooks", "example_cookbook") }
-    let(:relative_path) { Pathname.new("../fixtures/cookbooks/example_cookbook") }
-
-    subject { described_class.new(dependency, path: path) }
-
-    describe "#installed?" do
-      it "returns false" do
-        expect(subject.installed?).to be false
-      end
-    end
-
-    describe "#install" do
-      it "validates the cached cookbook" do
-        expect(subject).to receive(:validate_cached!).with(path)
-        subject.install
-      end
-    end
-
-    describe "#cached_cookbook" do
-      it "loads the cached cookbook at the path" do
-        expect(MockCachedCookbook).to receive(:from_path).with(path)
-        subject.cached_cookbook
-      end
-    end
-
-    describe "#relative_path" do
-      it "returns the path to the Berksfile" do
-        expect(subject.relative_path).to eq(relative_path)
-      end
-    end
-
-    describe "#expanded_path" do
-      it "returns the expanded path, relative to the Berksfile" do
-        absolute_path = Pathname.new(File.expand_path(relative_path, relative_paths_root))
-        expect(subject.expanded_path).to eq(absolute_path)
-      end
-    end
-
-    describe "#==" do
-      it "is false when compared with a non-PathLocation" do
-        this = PathLocation.new(dependency, path: ".")
-        that = "A string"
-        expect(this).to_not eq(that)
-      end
-
-      it "is false when the metadata? is not the same" do
-        this = PathLocation.new(dependency, path: ".")
-        that = PathLocation.new(dependency, path: ".", metadata: true)
-        expect(this).to_not eq(that)
-      end
-
-      it "is false when the expanded paths are different" do
-        this = PathLocation.new(dependency, path: ".")
-        that = PathLocation.new(dependency, path: "..")
-        expect(this).to_not eq(that)
-      end
-
-      it "is true when they are the same" do
-        this = PathLocation.new(dependency, path: ".", metadata: true)
-        that = PathLocation.new(dependency, path: ".", metadata: true)
-        expect(this).to eq(that)
-      end
-    end
-
-    describe "#to_lock" do
-      it "includes the path relative to the Berksfile" do
-        expect(subject.to_lock).to eq <<-EOH.gsub(/^ {10}/, "")
-              path: #{relative_path}
-        EOH
-      end
-
-      it "includes the metadata attribute" do
-        subject.stub(:metadata?).and_return(true)
-        expect(subject.to_lock).to eq <<-EOH.gsub(/^ {10}/, "")
-              path: #{relative_path}
-              metadata: true
-        EOH
-      end
-    end
-
-    describe "#lock_data" do
-      it "includes the path relative to the Berksfile" do
-        expect(subject.lock_data).to eq({ "path" => relative_path.to_s })
-      end
-
-      it "includes the metadata attribute" do
-        subject.stub(:metadata?).and_return(true)
-        expect(subject.lock_data).to eq({ "path" => relative_path.to_s, "metadata" => true })
-      end
-    end
-
-    describe "#to_s" do
-      it "uses the relative path" do
-        expect(subject.to_s).to eq("source at #{relative_path}")
-      end
-    end
-
-    describe "#inspect" do
-      it "includes the right information" do
-        subject.stub(:metadata?).and_return(true)
-        expect(subject.inspect).to eq("#<CookbookOmnifetch::PathLocation metadata: true, path: #{relative_path}>")
-      end
-    end
-  end
-end