Skip to content

Commit a13cf27

Browse files
luke-gruberpeterzhu2118
authored andcommitted
test_all_ractors: show errors immediately
1 parent 5a5efb1 commit a13cf27

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

tool/lib/test/unit.rb

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1709,22 +1709,25 @@ def _run_suite suite, type
17091709
r = Ractor.new(port) do |p|
17101710
instance = Ractor.receive
17111711
runner = Ractor.receive
1712-
instance.run runner
1712+
res = instance.run runner
17131713
movable_ivars = {:@_assertions => true, :@__passed__ => true, :@__name__ => true}
17141714
instance.instance_variables.each do |ivar|
17151715
unless movable_ivars[ivar]
17161716
instance.remove_instance_variable(ivar)
17171717
end
17181718
end
17191719
p.send(instance, move: true)
1720-
runner
1720+
p.send(runner)
1721+
res
17211722
end
17221723
r.send(inst, move: true)
17231724
r.send(self, move: false)
17241725
inst = port.receive
1725-
runner = r.value # done
1726+
runner = port.receive
1727+
result = r.value
17261728
_merge_results_from_ractor(runner)
17271729
port.close
1730+
result
17281731
else
17291732
inst.run self
17301733
end
@@ -1734,7 +1737,6 @@ def _run_suite suite, type
17341737
print "%.2f s = " % (Time.now - start_time) if @verbose
17351738
print result
17361739
puts if @verbose
1737-
$stdout.flush
17381740

17391741
leakchecker.check("#{inst.class}\##{inst.__name__}")
17401742

0 commit comments

Comments
 (0)