Use a less ambiguous separator for the version update suggestions

Author: andriusm

lib/bundler/audit/cli/formats/junit.rb

@@ -101,7 +101,7 @@ def bundle_title(result)
 
           def advisory_solution(advisory)
             unless advisory.patched_versions.empty?
-              "upgrade to #{advisory.patched_versions.join(', ')}"
+              "upgrade to #{advisory.patched_versions.join(' or ')}"
             else
               "remove or disable this gem until a patch is available!"
             end

lib/bundler/audit/cli/formats/text.rb

@@ -105,7 +105,7 @@ def print_advisory(gem, advisory)
 
             unless advisory.patched_versions.empty?
               say "Solution: upgrade to ", :red
-              say advisory.patched_versions.join(', ')
+              say advisory.patched_versions.join(" or ")
             else
               say "Solution: ", :red
               say "remove or disable this gem until a patch is available!", [:red, :bold]

spec/cli/formats/junit_spec.rb

@@ -241,7 +241,7 @@
 
         context "when Advisory#patched_versions is not empty" do
           it 'must print "Solution: upgrade to ..."' do
-            expect(output).to include("Solution: upgrade to #{CGI.escapeHTML(advisory.patched_versions.join(', '))}")
+            expect(output).to include("Solution: upgrade to #{CGI.escapeHTML(advisory.patched_versions.join(' or '))}")
           end
         end
 

spec/cli/formats/text_spec.rb

@@ -230,7 +230,7 @@
 
         context "when Advisory#patched_versions is not empty" do
           it 'must print "Solution: upgrade to ..."' do
-            expect(output_lines).to include("Solution: upgrade to #{advisory.patched_versions.join(', ')}")
+            expect(output_lines).to include("Solution: upgrade to #{advisory.patched_versions.join(' or ')}")
           end
         end