nodejs.rb in opal-0.7.0.beta2

- old
+ new

@@ -4,11 +4,11 @@
   module CliRunners
     class Nodejs
       def initialize(output)
         @output ||= output
       end
-      attr_reader :output
+      attr_reader :output, :exit_status
 
       def puts(*args)
         output.puts(*args)
       end
 
@@ -27,26 +27,32 @@
         raise MissingNodeJS, 'Please install Node.js to be able to run Opal scripts.'
       end
 
       # Let's support fake IO objects like StringIO
       def system_with_output(env, *cmd)
-        io_output = IO.try_convert(output)
-        return system(env,*cmd) if io_output
+        if (io_output = IO.try_convert(output))
+          system(env,*cmd)
+          @exit_status = $?.exitstatus
+          return
+        end
 
         if RUBY_PLATFORM == 'java'
           # JRuby has issues in dealing with subprocesses (at least up to 1.7.15)
           # @headius told me it's mostly fixed on master, but while we wait for it
           # to ship here's a tempfile workaround.
           require 'tempfile'
           require 'shellwords'
           tempfile = Tempfile.new('opal-node-output')
           system(env,cmd.shelljoin+" > #{tempfile.path}")
+          @exit_status = $?.exitstatus
           captured_output = File.read tempfile.path
           tempfile.close
         else
           require 'open3'
           captured_output, status = Open3.capture2(env,*cmd)
+          @exit_status = status.exitstatus
         end
+
         output.write captured_output
       end
 
       class MissingNodeJS < RunnerError
       end