Merge pull request #54 from librariesio/mellisio/nil-safe-valid

lib/spdx.rb

@@ -227,6 +227,8 @@ def self.licenses
   end
 
   def self.valid_spdx?(spdx_string)
+    return false unless spdx_string.is_a?(String)
+
     SpdxParser.parse(spdx_string)
     true
   rescue SpdxGrammar::SpdxParseError

lib/spdx/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Spdx
-  VERSION = "2.0.0"
+  VERSION = "2.0.1"
 end

spec/spdx_spec.rb

@@ -187,6 +187,7 @@
         expect(Spdx.valid_spdx?("AND AND")).to be false
         expect(Spdx.valid_spdx?("MIT OR MIT AND OR")).to be false
         expect(Spdx.valid_spdx?("MIT OR FAKEYLICENSE")).to be false
+        expect(Spdx.valid_spdx?(nil)).to be false
       end
       it "returns true for valid spdx" do
         expect(Spdx.valid_spdx?("(MIT OR MPL-2.0)")).to be true