Code review comment for lp:~psivaa/ubuntu-test-cases/skip-reminders

Revision history for this message
Evan (ev) wrote :

I don't think this is the correct approach. We shouldn't be hacking around buggy tests in the infrastructure. The problem should be fixed at the source, in the tests themselves.

In this instance, the developers could implement a per-test timeout within the reminders tests. In the future, they could use this feature from Autopilot itself:

https://code.launchpad.net/~thomir/autopilot/test-timeouts/+merge/233457

I'm going to reach out to the test owners on this.

review: Disapprove

« Back to merge proposal