mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
6a8c166498
Assuming EXIT_SUCCESS equals 0 is not portable --- test/ruby/test_fiber.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) `rb_fiber_terminate` must not return [Bug #18497] In a forked process from a fiber, the fiber becomes the only fiber, `fiber_switch` does nothing as there is no other fibers, `rb_fiber_terminate` does not terminate the fiber. In that case, reaches the end of `fiber_entry` finaly, which is declared as "COROUTINE" and should never return. --- cont.c | 3 ++- eval_intern.h | 2 +- test/fiber/test_process.rb | 15 +++++++++++++++ test/ruby/test_fiber.rb | 5 +++++ 4 files changed, 23 insertions(+), 2 deletions(-)
51 lines
1 KiB
Ruby
51 lines
1 KiB
Ruby
# frozen_string_literal: true
|
|
require 'test/unit'
|
|
require_relative 'scheduler'
|
|
|
|
class TestFiberProcess < Test::Unit::TestCase
|
|
def test_process_wait
|
|
Thread.new do
|
|
scheduler = Scheduler.new
|
|
Fiber.set_scheduler scheduler
|
|
|
|
Fiber.schedule do
|
|
pid = Process.spawn("true")
|
|
Process.wait(pid)
|
|
|
|
# TODO test that scheduler was invoked.
|
|
|
|
assert_predicate $?, :success?
|
|
end
|
|
end.join
|
|
end
|
|
|
|
def test_system
|
|
Thread.new do
|
|
scheduler = Scheduler.new
|
|
Fiber.set_scheduler scheduler
|
|
|
|
Fiber.schedule do
|
|
system("true")
|
|
|
|
# TODO test that scheduler was invoked (currently it's not).
|
|
|
|
assert_predicate $?, :success?
|
|
end
|
|
end.join
|
|
end
|
|
|
|
def test_fork
|
|
omit 'fork not supported' unless Process.respond_to?(:fork)
|
|
Thread.new do
|
|
scheduler = Scheduler.new
|
|
Fiber.set_scheduler scheduler
|
|
|
|
Fiber.schedule do
|
|
pid = Process.fork {}
|
|
Process.wait(pid)
|
|
|
|
assert_predicate $?, :success?
|
|
end
|
|
end.join
|
|
end
|
|
end
|