move exec into Command class

Author: monkstone

lib/jruby_art/command.rb

@@ -13,8 +13,17 @@ def initialize(executable:, runner:, args:, filename:)
     end
 
     def cmd(root)
-      return [executable, JRubyOpts.new(root).opts, runner, filename, args].flatten if executable =~ /jruby/
-      [executable, JavaOpts.new(root).opts, '-cp', JRubyComplete.complete, 'org.jruby.Main', runner, filename, args].flatten
+      if executable =~ /jruby/
+        cmda = [executable, JRubyOpts.new(root).opts, runner, filename, args].flatten
+      else
+        cmda = [executable, JavaOpts.new(root).opts, '-cp', JRubyComplete.complete, 'org.jruby.Main', runner, filename, args].flatten
+      end
+      begin
+        # exec(*command)
+        exec(*cmda)
+        # exec replaces the Ruby process with the JRuby one.
+      rescue Java::JavaLang::ClassNotFoundException
+      end
     end
   end
 end

lib/jruby_art/runner.rb

@@ -169,12 +169,7 @@ def spin_up(starter_script, filename, argc)
         args: argc,
         filename: filename
       )
-      begin
-        # exec(*command)
-        exec(*build.cmd(SKETCH_ROOT))
-        # exec replaces the Ruby process with the JRuby one.
-      rescue Java::JavaLang::ClassNotFoundException
-      end
+      build.cmd(SKETCH_ROOT)
     end
 
     # NB: We really do mean to use 'and' not '&&' for flow control purposes