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 95fa5d3
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.rb
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ def initialize(runnable, method_name)
end

def id
@id ||= "#{@runnable}##{@method_name}".freeze
@id ||= "#{@runnable.class.name}##{@method_name}".freeze
end

def <=>(other)
Expand Down

0 comments on commit 95fa5d3

Please sign in to comment.