license_finder 0.8.1-java → 0.8.2-java

data/release.md

@@ -2,10 +2,15 @@
 
 Build the gem for both ruby and jruby (use a later version of each ruby, if desired)
 
+
+The first time you rvm install jruby, you may also have to bundle. This will require you to delete
+any existing Gemfile.lock in the directory.
+
+
 ```sh
-$ rvm use jruby-1.7.3-d19
+$ rvm use jruby-1.7.4
 $ rake build
-$ rvm use ruby-1.9.3-p392
+$ rvm use ruby-2.0.0
 $ rake build
 ```
 

data/spec/lib/license_finder/bundle_spec.rb

@@ -26,20 +26,15 @@ module LicenseFinder
       end
     end
 
-    describe '.current_gem_dependencies' do
+    describe '.current_gems' do
       subject do
-        Bundle.current_gem_dependencies(definition)
+        Bundle.current_gems(definition)
       end
 
       it "should have 2 dependencies" do
         subject.size.should == 2
       end
 
-      it "returns persisted dependencies" do
-        subject.first.id.should be
-        subject.last.id.should be
-      end
-
       context "when initialized with a parent and child gem" do
         before do
           definition.stub(:specs_for).and_return([
@@ -49,11 +44,9 @@ module LicenseFinder
         end
 
         it "should update the child dependency with its parent data" do
-          gem1 = subject.first.reload
-          gem2 = subject.last.reload
+          gem1 = subject.first
 
-          gem2.parents.should == [gem1]
-          gem1.children.should == [gem2]
+          gem1.children.should == ["gem2"]
         end
       end
     end

data/spec/lib/license_finder/{gem_saver_spec.rb → bundled_gem_saver_spec.rb}

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 module LicenseFinder
-  describe GemSaver do
+  describe BundledGemSaver do
     let(:gemspec) do
       Gem::Specification.new do |s|
         s.name = 'spec_name'
@@ -30,6 +30,7 @@ module LicenseFinder
 
         it "associates children" do
           subject.children.map(&:name).should == ['foo']
+          subject.children.each { |child| child.approval.should be }
         end
 
         it "marks depenency as unapproved by default" do
@@ -37,7 +38,7 @@ module LicenseFinder
         end
 
         context "with a bundler dependency" do
-          let(:bundled_gem) { BundledGem.new(gemspec, stub(:bundler_dependency, groups: %w[1 2 3]))}
+          let(:bundled_gem) { BundledGem.new(gemspec, double(:bundler_dependency, groups: %w[1 2 3]))}
 
           it "saves the bundler groups" do
             subject.bundler_groups.map(&:name).should =~ %w[1 2 3]
@@ -47,13 +48,15 @@ module LicenseFinder
 
       context "when the dependency already existed" do
         let!(:old_copy) do
-          Dependency.create(
+          dep = Dependency.create(
             name: 'spec_name',
             version: '0.1.2',
             summary: 'old summary',
             description: 'old desription',
             homepage: 'old homepage'
           )
+          dep.approval = Approval.create
+          dep
         end
 
         it "merges in the latest data" do
@@ -85,7 +88,7 @@ module LicenseFinder
         end
 
         context "with a bundler dependency" do
-          let(:bundled_gem) { BundledGem.new(gemspec, stub(:bundler_dependency, groups: %w[1 2 3]))}
+          let(:bundled_gem) { BundledGem.new(gemspec, double(:bundler_dependency, groups: %w[1 2 3]))}
 
           before do
             old_copy.add_bundler_group BundlerGroup.find_or_create(name: 'a')

data/spec/lib/license_finder/bundled_gem_spec.rb

@@ -54,7 +54,7 @@ module LicenseFinder
 
     describe "#license_files" do
       it "delegates to the license files helper" do
-        LicenseFiles.should_receive(:new).with(gemspec.full_gem_path) { stub(files: [] )}
+        PossibleLicenseFiles.should_receive(:new).with(gemspec.full_gem_path) { double(find: [] )}
         subject.license_files
       end
     end

data/spec/lib/license_finder/bundler_group_manager_spec.rb

@@ -0,0 +1,60 @@
+require "spec_helper"
+
+module LicenseFinder
+  describe BundlerGroupManager do
+    let(:config) { Configuration.new }
+
+    before do
+      LicenseFinder.stub(:config).and_return config
+      config.ignore_groups = ignore_groups
+    end
+
+    describe ".add_ignored_group" do
+      describe "when the group is already ignored" do
+        let(:ignore_groups) { ["test", "other_group"] }
+
+        it "does not create a duplicate entry" do
+          config.should_not_receive(:save_to_yaml)
+
+          described_class.add_ignored_group("test")
+        end
+      end
+
+      describe "when the group is not ignored" do
+        let(:ignore_groups) { [] }
+
+        it "adds the group to the ignored bundler groups" do
+          config.should_receive(:save_to_yaml)
+
+          described_class.add_ignored_group("test")
+
+          config.ignore_groups.should include(:test)
+        end
+      end
+    end
+
+    describe ".remove_ignored_group" do
+      describe "when the group is not ignored" do
+        let(:ignore_groups) { [] }
+
+        it "does not call save_to_yaml on config" do
+          config.should_not_receive(:save_to_yaml)
+
+          described_class.remove_ignored_group("test")
+        end
+      end
+
+      describe "when the group is already ignored" do
+        let(:ignore_groups) { ["test", "other_group"] }
+
+        it "removes the group from the ignored bundler group list" do
+          config.should_receive(:save_to_yaml)
+
+          described_class.remove_ignored_group("test")
+
+          config.ignore_groups.should_not include(:test)
+        end
+      end
+    end
+  end
+end

data/spec/lib/license_finder/cli_spec.rb

@@ -1,36 +1,136 @@
 require "spec_helper"
 
 module LicenseFinder
-  describe CLI do
-    describe "#execute!(options)" do
-      before { CLI.stub(:check_for_action_items) }
+  module CLI
+    describe Dependencies do
+      describe "add" do
+        it "adds a dependency" do
+          DependencyManager.should_receive(:create_non_bundler).with("MIT", "js_dep", "1.2.3")
 
-      context "when the approve option is provided" do
-        it "should approve the requested gem" do
-          dependency = double('dependency', :name => nil)
-          dependency.should_receive(:approve!)
+          silence_stdout do
+            subject.add("MIT", "js_dep", "1.2.3")
+          end
+        end
 
-          Dependency.stub(:first).with(name: 'foo').and_return(dependency)
+        it "does not require a version" do
+          DependencyManager.should_receive(:create_non_bundler).with("MIT", "js_dep", nil)
 
-          CLI.execute! approve: true, dependency: 'foo'
+          silence_stdout do
+            subject.add("MIT", "js_dep")
+          end
+        end
+
+        it "has an -a option to approve the added dependency" do
+          DependencyManager.should_receive(:create_non_bundler).with("MIT", "js_dep", "1.2.3")
+          DependencyManager.should_receive(:approve!).with("js_dep")
+
+          silence_stdout do
+            LicenseFinder::CLI::Main.start(["dependencies", "add", "--approve", "MIT", "js_dep", "1.2.3"])
+          end
         end
       end
 
-      context "when the -l (--license) switch is provided" do
-        it "should update the license on the requested gem" do
-          dependency = double :dependency, :name => nil
-          dependency.should_receive(:set_license_manually).with("foo")
+      describe "remove" do
+        it "removes a dependency" do
+          DependencyManager.should_receive(:destroy_non_bundler).with("js_dep")
+          silence_stdout do
+            subject.remove("js_dep")
+          end
+        end
+      end
+    end
 
-          Dependency.stub(:first).with(name: "foo_gem").and_return dependency
+    describe IgnoredBundlerGroups do
+      describe "list" do
+        it "shows the ignored groups in the standard output" do
+          LicenseFinder.config.should_receive(:ignore_groups).and_return([])
 
-          CLI.execute! license: "foo", dependency: 'foo_gem'
+          silence_stdout do
+            subject.list
+          end
         end
       end
 
-      context "when no options are provided" do
-        it "should check for action items" do
-          CLI.should_receive(:check_for_action_items)
-          CLI.execute!
+      describe "add" do
+        it "adds the specified group to the ignored groups list" do
+          BundlerGroupManager.should_receive(:add_ignored_group).with("test")
+
+          silence_stdout do
+            subject.add("test")
+          end
+        end
+      end
+
+      describe "remove" do
+        it "removes the specified group from the ignored groups list" do
+          BundlerGroupManager.should_receive(:remove_ignored_group).with("test")
+
+          silence_stdout do
+            subject.remove("test")
+          end
+        end
+      end
+    end
+
+    describe Main do
+      describe "default" do
+        it "checks for action items" do
+          DependencyManager.should_receive(:sync_with_bundler)
+          Dependency.stub(:unapproved) { [] }
+          silence_stdout do
+            described_class.start([])
+          end
+        end
+      end
+
+      describe "#rescan" do
+        it "resyncs with Gemfile" do
+          DependencyManager.should_receive(:sync_with_bundler)
+          Dependency.stub(:unapproved) { [] }
+
+          silence_stdout do
+            subject.rescan
+          end
+        end
+      end
+
+      describe "#license" do
+        it "updates the license on the requested gem" do
+          DependencyManager.should_receive(:license!).with("foo_gem", "foo")
+
+          silence_stdout do
+            subject.license 'foo', 'foo_gem'
+          end
+        end
+      end
+
+      describe "#approve" do
+        it "approves the requested gem" do
+          DependencyManager.should_receive(:approve!).with("foo")
+
+          silence_stdout do
+            subject.approve 'foo'
+          end
+        end
+      end
+
+      describe "#action_items" do
+        it "reports unapproved dependencies" do
+          Dependency.stub(:unapproved) { ['one dependency'] }
+          TextReport.stub(:new) { double(:report, to_s: "a report!") }
+          silence_stdout do
+            $stdout.stub(:puts)
+            $stdout.should_receive(:puts).with(/dependencies/i)
+            expect { subject.action_items }.to raise_error(SystemExit)
+          end
+        end
+
+        it "reports that all dependencies are approved" do
+          Dependency.stub(:unapproved) { [] }
+          silence_stdout do
+            $stdout.should_receive(:puts).with(/approved/i)
+            expect { subject.action_items }.to_not raise_error
+          end
         end
       end
     end

data/spec/lib/license_finder/configuration_spec.rb

@@ -24,9 +24,12 @@ module LicenseFinder
           subject.dependencies_dir.should == attributes['dependencies_file_dir']
         end
       end
+    end
 
-      it "uses absolute path in database_path" do
-        subject.database_path.should_not start_with(".")
+    describe "#database_uri" do
+      it "should URI escape absolute path the dependencies_file_dir" do
+        config = described_class.new('dependencies_file_dir' => 'test path')
+        config.database_uri.should =~ /test%20path\/dependencies\.db$/
       end
     end
 
@@ -56,14 +59,34 @@ module LicenseFinder
       end
     end
 
-    describe "#ignore_groups" do
-      it "should default to an empty array" do
-        config.ignore_groups.should == []
+    describe "#save_to_yaml" do
+      let(:tmp_yml) { '.tmp.configuration_spec.yml' }
+
+      before do
+        Configuration.stub(:config_file_path).and_return(tmp_yml)
+        config.whitelist = ['my_gem']
+        config.ignore_groups = ['other_group', 'test']
       end
 
-      it "should always return symbolized versions of the ignore groups" do
-        config.ignore_groups = %w[test development]
-        config.ignore_groups.should == [:test, :development]
+      after do
+        File.delete(tmp_yml)
+      end
+
+      it "writes the whitelist to the yaml file" do
+        config.save_to_yaml
+
+        yaml = YAML.load(File.read(tmp_yml))
+
+        yaml["whitelist"].should include("my_gem")
+      end
+
+      it "writes the ignored bundler groups to the yaml file" do
+        config.save_to_yaml
+
+        yaml = YAML.load(File.read(tmp_yml))
+
+        yaml["ignore_groups"].should include("other_group")
+        yaml["ignore_groups"].should include("test")
       end
     end
   end

data/spec/lib/license_finder/dependency_manager_spec.rb

@@ -0,0 +1,107 @@
+require 'spec_helper'
+
+module LicenseFinder
+  describe DependencyManager do
+    let(:config) { Configuration.new }
+
+    before do
+      LicenseFinder.stub(:config).and_return config
+      config.whitelist = ["MIT", "other"]
+      Reporter.stub(:write_reports)
+    end
+
+    describe "#sync_with_bundler" do
+      it "destroys every dependency except for the ones Bundler reports as 'current' or are marked as 'manual'" do
+        cur1 = Dependency.create(name: "current dependency 1")
+        cur2 = Dependency.create(name: "current dependency 2")
+        man1 = Dependency.create(name: "manual dependency", manual: true)
+        Dependency.create(name: "old dependency 1")
+        Dependency.create(name: "old dependency 2")
+
+        current_gems = [
+          double(:gem1, save_as_dependency: cur1),
+          double(:gem2, save_as_dependency: cur2)
+        ]
+        Bundle.stub(:current_gems) { current_gems }
+
+        described_class.sync_with_bundler
+        Dependency.all.map(&:name).should =~ [cur1, cur2, man1].map(&:name)
+      end
+    end
+
+    describe ".create_non_bundler" do
+      it "should add a Dependency" do
+        expect do
+          described_class.create_non_bundler("MIT", "js_dep", "0.0.0")
+        end.to change(Dependency, :count).by(1)
+      end
+
+      it "should mark the dependency as manual" do
+        described_class.create_non_bundler("MIT", "js_dep", "0.0.0")
+          .should be_manual
+      end
+
+      it "should set the appropriate values" do
+        dep = described_class.create_non_bundler("GPL", "js_dep", "0.0.0")
+        dep.name.should == "js_dep"
+        dep.version.should == "0.0.0"
+        dep.license.name.should == "GPL"
+        dep.should_not be_approved
+      end
+
+      it "should complain if the dependency already exists" do
+        Dependency.create(name: "current dependency 1")
+        expect { described_class.create_non_bundler("GPL", "current dependency 1", "0.0.0") }
+          .to raise_error(LicenseFinder::Error)
+      end
+    end
+
+    describe ".destroy_non_bundler" do
+      it "should remove a non bundler Dependency" do
+        described_class.create_non_bundler("GPL", "a non-bundler dep", nil)
+        expect do
+          described_class.destroy_non_bundler("a non-bundler dep")
+        end.to change(Dependency, :count).by(-1)
+      end
+
+      it "should not remove a bundler Dependency" do
+        Dependency.create(name: "a bundler dep")
+        expect do
+          expect do
+            described_class.destroy_non_bundler("a bundler dep")
+          end.to raise_error(LicenseFinder::Error)
+        end.to_not change(Dependency, :count)
+      end
+    end
+
+    describe ".approve!" do
+      it "approves the dependency" do
+        dep = Dependency.named("current dependency")
+        dep.reload.should_not be_approved
+        described_class.approve!("current dependency")
+        dep.reload.should be_approved
+      end
+
+      it "should raise an error if it can't find the dependency" do
+        expect { described_class.approve!("non-existent dependency") }
+          .to raise_error(LicenseFinder::Error)
+      end
+    end
+
+    describe ".license!" do
+      it "adds a license for the dependency" do
+        dep = described_class.create_non_bundler("old license", "current dependency", nil)
+        dep.reload.license.name.should == "old license"
+        described_class.license!("current dependency", "a license")
+        dep.reload.license.name.should == "a license"
+      end
+
+      it "should raise an error if it can't find the dependency" do
+        expect { described_class.license!("non-existent dependency", "a license") }
+          .to raise_error(LicenseFinder::Error)
+      end
+    end
+
+  end
+end
+