Bug #10852: TestObjSpace#test_dump_flags sometimes fails on mswin-CI

• Status: Closed
• Assignee: ko1 (Koichi Sasada)
• Target version: -
• ruby -v: trunk
• Backport: 2.0.0: DONTNEED, 2.1: DONTNEED, 2.2: DONE

[usa (Usaku NAKAMURA)]

mswinのCIで、以下のfailureが時々(2割くらい?)出るのですが、何がまずいのでしょうか？
2.2だと頻度はぐっと下がりますが、出ないわけではないようです。

'''

1. Failure:
   TestObjSpace#test_dump_flags [D:/tmp/mswin-build20150213-6324-8fqgzk/ruby/test/objspace/test_objspace.rb:199]:
   Expected /"wb_protected":true, "old":true, "long_lived":true, "marked":true/ to match "{"address":"0x00000001f5cfe8", "type":"STRING", "class":"0x00000000db8ee0", "frozen":true, "embedded":true, "fstring":true, "bytesize":3, "value":"foo", "encoding":"UTF-8", "memsize":40, "flags":{"wb_protected":true, "old":true}}\n".
   '''

[ko1 (Koichi Sasada)]

これ、ずっと考えてたんですが、
よく見ると major gc の時、old であっても、long_lived":true が外れることがあることがわかりました（それで正常）。

なので、テストを修正しようと思います。

trunk (2.3) と 2.2 では、微妙に違うので、2.2 についてはパッチ案を出そうと思います。

[ko1 (Koichi Sasada)]

• Status changed from Assigned to Closed

r50914 で trunk は修正しました（ref 忘れた）。

以下は、2.2 向けです。

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 50914)
+++ ChangeLog	(working copy)
@@ -1,3 +1,8 @@
+Tue Jun 16 17:39:41 2015  Koichi Sasada  <[e-mail]>
+
+	* test/objspace/test_objspace.rb (test_dump_flags): relax pattern
+	  because long_lived flag and marked flag can be false at major GC.
+
 Mon Jun 15 03:24:23 2015  Koichi Sasada  <[e-mail]>
 
 	* vm_insnhelper.c (lep_svar_set): add WBs.
Index: test/objspace/test_objspace.rb
===================================================================
--- test/objspace/test_objspace.rb	(revision 50914)
+++ test/objspace/test_objspace.rb	(working copy)
@@ -196,7 +196,7 @@
 
   def test_dump_flags
     info = ObjectSpace.dump("foo".freeze)
-    assert_match /"wb_protected":true, "old":true, "long_lived":true, "marked":true/, info
+    assert_match /"wb_protected":true, "old":true/, info
     assert_match /"fstring":true/, info
   end
 

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.0.0: DONTNEED, 2.1: DONTNEED, 2.2: REQUIRED to 2.0.0: DONTNEED, 2.1: DONTNEED, 2.2: DONE

ruby_2_2 向けパッチありがとうございます。 r50923 で適用しました。が手違いで test_objspace.rb に関係ない変更を混ぜてしまったので r50924 でその部分を revert しています。