librarian-chef 0.0.1.beta.1

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.1.beta.1"
+  end
+end

data/librarian-chef.gemspec

@@ -0,0 +1,27 @@
+# -*- encoding: utf-8 -*-
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'librarian/chef/version'
+
+Gem::Specification.new do |gem|
+  gem.name          = "librarian-chef"
+  gem.version       = Librarian::Chef::VERSION
+  gem.authors       = ["Jay Feldblum"]
+  gem.email         = ["y_feldblum@yahoo.com"]
+  gem.description   = %q{Librarian::Chef}
+  gem.summary       = %q{Librarian::Chef}
+  gem.homepage      = ""
+
+  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.beta.1"
+  gem.add_dependency "chef", ">= 0.10"
+  gem.add_dependency "archive-tar-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,266 @@
+require 'pathname'
+require 'json'
+require 'webmock'
+
+require 'librarian'
+require 'librarian/helpers'
+require 'librarian/chef'
+require 'librarian/linter/source_linter'
+
+module Librarian
+  module Chef
+    module Source
+      describe Site do
+
+        include WebMock::API
+
+        let(:project_path) do
+          project_path = Pathname.new(__FILE__).expand_path
+          project_path = project_path.dirname until project_path.join("Rakefile").exist?
+          project_path
+        end
+        let(:tmp_path) { project_path.join("tmp/spec/functional/chef/source/site") }
+        after { tmp_path.rmtree if tmp_path && tmp_path.exist? }
+        let(:sample_path) { tmp_path.join("sample") }
+        let(:sample_metadata) do
+          Helpers.strip_heredoc(<<-METADATA)
+            version "0.6.5"
+          METADATA
+        end
+
+        let(:api_url) { "http://site.cookbooks.com" }
+
+        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
+
+        # depends on repo_path being defined in each context
+        let(:env) { Environment.new(:project_path => repo_path) }
+
+        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
+
+        after do
+          WebMock.reset!
+        end
+
+        let(:repo_path) { tmp_path.join("methods") }
+        before { repo_path.mkpath }
+
+        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(: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) }
+
+          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))
+            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
+
+      end
+    end
+  end
+end