0
@@ -12,9 +12,7 @@
0
   def initialize(executable, *arguments)
0
     @executable = File.expand_path(executable)
0
     @arguments  = arguments
0
-    unless File.exist?(@executable)
0
-      raise Ropen::InvalidExecutableError.new("#{executable} does not exist")
0
-    end
0
+    check_executable(executable)
0
     @stdout_events = Ropen::Events.new
0
     @stderr_events = Ropen::Events.new
0
     @stdin_spool = Ropen::Spool.new
0
@@ -54,6 +52,16 @@
0
   end
0
   
0
 private
0
+  
0
+  def check_executable(executable)
0
+    unless File.exist?(@executable)
0
+      raise Ropen::InvalidExecutableError.new("#{executable} does not exist")
0
+    end
0
+    
0
+    unless File.executable?(@executable)
0
+      raise Ropen::InvalidExecutableError.new("#{executable} is not executable")
0
+    end
0
+  end
0
   
0
   def initialize_streams
0
     @stdin  = Ropen::Pipe.new

0
@@ -1 +1,2 @@
0
+puts "no shebang, no +x, no service"

0
@@ -9,6 +9,7 @@
0
     @exits_with_error  = fixture(:exits_with_error)
0
     @asks_for_name = fixture(:asks_for_name)
0
     @processes_data = fixture(:processes_data)
0
+    @unexecutable = fixture(:unexecutable)
0
   end
0
   
0
   describe "initializing" do
0
@@ -26,6 +27,12 @@
0
       lambda {
0
         Ropen::Command.new("bleepblorp")
0
       }.should raise_error(Ropen::InvalidExecutableError, "bleepblorp does not exist")
0
+    end
0
+    
0
+    it "should raise a Ropen::InvalidExecutableError if the executable isn't executable" do
0
+      lambda {
0
+        Ropen::Command.new(@unexecutable)
0
+      }.should raise_error(Ropen::InvalidExecutableError, "./spec/fixtures/unexecutable.rb is not executable")
0
     end
0
     
0
     it "should accept a set of arguments" do