[ruby-dev:15231] Re: strange behavior about PTY.spawn

まつもと ゆきひろです

In message "[ruby-dev:15230] Re: strange behavior about PTY.spawn"
    on 01/11/20, akira yamada / やまだあきら <[e-mail]> writes:

|In article 15228, <[e-mail]>
|Y> こういう場合はAPIから考えた方が良いように思います。このパッ
|
|一応APIを考えてはみてるんですが… (^_^;
|APIをちゃんと示して提案しなければなりませんでしたね. 

ま、コードに全ての情報を含んでいるといえばその通りなのですが、
やはりわかりづらいですよね。

|そういう形でもよいです. 
|その点に関してどうするのがよいのかわかりませんでした. 
|
|  * Threadセーフになること
|  * 子供の終了時の状態がわかること
|
|ができるようになるとうれしいです. 
|あとforkした後でexecに失敗した場合に
|_exit(1)までいかなくなってしまったので
|その点についてちょっと手を入れているつもりです. 

ありがとうございます。その辺は取り込みます。
で、Process::Statusを返したほうが良いとは思うのですが、以下
のようなAPIになってしまったのは嬉しいんだか嬉しくないんだか
良く分かりません。意見を求めます。

ptyだけでなく本体も修正する必要がありました。

---
require 'runit/testcase'
require 'runit/cui/testrunner'
require 'pty'

class TestPTY < RUNIT::TestCase
  def setup
    @s = Signal::list
  end

  def test_STOP
    pid = nil
    e = assert_exception(PTY::ChildExited) {
      PTY.spawn('/bin/sh') {|rd,wr,pid|
	assert_equal(1, Process.kill(@s['STOP'], pid))
	sleep 1
      }
    }
    assert(pid != nil)
    assert_equal(1, Process.kill(0, pid))
    assert_equal(nil, e.status.exitstatus)
    assert_equal(nil, e.status.termsig)
    assert_equal(true, e.status.stopped?)
  end
  def test_KILL
    pid = nil
    e = assert_exception(PTY::ChildExited) {
      PTY.spawn('/bin/sh') {|rd,wr,pid|
	assert_equal(1, Process.kill(@s['KILL'], pid))
	sleep 1
      }
    }
    assert(pid == pid)
    assert(pid != nil)
    assert_exception(Errno::ESRCH) {Process.kill(0, pid)}
    assert_equal(nil, e.status.exitstatus)
    assert_equal(@s['KILL'], e.status.termsig)
    assert_equal(false, e.status.stopped?)
  end
  def test_exit1
    pid = nil
    e = assert_exception(PTY::ChildExited) {
      PTY.spawn('/bin/sh'){|rd,wr,pid|
	wr.puts 'echo 123'
	wr.puts 'exit 0'
	loop {rd.sysread(1024)} rescue IOError
      }
    }
    assert_exception(Errno::ESRCH) {Process.kill(0, pid)}
    assert_equal(0, e.status.exitstatus)
    assert_equal(nil, e.status.termsig)
    assert_equal(false, e.status.stopped?)
  end
  def test_exit2
    pid = nil
    e = assert_exception(PTY::ChildExited) {
      PTY.spawn('sleep 3') {|rd,wr,pid|
	assert_equal(1, Process.kill(0, pid))
	sleep
      }
    }
    assert_exception(Errno::ESRCH) {Process.kill(0, pid)}
    assert_equal(0, e.status.exitstatus)
    assert_equal(nil, e.status.termsig)
    assert_equal(false, e.status.stopped?)
  end
  def test_notexist
    pid = nil
    e = assert_exception(PTY::ChildExited) {
      PTY.spawn('/not exist') {|rd,wr,pid|
	sleep 3
	assert_fail
      }
    }
    assert_equal(1, e.status.exitstatus)
    assert_equal(nil, e.status.termsig)
    assert_equal(false, e.status.stopped?)
  end

  def test_thSTOP
    th = Thread.new {test_STOP}
    loop {sleep 0.5; break unless th.alive?}
  end
  def test_thKILL
    th = Thread.new {test_KILL}
    loop {sleep 0.5; break unless th.alive?}
  end
  def test_thexit1
    th = Thread.new {test_exit1}
    loop {sleep 0.5; break unless th.alive?}
  end
  def test_thexit2
    th = Thread.new {test_exit2}
    loop {sleep 0.5; break unless th.alive?}
  end
  def test_thnotexist
    th = Thread.new {test_notexist}
    loop {sleep 0.5; break unless th.alive?}
  end
end

if $0 == __FILE__
  if ARGV.size == 0
    suite = TestPTY.suite
  else
    suite = RUNIT::TestSuite.new
    ARGV.each do |testmethod|
      suite.add(TestPTY.new(testmethod))
    end
  end
  RUNIT::CUI::TestRunner.run(suite)
end