fix: extend the scope of rescue Exception

- Process.output Process.code add exception: option
- Fix func should call @status, not status

Author: initdc

.rubocop.yml

@@ -24,3 +24,6 @@ Style/RaiseArgs:
 
 Style/SpecialGlobalVars:
   EnforcedStyle: use_builtin_english_names
+
+Lint/RescueException:
+  Enabled: false

lib/rb/process.rb

@@ -25,11 +25,11 @@ def respond_to_missing?(name, include_private = false)
     end
 
     def exit_code
-      status.exitstatus
+      @status.exitstatus
     end
 
     def success?
-      status.success?
+      @status.success?
     end
 
     alias_method :ok?, :success?
@@ -94,33 +94,37 @@ def self.run(*args, out: $stdout, err: $stderr, exception: false, **options)
     else
       Err.new(out_strio.string, err_strio.string, status)
     end
-  rescue Errno::ENOENT => e
+  rescue Exception => e
     raise e if exception
 
     Err.new(nil, nil, $?)
   end
 
-  def self.output(*args, **options)
+  def self.output(*args, exception: false, **options)
     stdout_reader, stdout_writer = IO.pipe
     pid = Process.spawn(*args, **options, out: stdout_writer)
     stdout_writer.close
 
     pid, status = Process.wait2(pid)
-    if status.exited? && status.success?
+    if status.success?
       stdout_reader.read
     else
       nil
     end
-  rescue Errno::ENOENT
+  rescue Exception => e
+    raise e if exception
+
     nil
   ensure
     stdout_reader.close
   end
 
-  def self.code(...)
-    pid, status = Process.wait2(spawn(...))
+  def self.code(*args, exception: false, **options)
+    pid, status = Process.wait2(spawn(*args, **options))
     status.exitstatus
-  rescue Errno::ENOENT
-    127
+  rescue Exception => e
+    raise e if exception
+
+    $?.exitstatus
   end
 end

spec/rb/process_spec.rb

@@ -42,9 +42,17 @@
     end
   end
 
-  it "not raise by default when Errno::ENOENT" do
+  it "not raise by default when Exception" do
     expect { Process.run("unamea") }.not_to raise_error
-    expect { Process.run("unamea", exception: true) }.to raise_error
+    expect { Process.run("unamea", exception: true) }.to raise_error Errno::ENOENT
+    expect { Process.run("./Rakefile") }.not_to raise_error
+    expect { Process.run("./Rakefile", exception: true) }.to raise_error Errno::EACCES
+
+    expect { Process.output("unamea") }.not_to raise_error
+    expect { Process.output("unamea", exception: true) }.to raise_error Errno::ENOENT
+
+    expect { Process.output("unamea") }.not_to raise_error
+    expect { Process.output("unamea", exception: true) }.to raise_error Errno::ENOENT
   end
 
   it "get the output and not print" do