tests are sleeping, not waiting for a condition

Bug #1505962 reported by Alan Griffiths
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Triaged
Medium
Unassigned
mir (Ubuntu)
Triaged
Medium
Unassigned

Bug Description

This is bad test design as the tests are inherently flaky unless the time is "excessive", and slow as "excessive" waits add to the total runtime.

$ grep this_thread::sleep_for -R tests/ | wc -l
96

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I think the focus should be on build times more than this. Build times are at least an order of magnitude more than test run times, even on a fast machine.

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

There has been some small improvement:

$ grep this_thread::sleep_for -R tests/ | wc -l
83

Changed in mir:
status: New → Triaged
Changed in mir:
importance: Undecided → Medium
Revision history for this message
Michał Sawicz (saviq) wrote :

Syncing task from Mir.

Changed in mir (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.