librarian-chef-nochef 0.1.0

data/lib/librarian/chef/templates/Cheffile

@@ -0,0 +1,15 @@
+#!/usr/bin/env ruby
+#^syntax detection
+
+site 'http://community.opscode.com/api/v1'
+
+# cookbook 'chef-client'
+
+# cookbook 'apache2', '>= 1.0.0'
+
+# cookbook 'rvm',
+#   :git => 'https://github.com/fnichol/chef-rvm'
+
+# cookbook 'postgresql',
+#   :git => 'https://github.com/findsyou/cookbooks',
+#   :ref => 'postgresql-improvements'

data/lib/librarian/chef/version.rb

@@ -0,0 +1,5 @@
+module Librarian
+  module Chef
+    VERSION = "0.0.2"
+  end
+end

data/librarian-chef-nochef.gemspec

@@ -0,0 +1,26 @@
+# -*- encoding: utf-8 -*-
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+
+Gem::Specification.new do |gem|
+  gem.name          = "librarian-chef-nochef"
+  gem.version       = "0.1.0"
+  gem.authors       = ["Emiliano Ticci", "Jay Feldblum"]
+  gem.email         = ["emiticci@gmail.com", "y_feldblum@yahoo.com"]
+  gem.summary       = %q{A Bundler for your Chef Cookbooks that does not depends on chef.}
+  gem.description   = %q{A Bundler for your Chef Cookbooks that does not depends on chef.}
+  gem.homepage      = "https://github.com/emyl/librarian-chef-nochef"
+  gem.license       = "MIT"
+
+  gem.files         = `git ls-files`.split($/)
+  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
+  gem.require_paths = ["lib"]
+
+  gem.add_dependency "librarian", "~> 0.1.0"
+  gem.add_dependency "minitar", ">= 0.5.2"
+
+  gem.add_development_dependency "rake"
+  gem.add_development_dependency "rspec"
+  gem.add_development_dependency "webmock"
+end

data/spec/functional/chef/cli_spec.rb

@@ -0,0 +1,195 @@
+require "securerandom"
+
+require "librarian/rspec/support/cli_macro"
+
+require "librarian/chef/cli"
+
+module Librarian
+  module Chef
+    describe Cli do
+      include Librarian::RSpec::Support::CliMacro
+
+      describe "init" do
+        before do
+          cli! "init"
+        end
+
+        it "should create a file named Cheffile" do
+          pwd.should have_file "Cheffile"
+        end
+      end
+
+      describe "version" do
+        before do
+          cli! "version"
+        end
+
+        it "should print the version" do
+          stdout.should == strip_heredoc(<<-STDOUT)
+            librarian-#{Librarian::VERSION}
+            librarian-chef-#{Librarian::Chef::VERSION}
+          STDOUT
+        end
+      end
+
+      describe "install" do
+
+        context "a simple Cheffile with one cookbook" do
+          let(:metadata) { {
+            "name" => "apt",
+            "version" => "1.0.0",
+            "dependencies" => { },
+          } }
+
+          before do
+            write_json_file! "cookbook-sources/apt/metadata.json", metadata
+            write_file! "Cheffile", strip_heredoc(<<-CHEFFILE)
+              cookbook 'apt',
+                :path => 'cookbook-sources'
+            CHEFFILE
+
+            cli! "install"
+          end
+
+          it "should write a lockfile" do
+            pwd.should have_file "Cheffile.lock"
+          end
+
+          it "should install the cookbook" do
+            pwd.should have_json_file "cookbooks/apt/metadata.json", metadata
+          end
+        end
+
+        context "a simple Cheffile with one cookbook with one dependency" do
+          let(:main_metadata) { {
+            "name" => "main",
+            "version" => "1.0.0",
+            "dependencies" => {
+              "sub" => "1.0.0",
+            }
+          } }
+          let(:sub_metadata) { {
+            "name" => "sub",
+            "version" => "1.0.0",
+            "dependencies" => { },
+          } }
+
+          before do
+            write_json_file! "cookbook-sources/main/metadata.json", main_metadata
+            write_json_file! "cookbook-sources/sub/metadata.json", sub_metadata
+            write_file! "Cheffile", strip_heredoc(<<-CHEFFILE)
+              path 'cookbook-sources'
+              cookbook 'main'
+            CHEFFILE
+
+            cli! "install"
+          end
+
+          it "should write a lockfile" do
+            pwd.should have_file "Cheffile.lock"
+          end
+
+          it "should install the dependant cookbook" do
+            pwd.should have_json_file "cookbooks/main/metadata.json", main_metadata
+          end
+
+          it "should install the independent cookbook" do
+            pwd.should have_json_file "cookbooks/sub/metadata.json", sub_metadata
+          end
+        end
+
+      end
+
+      describe "show" do
+        let(:main_metadata) { {
+          "name" => "main",
+          "version" => "1.0.0",
+          "dependencies" => {
+            "sub" => "1.0.0",
+          }
+        } }
+        let(:sub_metadata) { {
+          "name" => "sub",
+          "version" => "1.0.0",
+          "dependencies" => { },
+        } }
+
+        before do
+          write_json_file! "cookbook-sources/main/metadata.json", main_metadata
+          write_json_file! "cookbook-sources/sub/metadata.json", sub_metadata
+          write_file! "Cheffile", strip_heredoc(<<-CHEFFILE)
+            path 'cookbook-sources'
+            cookbook 'main'
+          CHEFFILE
+
+          cli! "install"
+        end
+
+        context "showing all without a lockfile" do
+          before do
+            pwd.join("Cheffile.lock").delete
+
+            cli! "show"
+          end
+
+          specify { exit_status.should == 1 }
+
+          it "should print a warning" do
+            stdout.should == strip_heredoc(<<-STDOUT)
+              Be sure to install first!
+            STDOUT
+          end
+        end
+
+        context "showing all" do
+          before do
+            cli! "show"
+          end
+
+          specify { exit_status.should == 0 }
+
+          it "should print a summary" do
+            stdout.should == strip_heredoc(<<-STDOUT)
+              main (1.0.0)
+              sub (1.0.0)
+            STDOUT
+          end
+        end
+
+        context "showing one without dependencies" do
+          before do
+            cli! "show", "sub"
+          end
+
+          specify { exit_status.should == 0 }
+
+          it "should print the details" do
+            stdout.should == strip_heredoc(<<-STDOUT)
+              sub (1.0.0)
+                source: cookbook-sources
+            STDOUT
+          end
+        end
+
+        context "showing one with dependencies" do
+          before do
+            cli! "show", "main"
+          end
+
+          specify { exit_status.should == 0 }
+
+          it "should print the details" do
+            stdout.should == strip_heredoc(<<-STDOUT)
+              main (1.0.0)
+                source: cookbook-sources
+                dependencies:
+                  sub (= 1.0.0)
+            STDOUT
+          end
+        end
+
+      end
+
+    end
+  end
+end

data/spec/functional/chef/source/site_spec.rb

@@ -0,0 +1,409 @@
+require 'pathname'
+require 'json'
+require 'webmock'
+
+require 'librarian'
+require 'librarian/helpers'
+require 'librarian/chef'
+require 'librarian/linter/source_linter'
+
+require 'support/project_path'
+
+module Librarian
+  module Chef
+    module Source
+      describe Site do
+
+        include WebMock::API
+
+        let(:project_path) { ::Support::ProjectPath.project_path }
+        let(:tmp_path) { project_path.join("tmp/spec/functional/chef/source/site") }
+        after { tmp_path.rmtree if tmp_path && tmp_path.exist? }
+
+        let(:api_url) { "http://site.cookbooks.com" }
+
+        let(:repo_path) { tmp_path.join("repo") }
+        let(:env) { repo_path.mkpath ; Environment.new(:project_path => repo_path) }
+
+        after { WebMock.reset! }
+
+        describe "lint" do
+          it "lints" do
+            Librarian::Linter::SourceLinter.lint! described_class
+          end
+        end
+
+        describe "class methods" do
+
+          describe ".lock_name" do
+            specify { described_class.lock_name.should == "SITE" }
+          end
+
+          describe ".from_spec_args" do
+            it "gives the expected source" do
+              args = { }
+              source = described_class.from_spec_args(env, api_url, args)
+              source.uri.should == api_url
+            end
+
+            it "raises on unexpected args" do
+              args = {:k => 3}
+              expect { described_class.from_spec_args(env, api_url, args) }.
+                to raise_error Librarian::Error, "unrecognized options: k"
+            end
+          end
+
+          describe ".from_lock_options" do
+            it "gives the expected source" do
+              options = {:remote => api_url}
+              source = described_class.from_lock_options(env, options)
+              source.uri.should == api_url
+            end
+
+            it "roundtrips" do
+              options = {:remote => api_url}
+              source = described_class.from_lock_options(env, options)
+              source.to_lock_options.should == options
+            end
+          end
+
+        end
+
+        describe "instance methods" do
+
+          let(:sample_metadata) do
+            Helpers.strip_heredoc(<<-METADATA)
+              version "0.6.5"
+            METADATA
+          end
+
+          let(:sample_index_data) do
+            {
+              "name" => "sample",
+              "versions" => [
+                "#{api_url}/cookbooks/sample/versions/0_6_5"
+              ]
+            }
+          end
+          let(:sample_0_6_5_data) do
+            {
+              "version" => "0.6.5",
+              "file" => "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar.gz"
+            }
+          end
+          let(:sample_0_6_5_package) do
+            s = StringIO.new
+            z = Zlib::GzipWriter.new(s, Zlib::NO_COMPRESSION)
+            t = Archive::Tar::Minitar::Output.new(z)
+            t.tar.add_file_simple("sample/metadata.rb", :mode => 0700,
+              :size => sample_metadata.bytesize){|io| io.write(sample_metadata)}
+            t.close
+            z.close unless z.closed?
+            s.string
+          end
+
+          before do
+            stub_request(:get, "#{api_url}/cookbooks/sample").
+              to_return(:body => JSON.dump(sample_index_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5").
+              to_return(:body => JSON.dump(sample_0_6_5_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar.gz").
+              to_return(:body => sample_0_6_5_package)
+          end
+
+          let(:source) { described_class.new(env, api_url) }
+
+          describe "#manifests" do
+            it "gives a list of all manifests" do
+              manifests = source.manifests("sample")
+              manifests.should have(1).item
+              manifest = manifests.first
+              manifest.source.should be source
+              manifest.version.should == Manifest::Version.new("0.6.5")
+              manifest.dependencies.should be_empty
+            end
+          end
+
+          describe "#fetch_version" do
+            it "fetches the version based on extra" do
+              extra = "#{api_url}/cookbooks/sample/versions/0_6_5"
+              source.fetch_version("sample", extra).should == "0.6.5"
+            end
+          end
+
+          describe "#fetch_dependencies" do
+            it "fetches the dependencies based on extra" do
+              extra = "#{api_url}/cookbooks/sample/versions/0_6_5"
+              source.fetch_dependencies("sample", "0.6.5", extra).should == [ ]
+            end
+          end
+
+          describe "#pinned?" do
+            it "returns false" do
+              source.should_not be_pinned
+            end
+          end
+
+          describe "#unpin!" do
+            it "is a no-op" do
+              source.unpin!
+            end
+          end
+
+          describe "#install!" do
+            before { env.install_path.mkpath }
+
+            context "directly" do
+              it "installs the manifest" do
+                manifest = Manifest.new(source, "sample")
+                manifest.version = "0.6.5"
+                source.install!(manifest)
+                text = env.install_path.join("sample/metadata.rb").read
+                text.should == sample_metadata
+              end
+            end
+
+            context "indirectly" do
+              it "installs the manifest" do
+                manifest = source.manifests("sample").first
+                source.install!(manifest)
+                text = env.install_path.join("sample/metadata.rb").read
+                text.should == sample_metadata
+              end
+            end
+          end
+
+          describe "#to_spec_args" do
+            it "gives the expected spec args" do
+              source.to_spec_args.should == [api_url, { }]
+            end
+          end
+
+          describe "#to_lock_options" do
+            it "gives the expected lock options" do
+              source.to_lock_options.should == {:remote => api_url}
+            end
+
+            it "roundtrips" do
+              options = source.to_lock_options
+              described_class.from_lock_options(env, options).should == source
+            end
+          end
+
+        end
+
+        describe "following http redirects" do
+          let(:source) { described_class.new(env, api_url) }
+
+          let(:sample_metadata) do
+            Helpers.strip_heredoc(<<-METADATA)
+              version "0.6.5"
+            METADATA
+          end
+
+          let(:sample_index_data) do
+            {
+              "name" => "sample",
+              "versions" => [
+                "#{api_url}/cookbooks/sample/versions/0_6_5"
+              ]
+            }
+          end
+          let(:sample_0_6_5_data) do
+            {
+              "version" => "0.6.5",
+              "file" => "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar.gz"
+            }
+          end
+          let(:sample_0_6_5_package) do
+            s = StringIO.new
+            z = Zlib::GzipWriter.new(s, Zlib::NO_COMPRESSION)
+            t = Archive::Tar::Minitar::Output.new(z)
+            t.tar.add_file_simple("sample/metadata.rb", :mode => 0700,
+              :size => sample_metadata.bytesize){|io| io.write(sample_metadata)}
+            t.close
+            z.close unless z.closed?
+            s.string
+          end
+
+          def redirect_to(url)
+            {:status => 302, :headers => {"Location" => url}}
+          end
+
+          context "with a sequence of http redirects" do
+            before do
+              stub_request(:get, "#{api_url}/cookbooks/sample").
+                to_return redirect_to "#{api_url}/cookbooks/sample-1"
+              stub_request(:get, "#{api_url}/cookbooks/sample-1").
+                to_return redirect_to "#{api_url}/cookbooks/sample-2"
+              stub_request(:get, "#{api_url}/cookbooks/sample-2").
+                to_return(:body => JSON.dump(sample_index_data))
+
+              stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5").
+                to_return(:body => JSON.dump(sample_0_6_5_data))
+
+              stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar.gz").
+                to_return(:body => sample_0_6_5_package)
+            end
+
+            it "follows a sequence of redirects" do
+              manifest = source.manifests("sample").first
+              manifest.version.to_s.should == "0.6.5"
+            end
+          end
+
+          context "with too many http redirects" do
+            before do
+              stub_request(:get, "#{api_url}/cookbooks/sample").
+                to_return redirect_to "#{api_url}/cookbooks/sample-1"
+              (1 .. 11).each do |i|
+                stub_request(:get, "#{api_url}/cookbooks/sample-#{i}").
+                  to_return redirect_to "#{api_url}/cookbooks/sample-#{i + 1}"
+              end
+              stub_request(:get, "#{api_url}/cookbooks/sample-12").
+                to_return(:body => JSON.dump(sample_index_data))
+            end
+
+            it "raises, warning of too many redirects" do
+              expect { source.manifests("sample").first }.
+                to raise_error Librarian::Error, /because too many redirects!/
+            end
+          end
+
+          context "with a redirect cycle" do
+            before do
+              stub_request(:get, "#{api_url}/cookbooks/sample").
+                to_return redirect_to "#{api_url}/cookbooks/sample-1"
+              (1 .. 8).each do |i|
+                stub_request(:get, "#{api_url}/cookbooks/sample-#{i}").
+                  to_return redirect_to "#{api_url}/cookbooks/sample-#{i + 1}"
+              end
+              stub_request(:get, "#{api_url}/cookbooks/sample-9").
+                to_return redirect_to "#{api_url}/cookbooks/sample-6"
+            end
+
+            it "raises, warning of a redirect cycle" do
+              expect { source.manifests("sample").first }.
+                to raise_error Librarian::Error, /because redirect cycle!/
+            end
+          end
+        end
+
+        describe "extracting .tar packages" do
+          let(:source) { described_class.new(env, api_url) }
+
+          let(:sample_metadata) do
+            Helpers.strip_heredoc(<<-METADATA)
+              version "0.6.5"
+            METADATA
+          end
+
+          let(:sample_index_data) do
+            {
+              "name" => "sample",
+              "versions" => [
+                "#{api_url}/cookbooks/sample/versions/0_6_5"
+              ]
+            }
+          end
+          let(:sample_0_6_5_data) do
+            {
+              "version" => "0.6.5",
+              "file" => "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar"
+            }
+          end
+          let(:sample_0_6_5_package) do
+            s = StringIO.new
+            t = Archive::Tar::Minitar::Output.new(s)
+            t.tar.add_file_simple("sample/metadata.rb", :mode => 0700,
+              :size => sample_metadata.bytesize){|io| io.write(sample_metadata)}
+            t.close
+            s.string
+          end
+
+          before do
+            stub_request(:get, "#{api_url}/cookbooks/sample").
+              to_return(:body => JSON.dump(sample_index_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5").
+              to_return(:body => JSON.dump(sample_0_6_5_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar").
+              to_return(:body => sample_0_6_5_package)
+          end
+
+          it "installs the manifest" do
+            env.install_path.mkpath
+            manifest = source.manifests("sample").first
+            source.install!(manifest)
+            text = env.install_path.join("sample/metadata.rb").read
+            text.should == sample_metadata
+          end
+
+        end
+
+        describe "extracting .tar packages with long filenames" do
+          let(:source) { described_class.new(env, api_url) }
+
+          before { tmp_path.mkpath unless tmp_path.exist? }
+          let(:bad_name_size) { Librarian::Posix.run!(%W[getconf NAME_MAX #{tmp_path}]).to_i }
+          let(:bad_name_path) { SecureRandom.hex((bad_name_size.to_f/2).ceil)[0...bad_name_size] }
+
+          before { expect(bad_name_path.size).to eq bad_name_size }
+
+          let(:sample_metadata) do
+            Helpers.strip_heredoc(<<-METADATA)
+              version "0.6.5"
+            METADATA
+          end
+
+          let(:sample_index_data) do
+            {
+              "name" => "sample",
+              "versions" => [
+                "#{api_url}/cookbooks/sample/versions/0_6_5"
+              ]
+            }
+          end
+          let(:sample_0_6_5_data) do
+            {
+              "version" => "0.6.5",
+              "file" => "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar"
+            }
+          end
+          let(:sample_0_6_5_package) do
+            tar_path = tmp_path + "tar-source"
+            tar_path.mkpath
+            tar_path.join("sample").mkpath
+            tar_path.join("sample/metadata.rb").open("w"){|io| io.write(sample_metadata)}
+            tar_path.join("sample/#{bad_name_path}").open("w"){|io| io.write("?")}
+            Librarian::Posix.run!(%W[tar cz -C #{tar_path} . ])
+          end
+
+          before do
+            stub_request(:get, "#{api_url}/cookbooks/sample").
+              to_return(:body => JSON.dump(sample_index_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5").
+              to_return(:body => JSON.dump(sample_0_6_5_data))
+
+            stub_request(:get, "#{api_url}/cookbooks/sample/versions/0_6_5/file.tar").
+              to_return(:body => sample_0_6_5_package)
+          end
+
+          it "installs the manifest" do
+            env.install_path.mkpath
+            manifest = source.manifests("sample").first
+            source.install!(manifest)
+            text = env.install_path.join("sample/#{bad_name_path}").read
+            text.should == "?"
+          end
+
+        end
+
+      end
+    end
+  end
+end