Merge pull request #264 from mttkay/mk-fix-ruby3-deprecation-warning

Fix warning: Kernel#open is deprecated

lib/license_scout/license.rb

@@ -106,7 +106,7 @@ def license_content(license_id, contents_url)
 
         begin
           LicenseScout::Log.debug("[license] Pulling license content for #{license_id} from #{new_url}")
-          open(new_url).read
+          URI.open(new_url).read
         rescue RuntimeError => e
           if e.message =~ /redirection forbidden/
             m = /redirection forbidden:\s+(.+)\s+->\s+(.+)/.match(e.message)

spec/license_scout/license_spec.rb

@@ -27,13 +27,13 @@
   let(:apache_license_content) { File.read(File.join(SPEC_FIXTURES_DIR, "empty_project", "LICENSE")) }
   let(:record) { described_class::Record.new(spdx, source, apache_license_content) }
 
-  before do
-    allow(described_class::Record).to receive(:new).with(spdx, source, apache_license_content).and_return(record)
-  end
-
   describe ".new" do
     let(:subject) { described_class.new(dependency_path) }
 
+    before do
+      allow(described_class::Record).to receive(:new).with(spdx, source, apache_license_content).and_return(record)
+    end
+
     context "when path is nil" do
       let(:dependency_path) { nil }
 
@@ -91,4 +91,23 @@
       it { is_expected.to be false }
     end
   end
+
+  describe "#add_license" do
+    let(:license_url) { "https://url/to/license" }
+    let(:options) { { a: 42 } }
+
+    subject { described_class.new(dependency_path) }
+
+    it "downloads license body and adds a new record" do
+      expect(URI).to receive(:open).with(license_url).and_return(StringIO.new(apache_license_content))
+
+      subject.add_license(spdx, source, license_url, options)
+
+      new_record = subject.records.last
+      expect(new_record).not_to be_nil
+      expect(new_record.id).to eq(spdx)
+      expect(new_record.source).to eq(source)
+      expect(new_record.content).to eq(apache_license_content)
+    end
+  end
 end