From 1290a86fa0fbdd76293abdfef78547a3126db00b Mon Sep 17 00:00:00 2001
From: Soutaro Matsumoto <matsumoto@soutaro.com>
Date: Thu, 9 Nov 2023 13:52:55 +0900
Subject: [PATCH] Add `--[no-]exit-error-on-syntax-error` option

---
 lib/rbs/cli.rb       |  6 +++++-
 test/rbs/cli_test.rb | 10 +++++++---
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/lib/rbs/cli.rb b/lib/rbs/cli.rb
index 92a5ac996..120a2bb5d 100644
--- a/lib/rbs/cli.rb
+++ b/lib/rbs/cli.rb
@@ -445,6 +445,7 @@ def run_method(args, options)
 
     def run_validate(args, options)
       stdout = stdout()
+      exit_error = false
 
       OptionParser.new do |opts|
         opts.banner = <<EOU
@@ -460,6 +461,9 @@ def run_validate(args, options)
         opts.on("--silent") do
           stdout = StringIO.new
         end
+        opts.on("--[no-]exit-error-on-syntax-error", "exit(1) if syntax error is detected") {|bool|
+          exit_error = bool
+        }
       end.parse!(args)
 
       loader = options.loader()
@@ -632,7 +636,7 @@ def run_validate(args, options)
         syntax_errors.each do |message|
           self.stdout.puts message
         end
-        exit(1)
+        exit 1 if exit_error
       end
     end
 
diff --git a/test/rbs/cli_test.rb b/test/rbs/cli_test.rb
index 77b51aea2..3cbf1745b 100644
--- a/test/rbs/cli_test.rb
+++ b/test/rbs/cli_test.rb
@@ -432,12 +432,16 @@ class Foo
       with_cli do |cli|
         Dir.mktmpdir do |dir|
           (Pathname(dir) + 'a.rbs').write(rbs)
-          assert_raises SystemExit do
-            cli.run(["-I", dir, "validate"])
-          end
+
+          cli.run(["-I", dir, "validate"])
 
           last_lines = stdout.string.lines.last(3)
           assert_match(/void|self|instance|class/, last_lines.join("\n"))
+
+          cli.run(["-I", dir, "validate", "--no-exit-error-on-syntax-error"])
+          assert_raises SystemExit do
+            cli.run(["-I", dir, "validate", "--exit-error-on-syntax-error"])
+          end
         end
       end
     end