diff options
author | Noah Gibbs <noah.gibbs@shopify.com> | 2022-07-20 15:48:58 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-20 10:48:58 -0400 |
commit | 6140edb5df29bc9362ded379d6e2e72e4584d07a (patch) | |
tree | c8033042278c291b8e81000ec969a7398e0c3625 /test | |
parent | 86d061294d3cc1656e18d0e1fd4b4f290da16944 (diff) | |
download | ruby-6140edb5df29bc9362ded379d6e2e72e4584d07a.tar.gz |
Match +YJIT in Ruby desc when testing segv (#6141)
In test_bug_reporter and test_rubyoptions we intentionally
test child processes that cause SEGV. We run them with YJIT
if the parent uses YJIT so that the text description
matches the parent RUBY_DESCRIPTION.
Diffstat (limited to 'test')
-rw-r--r-- | test/-ext-/bug_reporter/test_bug_reporter.rb | 5 | ||||
-rw-r--r-- | test/ruby/test_rubyoptions.rb | 4 |
2 files changed, 9 insertions, 0 deletions
diff --git a/test/-ext-/bug_reporter/test_bug_reporter.rb b/test/-ext-/bug_reporter/test_bug_reporter.rb index 8e1121f908..81d2ca4674 100644 --- a/test/-ext-/bug_reporter/test_bug_reporter.rb +++ b/test/-ext-/bug_reporter/test_bug_reporter.rb @@ -4,6 +4,10 @@ require 'tmpdir' require_relative '../../lib/jit_support' class TestBugReporter < Test::Unit::TestCase + def yjit_enabled? + defined?(RubyVM::YJIT.enabled?) && RubyVM::YJIT.enabled? + end + def test_bug_reporter_add omit if ENV['RUBY_ON_BUG'] @@ -22,6 +26,7 @@ class TestBugReporter < Test::Unit::TestCase no_core = "Process.setrlimit(Process::RLIMIT_CORE, 0); " if defined?(Process.setrlimit) && defined?(Process::RLIMIT_CORE) args = ["--disable-gems", "-r-test-/bug_reporter", "-C", tmpdir] + args.push("--yjit") if yjit_enabled? # We want the printed description to match this process's RUBY_DESCRIPTION stdin = "#{no_core}register_sample_bug_reporter(12345); Process.kill :SEGV, $$" assert_in_out_err(args, stdin, [], expected_stderr, encoding: "ASCII-8BIT") ensure diff --git a/test/ruby/test_rubyoptions.rb b/test/ruby/test_rubyoptions.rb index 6d2c0fa5cd..ed2bc3538c 100644 --- a/test/ruby/test_rubyoptions.rb +++ b/test/ruby/test_rubyoptions.rb @@ -778,6 +778,10 @@ class TestRubyOptions < Test::Unit::TestCase def assert_segv(args, message=nil) omit if ENV['RUBY_ON_BUG'] + # We want YJIT to be enabled in the subprocess if it's enabled for us + # so that the Ruby description matches. + args.unshift("--yjit") if self.class.yjit_enabled? + test_stdin = "" opt = SEGVTest::ExecOptions.dup list = SEGVTest::ExpectedStderrList |