librarian-puppet 0.0.1.pre

data/vendor/librarian/spec/unit/dependency_spec.rb

@@ -0,0 +1,30 @@
+require "librarian/dependency"
+
+describe Librarian::Dependency do
+
+  describe "validations" do
+
+    context "when the name is blank" do
+      it "raises" do
+        expect { described_class.new("", [], nil) }.
+          to raise_error(ArgumentError, %{name ("") must be sensible})
+      end
+    end
+
+    context "when the name has leading whitespace" do
+      it "raises" do
+        expect { described_class.new("  the-name", [], nil) }.
+          to raise_error(ArgumentError, %{name ("  the-name") must be sensible})
+      end
+    end
+
+    context "when the name has trailing whitespace" do
+      it "raises" do
+        expect { described_class.new("the-name  ", [], nil) }.
+          to raise_error(ArgumentError, %{name ("the-name  ") must be sensible})
+      end
+    end
+
+  end
+
+end

data/vendor/librarian/spec/unit/dsl_spec.rb

@@ -0,0 +1,194 @@
+require 'librarian'
+require 'librarian/mock'
+
+module Librarian
+  module Mock
+
+    describe Dsl do
+
+      let(:env) { Environment.new }
+
+      context "a single source and a single dependency with a blank name" do
+        it "should not not run with a blank name" do
+          expect do
+            env.dsl do
+              src 'source-1'
+              dep ''
+            end
+          end.to raise_error(ArgumentError, %{name ("") must be sensible})
+        end
+      end
+
+      context "a single dependency but no applicable source" do
+
+        it "should not run without any sources" do
+          expect do
+            env.dsl do
+              dep 'dependency-1'
+            end
+          end.to raise_error(Dsl::Error)
+        end
+
+        it "should not run when a block source is defined but the dependency is outside the block" do
+          expect do
+            env.dsl do
+              src 'source-1' do end
+              dep 'dependency-1'
+            end
+          end.to raise_error(Dsl::Error)
+        end
+
+      end
+
+      context "a simple specfile - a single source, a single dependency, no transitive dependencies" do
+
+        it "should run with a hash source" do
+          spec = env.dsl do
+            dep 'dependency-1',
+              :src => 'source-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-1'
+          spec.source.should be_nil
+        end
+
+        it "should run with a shortcut source" do
+          spec = env.dsl do
+            dep 'dependency-1',
+              :source => :a
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-a'
+          spec.source.should be_nil
+        end
+
+        it "should run with a block hash source" do
+          spec = env.dsl do
+            source :src => 'source-1' do
+              dep 'dependency-1'
+            end
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-1'
+          spec.source.should be_nil
+        end
+
+        it "should run with a block named source" do
+          spec = env.dsl do
+            src 'source-1' do
+              dep 'dependency-1'
+            end
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-1'
+          spec.source.should be_nil
+        end
+
+        it "should run with a default hash source" do
+          spec = env.dsl do
+            source :src => 'source-1'
+            dep 'dependency-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-1'
+          spec.source.should_not be_nil
+          spec.dependencies.first.source.should == spec.source
+        end
+
+        it "should run with a default named source" do
+          spec = env.dsl do
+            src 'source-1'
+            dep 'dependency-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-1'
+          spec.source.should_not be_nil
+          spec.dependencies.first.source.should == spec.source
+        end
+
+        it "should run with a default shortcut source" do
+          spec = env.dsl do
+            source :a
+            dep 'dependency-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-a'
+          spec.source.should_not be_nil
+          spec.dependencies.first.source.should == spec.source
+        end
+
+        it "should run with a shortcut source hash definition" do
+          spec = env.dsl do
+            source :b, :src => 'source-b'
+            dep 'dependency-1', :source => :b
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-b'
+          spec.source.should be_nil
+        end
+
+        it "should run with a shortcut source block definition" do
+          spec = env.dsl do
+            source :b, proc { src 'source-b' }
+            dep 'dependency-1', :source => :b
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-b'
+          spec.source.should be_nil
+        end
+
+        it "should run with a default shortcut source hash definition" do
+          spec = env.dsl do
+            source :b, :src => 'source-b'
+            source :b
+            dep 'dependency-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-b'
+          spec.source.should_not be_nil
+          spec.source.name.should == 'source-b'
+        end
+
+        it "should run with a default shortcut source block definition" do
+          spec = env.dsl do
+            source :b, proc { src 'source-b' }
+            source :b
+            dep 'dependency-1'
+          end
+          spec.dependencies.should_not be_empty
+          spec.dependencies.first.name.should == 'dependency-1'
+          spec.dependencies.first.source.name.should == 'source-b'
+          spec.source.should_not be_nil
+          spec.source.name.should == 'source-b'
+        end
+
+      end
+
+      context "validating source options" do
+
+        it "should raise when given unrecognized optiosn options" do
+          expect do
+            env.dsl do
+              dep 'dependency-1',
+                :src => 'source-1',
+                :huh => 'yikes'
+            end
+          end.to raise_error(Error, %{unrecognized options: huh})
+        end
+
+      end
+
+    end
+
+  end
+end

data/vendor/librarian/spec/unit/environment_spec.rb

@@ -0,0 +1,9 @@
+require "librarian/environment"
+
+module Librarian
+  describe Environment do
+
+    let(:env) { described_class.new }
+
+  end
+end

data/vendor/librarian/spec/unit/lockfile/parser_spec.rb

@@ -0,0 +1,162 @@
+require "librarian/helpers"
+require "librarian/lockfile/parser"
+require "librarian/mock"
+
+module Librarian
+  describe Lockfile::Parser do
+
+    let(:env) { Mock::Environment.new }
+    let(:parser) { described_class.new(env) }
+    let(:resolution) { parser.parse(lockfile) }
+
+    context "a mock lockfile with one source and no dependencies" do
+      let(:lockfile) do
+        Helpers.strip_heredoc <<-LOCKFILE
+          MOCK
+            remote: source-a
+            specs:
+
+          DEPENDENCIES
+
+        LOCKFILE
+      end
+
+      it "should give an empty list of dependencies" do
+        resolution.dependencies.should be_empty
+      end
+
+      it "should give an empty list of manifests" do
+        resolution.manifests.should be_empty
+      end
+    end
+
+    context "a mock lockfile with one source and one dependency" do
+      let(:lockfile) do
+        Helpers.strip_heredoc <<-LOCKFILE
+          MOCK
+            remote: source-a
+            specs:
+              jelly (1.3.5)
+
+          DEPENDENCIES
+            jelly (!= 1.2.6, ~> 1.1)
+
+        LOCKFILE
+      end
+
+      it "should give a list of one dependency" do
+        resolution.should have(1).dependencies
+      end
+
+      it "should give a dependency with the expected name" do
+        dependency = resolution.dependencies.first
+
+        dependency.name.should == "jelly"
+      end
+
+      it "should give a dependency with the expected requirement" do
+        dependency = resolution.dependencies.first
+
+        # Note: it must be this order because this order is lexicographically sorted.
+        dependency.requirement.to_s.should == "!= 1.2.6, ~> 1.1"
+      end
+
+      it "should give a dependency wth the expected source" do
+        dependency = resolution.dependencies.first
+        source = dependency.source
+
+        source.name.should == "source-a"
+      end
+
+      it "should give a list of one manifest" do
+        resolution.should have(1).manifests
+      end
+
+      it "should give a manifest with the expected name" do
+        manifest = resolution.manifests.first
+
+        manifest.name.should == "jelly"
+      end
+
+      it "should give a manifest with the expected version" do
+        manifest = resolution.manifests.first
+
+        manifest.version.to_s.should == "1.3.5"
+      end
+
+      it "should give a manifest with no dependencies" do
+        manifest = resolution.manifests.first
+
+        manifest.dependencies.should be_empty
+      end
+
+      it "should give a manifest with the expected source" do
+        manifest = resolution.manifests.first
+        source = manifest.source
+
+        source.name.should == "source-a"
+      end
+
+      it "should give the dependency and the manifest the same source instance" do
+        dependency = resolution.dependencies.first
+        manifest = resolution.manifests.first
+
+        dependency_source = dependency.source
+        manifest_source = manifest.source
+
+        manifest_source.should be dependency_source
+      end
+    end
+
+    context "a mock lockfile with one source and a complex dependency" do
+      let(:lockfile) do
+        Helpers.strip_heredoc <<-LOCKFILE
+          MOCK
+            remote: source-a
+            specs:
+              butter (2.5.3)
+              jelly (1.3.5)
+                butter (< 3, >= 1.1)
+
+          DEPENDENCIES
+            jelly (!= 1.2.6, ~> 1.1)
+
+        LOCKFILE
+      end
+
+      it "should give a list of one dependency" do
+        resolution.should have(1).dependencies
+      end
+
+      it "should have the expected dependency" do
+        dependency = resolution.dependencies.first
+
+        dependency.name.should == "jelly"
+      end
+
+      it "should give a list of all the manifests" do
+        resolution.should have(2).manifests
+      end
+
+      it "should include all the expected manifests" do
+        manifests = ManifestSet.new(resolution.manifests)
+
+        manifests.to_hash.keys.should =~ %w(butter jelly)
+      end
+
+      it "should have an internally consistent set of manifests" do
+        manifests = ManifestSet.new(resolution.manifests)
+
+        manifests.should be_consistent
+      end
+
+      it "should have an externally consistent set of manifests" do
+        dependencies = resolution.dependencies
+        manifests = ManifestSet.new(resolution.manifests)
+
+        manifests.should be_in_compliance_with dependencies
+      end
+    end
+
+  end
+end

data/vendor/librarian/spec/unit/lockfile_spec.rb

@@ -0,0 +1,65 @@
+require 'librarian'
+require 'librarian/mock'
+
+module Librarian
+  describe Lockfile do
+
+    let(:env) { Mock::Environment.new }
+
+    before do
+      env.registry :clear => true do
+        source 'source-1' do
+          spec 'alpha', '1.1'
+        end
+      end
+    end
+
+    let(:spec) do
+      env.dsl do
+        src 'source-1'
+        dep 'alpha', '1.1'
+      end
+    end
+
+    let(:resolver) { env.resolver }
+    let(:resolution) { resolver.resolve(spec) }
+
+    context "sanity" do
+      context "the resolution" do
+        subject { resolution }
+
+        it { should be_correct }
+        it { should have(1).manifests }
+      end
+    end
+
+    describe "#save" do
+      let(:lockfile) { env.ephemeral_lockfile }
+      let(:lockfile_text) { lockfile.save(resolution) }
+
+      context "just saving" do
+        it "should return the lockfile text" do
+          lockfile_text.should_not be_nil
+        end
+      end
+
+      context "saving and reloading" do
+        let(:reloaded_resolution) { lockfile.load(lockfile_text) }
+
+        it "should have the expected manifests" do
+          reloaded_resolution.manifests.count.should == resolution.manifests.count
+        end
+      end
+
+      context "bouncing" do
+        let(:bounced_resolution) { lockfile.load(lockfile_text) }
+        let(:bounced_lockfile_text) { lockfile.save(bounced_resolution) }
+
+        it "should return the same lockfile text after bouncing as before bouncing" do
+          bounced_lockfile_text.should == lockfile_text
+        end
+      end
+    end
+
+  end
+end