Don't kill the console

Use the "quit" command instead. This seems to prevents some weirdness on
OS X. See #9761.
This commit is contained in:
Jon Leighton 2013-03-22 14:24:36 +00:00
parent d8d6dada77
commit c91789c76b
1 changed files with 2 additions and 10 deletions

View File

@ -126,12 +126,6 @@ class FullStackConsoleTest < ActiveSupport::TestCase
assert_output "> "
end
def kill(pid)
Process.kill('TERM', pid)
Process.wait(pid)
rescue Errno::ESRCH
end
def spawn_console
pid = Process.spawn(
"#{app_path}/bin/rails console --sandbox",
@ -148,15 +142,13 @@ class FullStackConsoleTest < ActiveSupport::TestCase
write_prompt "Post.count", "=> 0"
write_prompt "Post.create"
write_prompt "Post.count", "=> 1"
kill pid
@master.puts "quit"
pid = spawn_console
write_prompt "Post.count", "=> 0"
write_prompt "Post.transaction { Post.create; raise }"
write_prompt "Post.count", "=> 0"
ensure
kill pid if pid
@master.puts "quit"
end
end