licensee 9.12.0 → 9.14.1

data/spec/licensee/matchers/reference_matcher_spec.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::Matchers::Reference do
+  subject { described_class.new(file) }
+
   let(:content) { 'Copyright 2015 Ben Balter' }
   let(:file) { Licensee::ProjectFiles::LicenseFile.new(content, 'LICENSE.txt') }
   let(:license) { Licensee::License.find('gpl-3.0') }
 
-  subject { described_class.new(file) }
-
   %i[title key nickname].each do |variation|
     context "with a license #{variation}" do
       let(:content) { "Licensed under the #{license.send(variation)} license" }

data/spec/licensee/matchers/spdx_matcher_spec.rb

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::Matchers::Spdx do
+  subject { described_class.new(file) }
+
   let(:content) { 'PackageLicenseDeclared: MIT' }
   let(:file) do
     Licensee::ProjectFiles::LicenseFile.new(content, 'LICENSE.spdx')
   end
   let(:mit) { Licensee::License.find('mit') }
   let(:other) { Licensee::License.find('other') }
-  subject { described_class.new(file) }
 
   it 'matches' do
     expect(subject.match).to eql(mit)
   end
 
   it 'has a confidence' do
-    expect(subject.confidence).to eql(90)
+    expect(subject.confidence).to be(90)
   end
 
   context 'no license field' do

data/spec/licensee/project_files/license_file_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::ProjectFiles::LicenseFile do
+  subject { described_class.new(content, filename) }
+
   let(:filename) { 'LICENSE.txt' }
   let(:gpl) { Licensee::License.find('gpl-3.0') }
   let(:mit) { Licensee::License.find('mit') }
   let(:content) { sub_copyright_info(mit) }
   let(:content_hash) { license_hashes['mit'] }
 
-  subject { described_class.new(content, filename) }
-
   it 'parses the attribution' do
     expect(subject.attribution).to eql('Copyright (c) 2018 Ben Balter')
   end
@@ -47,7 +47,7 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
   end
 
   it 'creates the wordset' do
-    expect(subject.wordset.count).to eql(91)
+    expect(subject.wordset.count).to be(92)
     expect(subject.wordset.first).to eql('permission')
   end
 
@@ -57,32 +57,33 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
 
   context 'filename scoring' do
     {
-      'license'            => 1.00,
-      'LICENCE'            => 1.00,
-      'unLICENSE'          => 1.00,
-      'unlicence'          => 1.00,
-      'license.md'         => 0.95,
-      'LICENSE.md'         => 0.95,
-      'license.txt'        => 0.95,
-      'COPYING'            => 0.90,
-      'copyRIGHT'          => 0.90,
-      'COPYRIGHT.txt'      => 0.85,
-      'copying.txt'        => 0.85,
-      'LICENSE.php'        => 0.80,
-      'LICENCE.docs'       => 0.80,
-      'copying.image'      => 0.75,
-      'COPYRIGHT.go'       => 0.75,
-      'LICENSE-MIT'        => 0.70,
-      'LICENSE_1_0.txt'    => 0.70,
-      'COPYING-GPL'        => 0.65,
-      'COPYRIGHT-BSD'      => 0.65,
-      'MIT-LICENSE.txt'    => 0.60,
-      'mit-license-foo.md' => 0.60,
-      'OFL.md'             => 0.50,
-      'ofl.textile'        => 0.45,
-      'ofl'                => 0.40,
-      'not-the-ofl'        => 0.00,
-      'README.txt'         => 0.00
+      'license'             => 1.00,
+      'LICENCE'             => 1.00,
+      'unLICENSE'           => 1.00,
+      'unlicence'           => 1.00,
+      'license.md'          => 0.95,
+      'LICENSE.md'          => 0.95,
+      'license.txt'         => 0.95,
+      'COPYING'             => 0.90,
+      'copyRIGHT'           => 0.90,
+      'COPYRIGHT.txt'       => 0.85,
+      'copying.txt'         => 0.85,
+      'LICENSE.php'         => 0.80,
+      'LICENCE.docs'        => 0.80,
+      'copying.image'       => 0.75,
+      'COPYRIGHT.go'        => 0.75,
+      'LICENSE-MIT'         => 0.70,
+      'LICENSE_1_0.txt'     => 0.70,
+      'COPYING-GPL'         => 0.65,
+      'COPYRIGHT-BSD'       => 0.65,
+      'MIT-LICENSE.txt'     => 0.60,
+      'mit-license-foo.md'  => 0.60,
+      'OFL.md'              => 0.50,
+      'ofl.textile'         => 0.45,
+      'ofl'                 => 0.40,
+      'not-the-ofl'         => 0.00,
+      'README.txt'          => 0.00,
+      '.pip-license-ignore' => 0.00
     }.each do |filename, expected|
       context "a file named #{filename}" do
         let(:score) { described_class.name_score(filename) }
@@ -119,11 +120,11 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
       end
 
       it 'does not match .md2' do
-        expect(described_class::PREFERRED_EXT_REGEX).to_not match('.md2')
+        expect(described_class::PREFERRED_EXT_REGEX).not_to match('.md2')
       end
 
       it 'does not match .md/foo' do
-        expect(described_class::PREFERRED_EXT_REGEX).to_not match('.md/foo')
+        expect(described_class::PREFERRED_EXT_REGEX).not_to match('.md/foo')
       end
     end
 
@@ -133,7 +134,7 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
       end
 
       it 'does not match .md/foo' do
-        expect(described_class::OTHER_EXT_REGEX).to_not match('.md/foo')
+        expect(described_class::OTHER_EXT_REGEX).not_to match('.md/foo')
       end
     end
 
@@ -158,11 +159,11 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
     let(:regex) { Licensee::ProjectFiles::LicenseFile::CC_FALSE_POSITIVE_REGEX }
 
     it "knows MIT isn't a potential false positive" do
-      expect(subject.content).to_not match(regex)
-      expect(subject).to_not be_a_potential_false_positive
+      expect(subject.content).not_to match(regex)
+      expect(subject).not_to be_a_potential_false_positive
     end
 
-    context 'a CC false positive without creative commons in the title' do
+    context 'a CC false positive with creative commons in the title' do
       let(:content) { 'Creative Commons Attribution-NonCommercial 4.0' }
 
       it "knows it's a potential false positive" do
@@ -191,10 +192,10 @@ RSpec.describe Licensee::ProjectFiles::LicenseFile do
 
     context 'CC-BY-ND with leading instructions' do
       let(:content) do
-        <<-LICENSE
-Creative Commons Corporation ("Creative Commons") is not a law firm
-======================================================================
-Creative Commons Attribution-NonCommercial 4.0
+        <<~LICENSE
+          Creative Commons Corporation ("Creative Commons") is not a law firm
+          ======================================================================
+          Creative Commons Attribution-NonCommercial 4.0
         LICENSE
       end
 
@@ -220,7 +221,7 @@ Creative Commons Attribution-NonCommercial 4.0
         let(:content) { sub_copyright_info(mit) }
 
         it 'is not lgpl' do
-          expect(subject).to_not be_lgpl
+          expect(subject).not_to be_lgpl
         end
       end
     end
@@ -229,7 +230,7 @@ Creative Commons Attribution-NonCommercial 4.0
       let(:filename) { 'COPYING' }
 
       it 'is not lgpl' do
-        expect(subject).to_not be_lgpl
+        expect(subject).not_to be_lgpl
       end
     end
   end
@@ -245,7 +246,7 @@ Creative Commons Attribution-NonCommercial 4.0
       let(:content) { sub_copyright_info(mit) }
 
       it 'is not GPL' do
-        expect(subject).to_not be_gpl
+        expect(subject).not_to be_gpl
       end
     end
   end

data/spec/licensee/project_files/package_info_spec.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::ProjectFiles::PackageManagerFile do
+  subject { described_class.new(content, filename) }
+
   let(:content) { '' }
   let(:filename) { '' }
-  subject { described_class.new(content, filename) }
 
   context 'name scoring' do
     {
@@ -19,6 +20,7 @@ RSpec.describe Licensee::ProjectFiles::PackageManagerFile do
     }.each do |filename, expected_score|
       context "a file named #{filename}" do
         let(:score) { described_class.name_score(filename) }
+
         it 'scores the file' do
           expect(score).to eql(expected_score)
         end
@@ -69,5 +71,13 @@ RSpec.describe Licensee::ProjectFiles::PackageManagerFile do
         expect(possible_matchers).to eql([Licensee::Matchers::Cran])
       end
     end
+
+    context 'with nuspec file' do
+      let(:filename) { 'foo.nuspec' }
+
+      it 'returns the NuGet matcher' do
+        expect(possible_matchers).to eql([Licensee::Matchers::NuGet])
+      end
+    end
   end
 end

data/spec/licensee/project_files/project_file_spec.rb

@@ -1,16 +1,19 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::ProjectFiles::ProjectFile do
+  subject { Licensee::ProjectFiles::LicenseFile.new(content, filename) }
+
   let(:filename) { 'LICENSE.txt' }
   let(:mit) { Licensee::License.find('mit') }
   let(:content) { mit.content }
   let(:possible_matchers) { [Licensee::Matchers::Exact] }
 
-  subject { Licensee::ProjectFiles::LicenseFile.new(content, filename) }
   before do
     allow(subject).to receive(:possible_matchers).and_return(possible_matchers)
   end
+
   before { allow(subject).to receive(:length).and_return(mit.length) }
+
   before { allow(subject).to receive(:wordset).and_return(mit.wordset) }
 
   it 'stores the content' do
@@ -26,7 +29,7 @@ RSpec.describe Licensee::ProjectFiles::ProjectFile do
   end
 
   it 'returns the confidence' do
-    expect(subject.confidence).to eql(100)
+    expect(subject.confidence).to be(100)
   end
 
   it 'returns the license' do

data/spec/licensee/project_files/readme_file_spec.rb

@@ -1,9 +1,10 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::ProjectFiles::ReadmeFile do
+  subject { described_class.new(content, filename) }
+
   let(:filename) { 'README.md' }
   let(:content) { '' }
-  subject { described_class.new(content, filename) }
 
   context 'scoring names' do
     {

data/spec/licensee/project_spec.rb

@@ -6,13 +6,14 @@
   Licensee::Projects::GitHubProject
 ].each do |project_type|
   RSpec.describe project_type do
+    subject { described_class.new(path) }
+
     let(:stubbed_org) { '_licensee_test_fixture' }
     let(:mit) { Licensee::License.find('mit') }
     let(:other) { Licensee::License.find('other') }
     let(:fixture) { 'mit' }
     let(:path) { fixture_path(fixture) }
     let(:api_base) { 'https://api.github.com/repos' }
-    subject { described_class.new(path) }
 
     if described_class == Licensee::Projects::GitProject
       before do
@@ -46,14 +47,16 @@
             .to_return(status: 200, body: File.read(file))
         end
       end
+
       let(:path) { "https://github.com/#{stubbed_org}/#{fixture}" }
     end
 
     if described_class == Licensee::Projects::GitProject
       context 'when initialized with a repo' do
-        let(:repo) { Rugged::Repository.new(path) }
         subject { described_class.new(repo) }
 
+        let(:repo) { Rugged::Repository.new(path) }
+
         it 'returns the repository' do
           expect(subject.repository).to be_a(Rugged::Repository)
         end
@@ -61,6 +64,7 @@
 
       context 'when initialized with a revision' do
         let(:revision) { subject.repository.last_commit.oid }
+
         before do
           subject.instance_variable_set('@revision', revision)
         end
@@ -98,9 +102,9 @@
       let(:files) { subject.send(:files) }
 
       it 'returns the file list' do
-        expect(files.count).to eql(2)
+        expect(files.count).to be(2)
         license = files.find { |f| f[:name] == 'LICENSE.txt' }
-        expect(license).to_not be_nil
+        expect(license).not_to be_nil
 
         if described_class == Licensee::Projects::GitProject
           expect(files.first).to have_key(:oid)
@@ -123,7 +127,7 @@
 
           it 'looks for licenses in parent directories up to the search root' do
             # should not include the license in 'license-in-parent-folder' dir
-            expect(files.count).to eql(1)
+            expect(files.count).to be(1)
             expect(files.first[:name]).to eql('LICENSE.txt')
           end
         end
@@ -132,7 +136,7 @@
           let(:fixture) { 'license-in-parent-folder/license-folder/package' }
 
           it 'looks for licenses in current directory only' do
-            expect(files.count).to eql(0)
+            expect(files.count).to be(0)
           end
         end
       end
@@ -149,9 +153,10 @@
     end
 
     context 'readme detection' do
-      let(:fixture) { 'readme' }
       subject { described_class.new(path, detect_readme: true) }
 
+      let(:fixture) { 'readme' }
+
       it 'returns the readme' do
         expect(subject.readme_file).to be_a(Licensee::ProjectFiles::ReadmeFile)
         expect(subject.readme_file.filename).to eql('README.md')
@@ -164,6 +169,8 @@
     end
 
     context 'package manager detection' do
+      subject { described_class.new(path, detect_packages: true) }
+
       let(:fixture) { 'gemspec' }
 
       # Using a `.gemspec` extension in the fixture breaks `gem release`
@@ -202,8 +209,6 @@
         FileUtils.rm("#{fixture_path(fixture)}/project.gemspec")
       end
 
-      subject { described_class.new(path, detect_packages: true) }
-
       it 'returns the package file' do
         expected = Licensee::ProjectFiles::PackageManagerFile
         expect(subject.package_file).to be_a(expected)
@@ -232,19 +237,19 @@
       end
 
       it 'returns both licenses' do
-        expect(subject.licenses.count).to eql(2)
+        expect(subject.licenses.count).to be(2)
         expect(subject.licenses.first).to eql(Licensee::License.find('mpl-2.0'))
         expect(subject.licenses.last).to eql(mit)
       end
 
       it 'returns both matched_files' do
-        expect(subject.matched_files.count).to eql(2)
+        expect(subject.matched_files.count).to be(2)
         expect(subject.matched_files.first.filename).to eql('LICENSE')
         expect(subject.matched_files.last.filename).to eql('LICENSE.txt')
       end
 
       it 'returns both license_files' do
-        expect(subject.license_files.count).to eql(2)
+        expect(subject.license_files.count).to be(2)
         expect(subject.license_files.first.filename).to eql('LICENSE')
         expect(subject.license_files.last.filename).to eql('LICENSE.txt')
       end
@@ -260,29 +265,29 @@
       end
 
       it 'matched_file returns copying.lesser' do
-        expect(subject.matched_file).to_not be_nil
+        expect(subject.matched_file).not_to be_nil
         expect(subject.matched_file.filename).to eql('COPYING.lesser')
       end
 
       it 'license_file returns copying.lesser' do
-        expect(subject.license_file).to_not be_nil
+        expect(subject.license_file).not_to be_nil
         expect(subject.license_file.filename).to eql('COPYING.lesser')
       end
 
       it 'returns both licenses' do
-        expect(subject.licenses.count).to eql(2)
+        expect(subject.licenses.count).to be(2)
         expect(subject.licenses.first).to eql(lgpl)
         expect(subject.licenses.last).to eql(gpl)
       end
 
       it 'returns both matched_files' do
-        expect(subject.matched_files.count).to eql(2)
+        expect(subject.matched_files.count).to be(2)
         expect(subject.matched_files.first.filename).to eql('COPYING.lesser')
         expect(subject.matched_files.last.filename).to eql('LICENSE')
       end
 
       it 'returns both license_files' do
-        expect(subject.license_files.count).to eql(2)
+        expect(subject.license_files.count).to be(2)
         expect(subject.license_files.first.filename).to eql('COPYING.lesser')
         expect(subject.license_files.last.filename).to eql('LICENSE')
       end

data/spec/licensee/projects/github_project_spec.rb

@@ -1,16 +1,17 @@
 # frozen_string_literal: true
 
 RSpec.describe Licensee::Projects::GitHubProject do
+  subject(:instance) { described_class.new(github_url) }
+
   let(:repo) { 'benbalter/licensee' }
   let(:github_url) { "https://github.com/#{repo}" }
   let(:mit) { Licensee::License.find('mit') }
   let(:readme_file) { File.read(fixture_path('mit/README.md')) }
   let(:license_file) { File.read(fixture_path('mit/LICENSE.txt')) }
-  subject(:instance) { described_class.new(github_url) }
 
   describe '#initialize' do
     context 'with a GitHub URI' do
-      it 'should set @repo' do
+      it 'sets @repo' do
         expect(instance.repo).to eq(repo)
       end
     end
@@ -18,7 +19,7 @@ RSpec.describe Licensee::Projects::GitHubProject do
     context 'with a GitHub git URI' do
       let(:github_url) { "https://github.com/#{repo}.git" }
 
-      it 'should set @repo, stripping the trailing extension' do
+      it 'sets @repo, stripping the trailing extension' do
         expect(instance.repo).to eq(repo)
       end
     end
@@ -26,7 +27,7 @@ RSpec.describe Licensee::Projects::GitHubProject do
     context 'with a non-GitHub URI' do
       let(:github_url) { "https://gitlab.com/#{repo}" }
 
-      it 'should raise an ArgumentError' do
+      it 'raises an ArgumentError' do
         expect { instance }.to raise_error(ArgumentError)
       end
     end
@@ -34,7 +35,7 @@ RSpec.describe Licensee::Projects::GitHubProject do
     context 'with a local folder' do
       let(:github_url) { fixture_path('mit') }
 
-      it 'should raise an ArgumentError' do
+      it 'raises an ArgumentError' do
         expect { instance }.to raise_error(ArgumentError)
       end
     end