Merge pull request #258 from mauro-oto/non-zero-exit-on-argument-error

Use non-zero exit code if there's an argument error when running through CLI

Author: postmodern

lib/bundler/audit/cli.rb

@@ -156,6 +156,10 @@ def print_advisory(gem, advisory)
         say
       end
 
+      def self.exit_on_failure?
+        true
+      end
+
     end
   end
 end

spec/cli_spec.rb

@@ -2,6 +2,19 @@
 require 'bundler/audit/cli'
 
 describe Bundler::Audit::CLI do
+  describe "#start" do
+    context "with wrong arguments" do
+      it "exits with error status code" do
+        expect {
+          described_class.start ["check", "--ignore CVE-2015-9284"]
+        }.to raise_error(SystemExit) do |error|
+          expect(error.success?).to eq(false)
+          expect(error.status).to eq(1)
+        end
+      end
+    end
+  end
+
   describe "#update" do
     context "not --quiet (the default)" do
       context "when update succeeds" do