diff options
author | k0kubun <k0kubun@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-10-22 15:49:22 +0000 |
---|---|---|
committer | k0kubun <k0kubun@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-10-22 15:49:22 +0000 |
commit | e38a0b46067fd261a88be608787927ccdf5b7877 (patch) | |
tree | 216dd07d4c2e0b5453d4cc195e52f0914d166334 /test/ruby | |
parent | 7f5bb3082d9190f24a9839182aafd1ae670b4b13 (diff) | |
download | ruby-e38a0b46067fd261a88be608787927ccdf5b7877.tar.gz |
revisit more MJIT test skips
r65308 passed both trunk-mjit and trunk-mjit-wait CIs. MJIT copy job
looks working fine. Then this commit skips 5 more tests. Some of them
were skipped in a very early stage and may still need to be skipped, but
I want to confirm them since they haven't been changed for a long time.
And this prefers having inline information on `RubyVM::MJIT.enabled?`.
This commit makes it easier to confirm whether there's suspicious test
skip by RubyVM::MJIT.enabled? or not.
After this commit, tentatively we're not skipping tests for MJIT other
than `assert_no_memory_leak` ones.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@65311 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'test/ruby')
-rw-r--r-- | test/ruby/test_io.rb | 8 | ||||
-rw-r--r-- | test/ruby/test_process.rb | 2 | ||||
-rw-r--r-- | test/ruby/test_rubyoptions.rb | 8 | ||||
-rw-r--r-- | test/ruby/test_thread_cv.rb | 2 |
4 files changed, 5 insertions, 15 deletions
diff --git a/test/ruby/test_io.rb b/test/ruby/test_io.rb index 2ccf25f1df..3faa4e1fdb 100644 --- a/test/ruby/test_io.rb +++ b/test/ruby/test_io.rb @@ -548,8 +548,6 @@ class TestIO < Test::Unit::TestCase if have_nonblock? def test_copy_stream_no_busy_wait - # JIT has busy wait on GC. It's hard to test this with JIT. - skip "MJIT has busy wait on GC. We can't test this with JIT." if RubyVM::MJIT.enabled? skip "multiple threads already active" if Thread.list.size > 1 msg = 'r58534 [ruby-core:80969] [Backport #13533]' @@ -2147,12 +2145,6 @@ class TestIO < Test::Unit::TestCase end def test_autoclose_true_closed_by_finalizer - if RubyVM::MJIT.enabled? - # This is skipped but this test passes with AOT mode. - # At least it should not be a JIT compiler's bug. - skip "MJIT worker does IO which is unexpected for this test" - end - feature2250 = '[ruby-core:26222]' pre = 'ft2250' t = Tempfile.new(pre) diff --git a/test/ruby/test_process.rb b/test/ruby/test_process.rb index 23e4f01c91..96660e5d16 100644 --- a/test/ruby/test_process.rb +++ b/test/ruby/test_process.rb @@ -1614,7 +1614,7 @@ class TestProcess < Test::Unit::TestCase Process.wait pid assert sig_r.wait_readable(5), 'self-pipe not readable' end - if RubyVM::MJIT.enabled? # MJIT may trigger extra SIGCHLD + if RubyVM::MJIT.enabled? # checking -DMJIT_FORCE_ENABLE. It may trigger extra SIGCHLD. assert_equal [true], signal_received.uniq, "[ruby-core:19744]" else assert_equal [true], signal_received, "[ruby-core:19744]" diff --git a/test/ruby/test_rubyoptions.rb b/test/ruby/test_rubyoptions.rb index 429f72b41c..4a26d6bfc7 100644 --- a/test/ruby/test_rubyoptions.rb +++ b/test/ruby/test_rubyoptions.rb @@ -8,7 +8,7 @@ require_relative '../lib/jit_support' class TestRubyOptions < Test::Unit::TestCase NO_JIT_DESCRIPTION = - if RubyVM::MJIT.enabled? + if RubyVM::MJIT.enabled? # checking -DMJIT_FORCE_ENABLE RUBY_DESCRIPTION.sub(/\+JIT /, '') else RUBY_DESCRIPTION @@ -117,7 +117,7 @@ class TestRubyOptions < Test::Unit::TestCase def test_verbose assert_in_out_err(["-vve", ""]) do |r, e| assert_match(VERSION_PATTERN, r[0]) - if RubyVM::MJIT.enabled? && !mjit_force_enabled? + if RubyVM::MJIT.enabled? && !mjit_force_enabled? # checking -DMJIT_FORCE_ENABLE assert_equal(NO_JIT_DESCRIPTION, r[0]) else assert_equal(RUBY_DESCRIPTION, r[0]) @@ -180,7 +180,7 @@ class TestRubyOptions < Test::Unit::TestCase def test_version assert_in_out_err(%w(--version)) do |r, e| assert_match(VERSION_PATTERN, r[0]) - if RubyVM::MJIT.enabled? + if RubyVM::MJIT.enabled? # checking -DMJIT_FORCE_ENABLE assert_equal(EnvUtil.invoke_ruby(['-e', 'print RUBY_DESCRIPTION'], '', true).first, r[0]) else assert_equal(RUBY_DESCRIPTION, r[0]) @@ -210,7 +210,7 @@ class TestRubyOptions < Test::Unit::TestCase ].each do |args| assert_in_out_err(args) do |r, e| assert_match(VERSION_PATTERN_WITH_JIT, r[0]) - if RubyVM::MJIT.enabled? + if RubyVM::MJIT.enabled? # checking -DMJIT_FORCE_ENABLE assert_equal(RUBY_DESCRIPTION, r[0]) else assert_equal(EnvUtil.invoke_ruby(['--jit', '-e', 'print RUBY_DESCRIPTION'], '', true).first, r[0]) diff --git a/test/ruby/test_thread_cv.rb b/test/ruby/test_thread_cv.rb index 4161ba2341..38bcc3b8fa 100644 --- a/test/ruby/test_thread_cv.rb +++ b/test/ruby/test_thread_cv.rb @@ -33,8 +33,6 @@ class TestThreadConditionVariable < Test::Unit::TestCase end def test_condvar_wait_exception_handling - skip "MJIT thread is unexpected for this test, especially with --jit-wait" if RubyVM::MJIT.enabled? - # Calling wait in the only thread running should raise a ThreadError of # 'stopping only thread' mutex = Mutex.new |