Display an accurate message when the test fails

Currently, the message when the test fails is overridden by a test that checks if the remote debuggee is terminated. For example, when `assert_line_text` method is failed, the message should be `Expected to include ~ in ~`, but it will be overridden and the message will be `Expected the remote program to finish ~`. We should not assert in ensure block to prevent the problem.

Author: ono-max

test/support/console_test_case.rb

@@ -176,6 +176,12 @@ def run_test_scenario cmd, test_info
             check_error(/DEBUGGEE Exception/, test_info)
             assert_empty_queue test_info
           end
+
+          if r = test_info.remote_info
+            assert_program_finish test_info, r.pid, :debuggee
+          end
+
+          assert_program_finish test_info, pid, :debugger
         # result of `gets` return this exception in some platform
         # https://github.com/ruby/ruby/blob/master/ext/pty/pty.c#L729-L736
         rescue Errno::EIO => e
@@ -189,13 +195,16 @@ def run_test_scenario cmd, test_info
           read.close
           write.close
           kill_safely pid, :debugger, test_info
-          if name = test_info.failed_process
-            assert_block(create_message("Expected the #{name} program to finish", test_info)) { false }
-          end
         end
       end
     end
 
+    def assert_program_finish test_info, pid, name
+      unless wait_pid pid, TIMEOUT_SEC
+        assert_block(create_message("Expected the #{name} program to finish", test_info)) { false }
+      end
+    end
+
     def prepare_test_environment(program, test_steps, &block)
       ENV['RUBY_DEBUG_NO_COLOR'] = 'true'
       ENV['RUBY_DEBUG_TEST_UI'] = 'terminal'

test/support/test_case.rb

@@ -118,6 +118,8 @@ def wait_pid pid, sec
       end
 
       false
+    rescue Errno::ECHILD
+      true
     end
 
     def kill_safely pid, name, test_info