1233892 - [JSMarionette] Increase search, script, and waitFor timeouts

Status: RESOLVED FIXED

(Testing Graveyard :: JSMarionette, defect)

Description

[Michael Henretty [:mikehenrty][:mhenretty]]

We've been seeing a lot of waitForElement timeouts, and I believe a lot of theme are merely down to the fact that we don't wait long enough. Sometimes, the VMs we get to run these tests are *really* slow, so we need to accomdate that. Let's increase our waiting threshold for all tests here.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] mikehenrty:bug-1233892-increase-all-timeouts > mozilla-b2g:master

Comment 2

[Michael Henretty [:mikehenrty][:mhenretty]]

Comment on attachment 8700232 [details] [review]
[gaia] mikehenrty:bug-1233892-increase-all-timeouts > mozilla-b2g:master

Gareth, what do you think?

Comment 3

[Michael Henretty [:mikehenrty][:mhenretty]]

Comment on attachment 8700232 [details] [review]
[gaia] mikehenrty:bug-1233892-increase-all-timeouts > mozilla-b2g:master

Or Aus.

Comment 4

[Aus Lacroix [:aus]]

Comment on attachment 8700232 [details] [review]
[gaia] mikehenrty:bug-1233892-increase-all-timeouts > mozilla-b2g:master

60s seems fine for now. Let's think about lowering that a little bit as we see things stabilize. :) We always end up wasting a little more computing time when we increase timeouts. Not a big deal, but always nice to lessen the churn if we can.

Comment 5

[Aus Lacroix [:aus]]

commit (master): https://github.com/mozilla-b2g/gaia/commit/9d8f052fe836bb83557c8a6c8d87b8fe1b066424

fixed.

Comment 6

[Michael Henretty [:mikehenrty][:mhenretty]]

Thanks again Aus!