Skip to content

Make Sidekiq Web UI tests less flaky

What does this MR do and why?

This backports !152420 (merged) to 16-11-stable-ee.

One of the Sidekiq Web UI test attempts to test that the latency shown in the queues page is 0.0. However, if there is some slowness on the runner the latency could tick up to a second, causing the test to fail.

Avoid this flakiness by stubbing Time.now so that we guarantee that the latency will always remain at 0.

Relates to #460842 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

  • This MR is backporting a bug fix, documentation update, or spec fix, previously merged in the default branch.
  • The MR that fixed the bug on the default branch has been deployed to GitLab.com (not applicable for documentation or spec changes).
  • This MR has a severity label assigned (if applicable).
  • Set the milestone of the merge request to match the target backport branch version.
  • This MR has been approved by a maintainer (only one approval is required).
  • Ensure the e2e:package-and-test-ee job has either succeeded or been approved by a Software Engineer in Test.

Note to the merge request author and maintainer

If you have questions about the patch release process, please:

Edited by Mario Celi

Merge request reports