Closed Bug 1162407 Opened 5 years ago Closed 5 years ago

Intermittent script timed out in test_replace_short_message_type.js

Categories

(Firefox OS Graveyard :: RIL, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(tracking-b2g:backlog, firefox41 fixed)

RESOLVED FIXED
tracking-b2g backlog
Tracking Status
firefox41 --- fixed

People

(Reporter: edgar, Assigned: edgar)

References

Details

Attachments

(1 file)

Sometimes each sub test took around 14~15 sec to finish (I guess try server was busy at that time), 120 sec is too short to finish whole test (9 sub tests).

> 05-05 10:30:54.124    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '00'
> 05-05 10:31:09.883    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '41'
> 05-05 10:31:23.113    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '42'
> 05-05 10:31:37.044    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '43'
> 05-05 10:31:50.244    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '44'
> 05-05 10:32:03.364    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '45'
> 05-05 10:32:16.144    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '46'
> 05-05 10:32:29.904    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '47'
> 05-05 10:32:43.563    44    44 I Gecko   : MARIONETTE LOG: INFO: Test message PID '5F'
> 05-05 10:32:54.643    44    44 I Gecko   : 1430821974644	Marionette	DEBUG	conn220 client <- ({"from":"0","value":[["INFO","TEST-START: /builds/slave/test/build/tests/marionette/tests/dom/mobilemessage/tests/marionette/test_replace_short_message_type.js","Tue May 05 2015 10:30:50 GMT+0000 (UTC)"],["INFO","navigator.mozMobileMessage is instance of function MozMobileMessageManager() {\n    [native code]\n}","Tue May 05 2015 10:30:53 GMT+0000 (UTC)"],["INFO","Test message PID '00'","Tue May 05 2015 10:30:54 GMT+0000 (UTC)"],["INFO","  Checking ('+1234567890', '00', 'B')","Tue May 05 2015 10:30:54 GMT+0000 (UTC)"],["INFO","  Checking ('+1234567891', '00', 'B')","Tue May 05 2015 10:30:55 GMT+0000 (UTC)"],["INFO","  Checking ('+1234567890', '41', 'B')","Tue May 05 2015 10:30:56 GMT+0000 (UTC)"],["INFO","  Checking ('+1234567891', '41', 'B')","Tue May 05 2015 10:30:57 GMT+0000 (UTC)"],["INFO","  Checking ('+1234567890', '42', 'B')","Tue May 05 2015 10:3
Attached patch Patch, v1Splinter Review
Just increase the time out value.
Blocks: 1081529
[Tracking Requested - why for this release]:
Comment on attachment 8602631 [details] [diff] [review]
Patch, v1

Review of attachment 8602631 [details] [diff] [review]:
-----------------------------------------------------------------

I didn't see intermittent script timed out in test_replace_short_message_type.js any more after applying this patch.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=064184812c03&exclusion_profile=false

Hi Bevis, may I have your review? Thank you.
Attachment #8602631 - Flags: review?(btseng)
Comment on attachment 8602631 [details] [diff] [review]
Patch, v1

Review of attachment 8602631 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks!
Attachment #8602631 - Flags: review?(btseng) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/675302681f29
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.