doubleshot 0.1.0-java → 0.2.0-java

data/test/dependencies/dependency_list_spec.rb

@@ -13,6 +13,28 @@ describe Doubleshot::Dependencies::DependencyList do
     @list.must_be_kind_of Enumerable
   end
 
+  describe "+" do
+    it "must allow you to concatenate two collections" do
+      one = Doubleshot::Dependencies::DependencyList.new
+      two = Doubleshot::Dependencies::DependencyList.new
+
+      listen    = one.fetch "listen"
+      minitest  = two.fetch "minitest"
+
+      (one + two).entries.must_equal [ listen, minitest ]
+    end
+
+    it "must raise an ArgumentError if the lists are not the same type" do
+      one = Doubleshot::Dependencies::DependencyList.new
+      two = Doubleshot::Dependencies::GemDependencyList.new
+
+      listen    = one.fetch "listen"
+      minitest  = two.fetch "minitest"
+
+      -> { one + two }.must_raise ArgumentError
+    end
+  end
+
   describe "add" do
     it "must only accept Dependency instances" do
       assert_raises(ArgumentError) do
@@ -30,18 +52,18 @@ describe Doubleshot::Dependencies::DependencyList do
       @list.add @dependency
       @list.size.must_equal 1
     end
-    
+
     it "must always return self for chainability" do
       @list.add(@dependency).must_equal @list
     end
   end
-  
+
   describe "fetch" do
     it "must find matching dependencies by name" do
       @list.add @dependency
       @list.fetch("listen").must_equal @dependency
     end
-    
+
     it "must always return a dependency" do
       @list.fetch("example").must_be_kind_of Doubleshot::Dependencies::Dependency
     end
@@ -70,4 +92,4 @@ describe Doubleshot::Dependencies::DependencyList do
     end
   end
 
-end
+end

data/test/dependencies/dependency_spec.rb

@@ -9,18 +9,6 @@ describe Doubleshot::Dependencies::Dependency do
     @other      = Doubleshot::Dependencies::Dependency.new "listen"
   end
 
-  describe "add_requirement" do
-    it "must return a Gem::Requirement object" do
-      @dependency.add_requirement("1.0").must_be_kind_of Gem::Requirement
-    end
-
-    it "must not duplicate requirements" do
-      @dependency.add_requirement("2.0")
-      @dependency.add_requirement("2.0")
-      @dependency.requirements.size.must_equal 1
-    end
-  end
-
   describe "Hash contract" do
 
     # Dependency#name must be immutable since
@@ -38,12 +26,25 @@ describe Doubleshot::Dependencies::Dependency do
     end
   end
 
-  describe "equality" do
+  describe "lock" do
+    it "must lock to a specific version" do
+      @dependency.lock "0.5.3"
+      @dependency.must_be :locked?
+    end
+  end
+
+  describe "to_s" do
+    before do
+      @dependency.lock "0.5.3"
+    end
+
+    it "must have a short-form" do
+      @dependency.to_s.must_equal "listen"
+    end
 
-    it "must override the equality operator to consider requirements" do
-      @dependency.must_be :==, @other
-      @other.add_requirement ">= 0.1.0"
-      @dependency.wont_be :==, @other
+    it "must have a long-form" do
+      @dependency.to_s(true).must_equal "listen (0.5.3)"
     end
   end
-end
+
+end

data/test/dependencies/gem_dependency_spec.rb

@@ -4,4 +4,58 @@ require_relative "../helper"
 
 describe Doubleshot::Dependencies::GemDependency do
 
+  before do
+    @dependency = Doubleshot::Dependencies::GemDependency.new "listen"
+    @other      = Doubleshot::Dependencies::GemDependency.new "listen"
+  end
+
+  describe "add_requirement" do
+    it "must return a Gem::Requirement object" do
+      @dependency.add_requirement("1.0").must_be_kind_of Gem::Requirement
+    end
+
+    it "must not duplicate requirements" do
+      @dependency.add_requirement("2.0")
+      @dependency.add_requirement("2.0")
+      @dependency.requirements.size.must_equal 1
+    end
+  end
+
+  describe "equality" do
+    it "must override the equality operator to consider requirements" do
+      @dependency.must_be :==, @other
+      @other.add_requirement ">= 0.1.0"
+      @dependency.wont_be :==, @other
+    end
+  end
+
+  describe "to_s" do
+    before do
+      @dependency.add_requirement ">= 0.5"
+      @dependency.add_requirement "= 0.5.3"
+    end
+
+    it "must have a short-form" do
+      @dependency.to_s.must_equal "listen"
+    end
+
+    it "must have a long-form that includes requirements" do
+      @dependency.to_s(true).must_equal "listen (= 0.5.3, >= 0.5)"
+    end
+  end
+
+  describe "gemspec" do
+    it "must return a Gem::Specification" do
+      skip
+      @dependency.gemspec = Gem::Specification.new
+      @dependency.gemspec.must_be_kind_of Gem::Specification
+    end
+  end
+
+  describe "dependencies" do
+    it "must return a Doubleshot::Dependencies::DependencyList" do
+      skip
+      @dependency.dependencies.must_be_kind_of Doubleshot::Dependencies::DependencyList
+    end
+  end
 end

data/test/dependencies/jar_dependency_spec.rb

@@ -3,5 +3,66 @@
 require_relative "../helper"
 
 describe Doubleshot::Dependencies::JarDependency do
+  # ch.qos.logback:logback-core:1.0.6
+  # ch.qos.logback:logback-core:jar:1.0.6
+  # ch.qos.logback:logback-core:jar:someclassifier:1.0.6
+  # <#JarDependency type: [pom|jar|maven-plugin|ejb|war|ear|rar|par|bundle] artifact: "logback-core" group: "ch.qos.logback" version: "1.0.6" classifier: "someclassifier">
 
-end
+  it "must only accept Maven coordinate syntax" do
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new "ch.qos.logback:logback-core:jar"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new "ch.qos.logback"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new "ch.qos.logback:logback-core:jar:"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new "ch.qos.logback:logback-core::1.0.6"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new "ch.qos.logback::jar:1.0.6"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new ":logback-core:jar:1.0.6"
+    end
+
+    assert_raises(ArgumentError) do
+      Doubleshot::Dependencies::JarDependency.new ":::"
+    end
+  end
+
+  it "returns a Maven coordinate as the name" do
+    Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:jar:1.0.6")
+      .name.must_equal "ch.qos.logback:logback-core:jar:1.0.6"
+  end
+
+  it "correctly parses the Maven coordinate" do
+    dependency = Doubleshot::Dependencies::JarDependency.new "ch.qos.logback:logback-core:jar:1.0.6"
+    dependency.packaging.must_equal "jar"
+    dependency.artifact.must_equal "logback-core"
+    dependency.group.must_equal "ch.qos.logback"
+    dependency.version.must_equal "1.0.6"
+  end
+
+  it "returns a Maven coordinate as the long form of to_s" do
+    Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:jar:1.0.6")
+      .to_s(true).must_equal "ch.qos.logback:logback-core:jar:1.0.6"
+  end
+
+  it "returns a 4-part Maven coordinate as the name if you used a 3-part Maven coordinate to create the dependency" do
+    Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:1.0.6")
+      .name.must_equal "ch.qos.logback:logback-core:jar:1.0.6"
+  end
+
+  it "returns a 4-part Maven coordinate as the long form of to_s if you used a 3-part Maven coordinate to create the dependency" do
+    Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:1.0.6")
+      .to_s(true).must_equal "ch.qos.logback:logback-core:jar:1.0.6"
+  end
+end

data/test/dependencies_spec.rb

@@ -10,7 +10,7 @@ describe Doubleshot::Dependencies do
 
   describe "gems" do
     it "must be a DependencyList" do
-      @dependencies.gems.must_be_kind_of Doubleshot::Dependencies::DependencyList 
+      @dependencies.gems.must_be_kind_of Doubleshot::Dependencies::DependencyList
     end
   end
 
@@ -27,16 +27,16 @@ describe Doubleshot::Dependencies do
       # NOTE: #fetch will add the dependency if it does
       # not yet exist in the List.
       @dependencies.gems.fetch "listen"
-      @dependencies.jars.fetch "jetty"
+      @dependencies.jars.fetch "commons-cli:commons-cli:jar:1.2"
     end
 
     it "must equal if their DependencyLists are equal" do
       @dependencies.wont_equal @other
 
       @other.gems.fetch "listen"
-      @other.jars.fetch "jetty"
+      @other.jars.fetch "commons-cli:commons-cli:jar:1.2"
 
       @dependencies.must_equal @other
     end
   end
-end
+end

data/test/doubleshot_spec.rb

@@ -4,6 +4,32 @@ require_relative "helper.rb"
 
 describe Doubleshot do
 
+  describe "classpath_cache" do
+    it "must be a Pathname" do
+      Doubleshot.new.classpath_cache.must_be_kind_of Pathname
+    end
+  end
+
+  describe "classpath" do
+    before do
+      @doubleshot = Doubleshot.new do |config|
+        config.jar "ch.qos.logback:logback-core:1.0.6"
+      end
+      Helper::tmp do |tmp|
+        @doubleshot.path = tmp + "Doubleshot.test"
+      end
+    end
+
+    it "must be empty on instantiation" do
+      @doubleshot.classpath.must_be_empty
+    end
+
+    it "won't be empty after setup (if you have JAR dependencies)" do
+      @doubleshot.setup!
+      @doubleshot.classpath.wont_be_empty
+    end
+  end
+
   describe "configuration" do
     it "must pass a Configuration object to the block" do
       called = false
@@ -12,7 +38,7 @@ describe Doubleshot do
         config.must_be_kind_of Doubleshot::Configuration
       end
       assert called, "block not called"
-    end 
+    end
   end
 
   it "must generate a valid gemspec" do
@@ -44,8 +70,14 @@ describe Doubleshot do
       config.source.ruby.to_s.must_equal "lib"
       config.target.to_s.must_equal "target"
     end
+
+    describe "lockfile" do
+      it "must return a Lockfile instance" do
+        Doubleshot::current.lockfile.must_be_kind_of Doubleshot::Lockfile
+      end
+    end
   end
 
 
 
-end
+end

data/test/helper.rb

@@ -1,9 +1,44 @@
-require_relative "../lib/doubleshot"
+require_relative "../lib/doubleshot/setup"
+
+require "doubleshot/resolver"
 
 require "minitest/autorun"
 require "minitest/pride"
 require "minitest/wscolor"
 
+module MiniTest
+  module Assertions
+    def assert_predicate o1, op, msg = nil
+      msg = message(msg) { "Expected #{mu_pp(o1)} to be #{op}" }
+      if !o1.respond_to?(op) && o1.respond_to?("#{op}?")
+        assert o1.__send__("#{op}?"), msg
+      else
+        assert o1.__send__(op), msg
+      end
+    end
+
+    def refute_predicate o1, op, msg = nil
+      msg = message(msg) { "Expected #{mu_pp(o1)} to not be #{op}" }
+      if !o1.respond_to?(op) && o1.respond_to?("#{op}?")
+        refute o1.__send__("#{op}?"), msg
+      else
+        refute o1.__send__(op), msg
+      end
+    end
+  end
+
+  module Expectations
+    # This is for aesthetics, so instead of:
+    #   something.must_be :validate
+    # Or:
+    #   something.validate.must_equal true
+    # Which are both terribly ugly, we can:
+    #   something.must :validate
+    infect_an_assertion :assert_operator, :must, :reverse
+    infect_an_assertion :refute_operator, :wont, :reverse
+  end
+end
+
 module Helper
   def self.tmp(path = "tmp")
     dir = Pathname(path.to_s)

data/test/lockfile_spec.rb

@@ -0,0 +1,236 @@
+#!/usr/bin/env jruby
+
+# encoding: utf-8
+
+require_relative "helper"
+
+describe Doubleshot::Lockfile do
+
+  def lockfile(name = "test.lock")
+    Helper::tmp do |tmp|
+      (tmp + name).open("w+") do |file|
+        file << {}.to_yaml
+      end
+      yield Doubleshot::Lockfile.new(tmp + name)
+    end
+  end
+
+  before do
+    @jar = Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:jar:1.0.6")
+    @gem = Doubleshot::Dependencies::GemDependency.new("listen")
+    @gem.lock("0.5.3")
+  end
+
+  describe "Pathname delegations" do
+    it "#exist?" do
+      lockfile do |lockfile|
+        lockfile.must :exist
+      end
+    end
+
+    it "#mtime" do
+      lockfile do |lockfile|
+        lockfile.mtime.must_be_kind_of Time
+      end
+    end
+
+    it "#delete" do
+      lockfile do |lockfile|
+        lockfile.must :exist
+        lockfile.delete
+        lockfile.wont :exist
+      end
+    end
+  end
+
+  describe "flush!" do
+    it "must write the JarDependencyList to the lockfile on disk such that an equal JarDependencyList will be loaded from that lockfile" do
+      lockfile do |lockfile|
+        lockfile.add Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:jar:1.0.6")
+        lockfile.flush!
+        Doubleshot::Lockfile.new(lockfile.path).jars.must_equal lockfile.jars
+      end
+    end
+  end
+
+  describe "path" do
+    it "must default to 'Doubleshot.lock'" do
+      Doubleshot::Lockfile.new.path.must_equal Pathname("Doubleshot.lock")
+    end
+
+    it "must let you specify a path" do
+      Doubleshot::Lockfile.new("myproject.lock").path.must_equal Pathname("myproject.lock")
+    end
+
+    it "must not create the lockfile on disk implicitly" do
+      Doubleshot::Lockfile.new "myproject.lock"
+      Pathname("myproject.lock").exist?.wont_equal true
+    end
+  end
+
+  describe "add" do
+    it "must not be empty after adding a dependency" do
+      lockfile do |lockfile|
+        lockfile.add @jar
+        lockfile.wont_be_empty
+      end
+    end
+
+    it "must return self" do
+      lockfile do |lockfile|
+        lockfile.add(@jar).must_be_same_as lockfile
+      end
+    end
+
+    it "must raise an ArgumentError if passing an abstract Dependency instance" do
+      assert_raises(ArgumentError) do
+        lockfile do |lockfile|
+          lockfile.add Doubleshot::Dependencies::Dependency.new "listen"
+        end
+      end
+    end
+
+    it "must raise if a non-dependency is passed" do
+      assert_raises(ArgumentError) do
+        lockfile do |lockfile|
+          lockfile.add "listen"
+        end
+      end
+    end
+
+    it "must not add a dependency twice" do
+      lockfile do |lockfile|
+        lockfile.add @jar
+        lockfile.add Doubleshot::Dependencies::JarDependency.new("ch.qos.logback:logback-core:jar:1.0.6")
+        lockfile.jars.size.must_equal 1
+      end
+    end
+
+    it "must reject unlocked dependencies" do
+      assert_raises(Doubleshot::Lockfile::UnlockedDependencyError) do
+        lockfile do |lockfile|
+          lockfile.add Doubleshot::Dependencies::GemDependency.new("listen")
+        end
+      end
+    end
+
+    it "must raise an error if it doesn't know how to handle the dependency you've passed" do
+      assert_raises(Doubleshot::Lockfile::UnknownDependencyTypeError) do
+        dependency = Class.new(Doubleshot::Dependencies::Dependency) do
+          def locked?
+            true
+          end
+        end.new("some-raa-tar")
+
+        lockfile do |lockfile|
+          lockfile.add(dependency)
+        end
+      end
+    end
+  end
+
+  describe "jars" do
+    it "must return the list of JAR dependencies" do
+      lockfile do |lockfile|
+        lockfile.add @jar
+        lockfile.jars.size.must_equal 1
+      end
+    end
+
+    it "must be readonly" do
+      lockfile do |lockfile|
+        lockfile.jars.must_be_kind_of Doubleshot::ReadonlyCollection
+      end
+    end
+  end
+
+  describe "gems" do
+    it "must return the list of Gem dependencies" do
+      lockfile do |lockfile|
+        lockfile.add @gem
+        lockfile.gems.size.must_equal 1
+      end
+    end
+
+    it "must be readonly" do
+      lockfile do |lockfile|
+        lockfile.gems.must_be_kind_of Doubleshot::ReadonlyCollection
+      end
+    end
+  end
+
+  describe "jar format" do
+    before do
+      @lockfile_contents = <<-EOS.margin
+        ---
+        GEMS: []
+        JARS:
+          - com.pyx4j:maven-plugin-log4j:jar:1.0.1
+          - org.springframework:spring-core:jar:3.1.2.RELEASE
+          - org.hibernate:hibernate-core:jar:4.1.7.Final
+      EOS
+    end
+
+    it "must handle proper YAML format" do
+      lockfile "test_good.lock" do |lockfile|
+        lockfile.path.open("w+") do |file|
+          file << @lockfile_contents
+        end
+        lockfile.jars.size.must_equal 3
+      end
+    end
+
+    it "must raise a Psych::SyntaxError for invalid YAML" do
+      lockfile "test_bad.lock" do |lockfile|
+        lockfile.path.open("w+") do |file|
+          file << "<(^.^)> <(^.^<) (>^.^)>" << @lockfile_contents
+        end
+        -> { lockfile.jars }.must_raise Psych::SyntaxError
+      end
+    end
+  end
+
+  describe "gem format" do
+    before do
+      @lockfile_contents = <<-EOS.margin
+        ---
+        GEMS:
+          backports (2.6.4): []
+          ffi (1.0.11): []
+          ffi (1.0.11-java): []
+          hitimes (1.1.1): []
+          hitimes (1.1.1-java): []
+          minitest (3.4.0): []
+          minitest-wscolor (0.0.3):
+          - minitest (>= 2.3.1)
+          multi_json (1.3.6): []
+          path (1.3.1): []
+          perfer (0.2.0):
+          - backports (~> 2.6.3)
+          - ffi (~> 1.0.11)
+          - hitimes (~> 1.1.1)
+          - path (~> 1.3.1)
+        JARS: []
+      EOS
+    end
+
+    it "must handle proper YAML format" do
+      skip
+      lockfile "test_good.lock" do |lockfile|
+        lockfile.path.open("w+") do |file|
+          file << @lockfile_contents
+        end
+        lockfile.gems.size.must_equal 10
+      end
+    end
+
+    it "must raise a Psych::SyntaxError for invalid YAML" do
+      lockfile "test_bad.lock" do |lockfile|
+        lockfile.path.open("w+") do |file|
+          file << "<(^.^)> <(^.^<) (>^.^)>" << @lockfile_contents
+        end
+        -> { lockfile.gems }.must_raise Psych::SyntaxError
+      end
+    end
+  end
+end