deriving_license 0.2.8 → 0.2.9

data/.travis.yml

@@ -0,0 +1,6 @@
+language: ruby
+rvm:
+  - "1.9.3"
+  - "2.0.0"
+# uncomment this line if your project needs to run something other than `rake`:
+# script: bundle exec rspec spec

data/README.md

@@ -9,6 +9,8 @@ Strategies:
 * from\_scraping\_homepage
 * from\_parsing\_readme
 
+[![Build Status](https://travis-ci.org/Schwolop/deriving_license.png)](https://travis-ci.org/Schwolop/deriving_license)
+
 Example output:
 
     $ deriving_license ~/Code/rails_sample_app/Gemfile
@@ -34,4 +36,4 @@ Example output:
 	Ruby: Ruby License (6 instances)[http://www.ruby-lang.org/en/about/license.txt]
 	BSD: FreeBSD Copyright (2 instances)[http://www.freebsd.org/copyright/freebsd-license.html]
 	GPL: GNU General Public License (2 instances)[http://en.wikipedia.org/wiki/GNU_General_Public_License]
-	The following dependencies have custom licenses: adt, bcrypt-ruby, bootstrap-sass, rack-protection, sass-rails, coffee-rails, sqlite3
+	The following dependencies have custom licenses: adt, bcrypt-ruby, bootstrap-sass, rack-protection, sqlite3

data/bin/deriving_license

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 
 require 'deriving_license'
-licenses = DerivingLicense.run(ARGV[0])
+  licenses = DerivingLicense.run(ARGV[0])
 print "\n"
 DerivingLicense.describe(licenses)

data/deriving_license.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name        = 'deriving_license'
-  s.version     = '0.2.8'
+  s.version     = '0.2.9'
   s.summary     = "Deriving Licence finds the license agreements for all gems in your Gemfile"
   s.description = "Deriving Licence finds the license agreements for all gems in your Gemfile if included in your project, or in a Gemfile passed to the included binary"
   s.authors     = ["Tom Allen"]

data/lib/deriving_license.rb

@@ -57,6 +57,11 @@ class DerivingLicense
       raise "Invalid path to gemfile or gemspec."
     end
     
+    strategies.each do |s|
+      unless @strategies.include?(s)
+        raise "Supplied strategies must be from the following list: [#{@strategies.join(', ')}]"
+      end
+    end unless strategies.nil?
     available_strategies = strategies.nil? ? @strategies : strategies
     
     if /(gemspec)+/.match(path.downcase)

data/test/requires_scraping_but_invalid_homepage.gemfile

@@ -0,0 +1,3 @@
+source "https://rubygems.org"
+
+gem "adt" # Doesn't specify its homepage.

data/test/test_deriving_license.rb

@@ -40,7 +40,7 @@ class DerivingLicenseTest < Test::Unit::TestCase
     assert_equal( true, DerivingLicense.run("./test/empty.gemfile").empty? )
   end
   
-  def test_run_with_valid_arg
+  def test_run_with_valid_gemfile_arg
     assert_nothing_raised do
       output = capture_stdout do
         @result = DerivingLicense.run("Gemfile")
@@ -49,6 +49,23 @@ class DerivingLicenseTest < Test::Unit::TestCase
     assert_equal( true, @result.has_key?("MIT") )
   end
   
+  def test_run_with_valid_gemspec_arg
+    assert_nothing_raised do
+      output = capture_stdout do
+        @result = DerivingLicense.run("deriving_license.gemspec")
+      end
+    end
+    assert_equal( true, @result.has_key?("MIT") )
+  end
+  
+  def test_run_with_non_existant_strategies
+    assert_raise RuntimeError do
+      output = capture_stdout do
+        DerivingLicense.run("Gemfile", ["cheese_strategy"])
+      end
+    end
+  end
+  
   def test_describe_with_known_license
     output = capture_stdout do
       DerivingLicense.describe({"MIT" => 1})
@@ -66,6 +83,13 @@ class DerivingLicenseTest < Test::Unit::TestCase
     assert_equal( false, /unrecognized/.match( output.string ).nil? )
   end
   
+  def test_describe_with_custom_license
+    output = capture_stdout do
+      DerivingLicense.describe( {"custom" => ["fake-gem"]} )
+    end
+    assert_equal( false, /have custom license/i.match( output.string ).nil? )
+  end
+  
   def test_from_scraping_strategy
     output = capture_stdout do
       @result = DerivingLicense.run("./test/requires_scraping.gemfile", ["from_scraping_homepage"])
@@ -73,6 +97,14 @@ class DerivingLicenseTest < Test::Unit::TestCase
     assert_equal( false, @result.empty? )
     assert_equal( false, /from_scraping_homepage strategy...SUCCESS/.match( output.string ).nil? ) # Should be SUCCESS
   end
+  
+  def test_from_scraping_strategy_with_invalid_homepage
+    output = capture_stdout do
+      @result = DerivingLicense.run("./test/requires_scraping_but_invalid_homepage.gemfile", ["from_scraping_homepage"])
+    end
+    assert_equal( true, @result.empty? )
+    assert_equal( false, /from_scraping_homepage strategy...FAILED/.match( output.string ).nil? ) # Should be FAILED
+  end
 
   def test_from_license_filename
     output = capture_stdout do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deriving_license
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.2.9
   prerelease: 
 platform: ruby
 authors:
@@ -84,6 +84,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.md
@@ -97,6 +98,7 @@ files:
 - test/requires_license_filename.gemfile
 - test/requires_readme_file_parsing.gemfile
 - test/requires_scraping.gemfile
+- test/requires_scraping_but_invalid_homepage.gemfile
 - test/test_deriving_license.rb
 homepage: http://www.github.com/Schwolop/deriving_license
 licenses:
@@ -130,5 +132,6 @@ test_files:
 - test/requires_license_filename.gemfile
 - test/requires_readme_file_parsing.gemfile
 - test/requires_scraping.gemfile
+- test/requires_scraping_but_invalid_homepage.gemfile
 - test/test_deriving_license.rb
 has_rdoc: