Bug #6882: parallel test crashes when unknown exception is occured in a test

• Status: Closed
• Assignee: usa (Usaku NAKAMURA)
• Target version: 2.0.0
• ruby -v: ruby 2.0.0dev (2012-08-16 trunk 36711)

[usa (Usaku NAKAMURA)]

test-allを-j付きで実行していると、例えばrdocあたりがバグっていて
RDoc::Errorあたりでテストに失敗した場合、masterはRDoc::Errorという
例外を知らないわけですが、にもかかわらずMarshal.loadしようとするので
master自体がクラッシュしてしまいます。

worker側からは、masterが知ってる例外が何かを知る方法はおそらくないので、
全てのエラー時の例外を適当にラップして渡すか、そもそも例外をMashal.dump
するのをやめるか、のいずれかの方法しかないような気がします。

後者は大変めんどくさいのですが、前者はやってみたらできたのでパッチを
添付します。
実際のレポート表示に使われるメッセージは例外オブジェクトから組み立ててる
わけじゃなくてworker側のメッセージを利用してるようなので、この程度で問題
なさそうに見えます。

[sorah (Sorah Fukumori)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r36730.
Usaku, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• lib/test/unit.rb, lib/test/unit/parallel.rb:
  generate error message (String) in parallel.rb instead of
  marshalling Exception. Fixes [Bug #6882] [ruby-dev:46054]

[usa (Usaku NAKAMURA)]

• Status changed from Closed to Assigned

これっぽっちも直ってないのでreopen

[usa (Usaku NAKAMURA)]

• Status changed from Assigned to Closed

This issue was solved with changeset r36746.
Usaku, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• test/testunit/tests_for_parallel/ptest_forth.rb: added a test case
  which causes an error.

• test/testunit/test_parallel.rb: follow above change.
  see [Bug #6882]

[sorah (Sorah Fukumori)]

• Status changed from Closed to Assigned
• Assignee changed from sorah (Sorah Fukumori) to usa (Usaku NAKAMURA)

sorry for late reply, go ahead.

[usa (Usaku NAKAMURA)]

• Status changed from Assigned to Closed

順序前後。もう入れました。