Skip to content

Resolve "Flaky test : expecting time not to mutate"

Ash McKenzie requested to merge 1196-flaky-test-expecting-time-not-to-mutate into main

What does this Merge Request do and why?

This MR fixes a flaky test where time wasn't frozen, despite https://api.rubyonrails.org/v5.2.4.4/classes/ActiveSupport/Testing/TimeHelpers.html#method-i-freeze_time being used. The cause of the issue was the use of Time.new instead of Time.now.

Merge Request checklist

  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.
  • Add the ~highlight label if this MR should be included in the CHANGELOG.md.

Closes #1196 (closed)

Merge request reports