Return :epipe if command is not running

akash-akya · Sep 25, 2024 · b8a131f · b8a131f
1 parent 9fcb065
commit b8a131f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/ex_cmd/process.ex b/lib/ex_cmd/process.ex
@@ -636,6 +636,9 @@ defmodule ExCmd.Process do
 
   def handle_call({:write_stdin, binary}, from, state) do
     case State.pop_operation(state, :write_stdin) do
+      _ when state.status != :running ->
+        {:reply, {:error, :epipe}, state}
+
       {:error, :operation_not_found} ->
         case Operations.pending_input(state, from, binary) do
           {:noreply, state} ->

diff --git a/test/ex_cmd/process_test.exs b/test/ex_cmd/process_test.exs
@@ -21,7 +21,7 @@ defmodule ExCmd.ProcessTest do
       assert {:ok, 0} == Process.await_exit(s, 500)
 
       # wait for the process to terminate
-      :timer.sleep(100)
+      :timer.sleep(200)
       refute Elixir.Process.alive?(s.pid)
     end
 
@@ -42,7 +42,7 @@ defmodule ExCmd.ProcessTest do
       assert {:ok, 0} == Process.await_exit(s, 200)
 
       # wait for the process to terminate
-      :timer.sleep(100)
+      :timer.sleep(200)
       refute Elixir.Process.alive?(s.pid)
     end