summaryrefslogtreecommitdiff
path: root/tool/lib
diff options
context:
space:
mode:
authorNobuyoshi Nakada <nobu@ruby-lang.org>2021-09-13 15:47:45 +0900
committerNobuyoshi Nakada <nobu@ruby-lang.org>2021-09-13 15:47:45 +0900
commitaaa9805e7e26c50f21eff01f9ca67ce366226ca4 (patch)
tree10cc18033b689430118c93441fca06ae03416ee9 /tool/lib
parentdbdceb8a191b540caae534d28cee6f20a9759d50 (diff)
Add unique token to separated runner
Same as Test::Unit::CoreAssertions#assert_no_memory_leak.
Diffstat (limited to 'tool/lib')
-rw-r--r--tool/lib/core_assertions.rb20
1 files changed, 12 insertions, 8 deletions
diff --git a/tool/lib/core_assertions.rb b/tool/lib/core_assertions.rb
index 3bfc4ef32b..9530187db4 100644
--- a/tool/lib/core_assertions.rb
+++ b/tool/lib/core_assertions.rb
@@ -105,9 +105,7 @@ module Test
require_relative 'memory_status'
raise Test::Unit::PendedError, "unsupported platform" unless defined?(Memory::Status)
- token = "\e[7;1m#{$$.to_s}:#{Time.now.strftime('%s.%L')}:#{rand(0x10000).to_s(16)}:\e[m"
- token_dump = token.dump
- token_re = Regexp.quote(token)
+ token_dump, token_re = new_test_token
envs = args.shift if Array === args and Hash === args.first
args = [
"--disable=gems",
@@ -244,11 +242,11 @@ module Test
ABORT_SIGNALS = Signal.list.values_at(*%w"ILL ABRT BUS SEGV TERM")
- def separated_runner(out = nil)
+ def separated_runner(token, out = nil)
include(*Test::Unit::TestCase.ancestors.select {|c| !c.is_a?(Class) })
out = out ? IO.new(out, 'w') : STDOUT
at_exit {
- out.puts [Marshal.dump($!)].pack('m'), "assertions=#{self._assertions}"
+ out.puts "#{token}<error>", [Marshal.dump($!)].pack('m'), "#{token}</error>", "#{token}assertions=#{self._assertions}"
}
Test::Unit::Runner.class_variable_set(:@@stop_auto_run, true) if defined?(Test::Unit::Runner)
end
@@ -266,11 +264,12 @@ module Test
res_p, res_c = IO.pipe
opt[:ios] = [res_c]
end
+ token_dump, token_re = new_test_token
src = <<eom
# -*- coding: #{line += __LINE__; src.encoding}; -*-
BEGIN {
require "test/unit";include Test::Unit::Assertions;include Test::Unit::CoreAssertions;require #{__FILE__.dump}
- separated_runner #{res_c&.fileno}
+ separated_runner #{token_dump}, #{res_c&.fileno || 'nil'}
}
#{line -= __LINE__; src}
eom
@@ -288,9 +287,9 @@ eom
raise if $!
abort = status.coredump? || (status.signaled? && ABORT_SIGNALS.include?(status.termsig))
assert(!abort, FailDesc[status, nil, stderr])
- self._assertions += res[/^assertions=(\d+)/, 1].to_i
+ self._assertions += res[/^#{token_re}assertions=(\d+)/, 1].to_i
begin
- res = Marshal.load(res.unpack1("m"))
+ res = Marshal.load(res[/^#{token_re}<error>\n\K.*\n(?=#{token_re}<\/error>$)/m].unpack1("m"))
rescue => marshal_error
ignore_stderr = nil
res = nil
@@ -763,6 +762,11 @@ eom
end
q.output
end
+
+ def new_test_token
+ token = "\e[7;1m#{$$.to_s}:#{Time.now.strftime('%s.%L')}:#{rand(0x10000).to_s(16)}:\e[m"
+ return token.dump, Regexp.quote(token)
+ end
end
end
end