Bug #525

test_convert(TestBignum) on NETBSD

Added by naruse (Yui NARUSE) over 3 years ago. Updated about 1 year ago.

[ruby-dev:36062]
Status:Closed Start date:08/31/2008
Priority:Normal Due date:
Assignee:- % Done:

100%
Category:core
Target version:1.9.1 Release Candidate
ruby -v:

Description

NetBSD 4.99.72 i386 にて、test/ruby/test_bignum.rb が以下のとおり失敗します。

test_convert(TestBignum) [/home/naruse/src/ruby-trunk/test/ruby/test_bignum.rb:199]:
<Errno::EINVAL> exception expected but was
Class: <RangeError>
Message: <"bignum too big to convert into `unsigned long'">
---Backtrace---
/home/naruse/src/ruby-trunk/test/ruby/test_bignum.rb:201:in `wait'
/home/naruse/src/ruby-trunk/test/ruby/test_bignum.rb:201:in `block in test_convert'
/home/naruse/src/ruby-trunk/test/ruby/test_bignum.rb:199:in `test_convert'
---------------

Associated revisions

Revision 20621
Added by matz (Yukihiro Matsumoto) over 3 years ago

* test/ruby/test_bignum.rb (TestBignum#test_convert): remove platform dependent call of Process.wait. coverage should be accomplished by other method. [ruby-dev:36062]

History

#1
Updated by ko1 (Koichi Sasada) over 3 years ago

  • Assignee set to nobu (Nobuyoshi Nakada)

#2
Updated by ko1 (Koichi Sasada) over 3 years ago

  • Assignee changed from nobu (Nobuyoshi Nakada) to usa (Usaku NAKAMURA)

#3
Updated by usa (Usaku NAKAMURA) over 3 years ago

  • Assignee deleted (usa (Usaku NAKAMURA))
子プロセスが存在しないときにwaitpid(2)の第1引数に0を与え、かつ、第3引数に不正な値を指定した場合、第1引数のチェックが優先されてECHILDになるか、第3引数のチェックが優先されてEINVALになるか、は、POSIXでは規定されていないように思います。
よって、Rubyでどちらかになることを強制する気がない限り、このテストはプラットフォーム依存の結果をチェックしているに過ぎないことになります。

仕様を決めるか、テストを直すか、または上記の見解が間違っていることを確認するか、いずれかの対応が必要なのではないかと愚考します。

# 誰が担当すべきかわからないので担当者は外します。

#4
Updated by ko1 (Koichi Sasada) over 3 years ago

遠藤さん，どう思います？

#5
Updated by yugui (Yuki Sonoda) over 3 years ago

  • Target version set to 1.9.1 Release Candidate

#6
Updated by matz (Yukihiro Matsumoto) over 3 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100
Applied in changeset r20621.

Also available in: Atom PDF