Skip to content

Commit

Permalink
Fix mismatch between test IDs in order reporter and in-memory lookup
Browse files Browse the repository at this point in the history
  • Loading branch information
zarifmahfuz committed Mar 22, 2024
1 parent cfb741e commit bce69e5
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions ruby/lib/ci/queue/static.rb
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,10 @@ def retry_queue

def populate(tests, random: nil)
@index = tests.map { |t| [t.id, t] }.to_h
puts "Loaded tests:"
@index.each do |key, value|
puts "#{key}: #{value.inspect}"
end
self
end

Expand Down
2 changes: 1 addition & 1 deletion ruby/lib/minitest/queue/order_reporter.rb
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ def start

def before_test(test)
super
@file.puts("#{test.class.name}##{test.name}")
@file.puts("#{test.class}##{test.name}")
@file.flush
end

Expand Down

0 comments on commit bce69e5

Please sign in to comment.