1162407 - Intermittent script timed out in test_replace_short_message_type.js

Status: RESOLVED FIXED

(Firefox OS Graveyard :: RIL, defect)

Description

[Edgar Chen [:edgar]]

https://treeherder.mozilla.org/logviewer.html#?job_id=7117115&repo=try
https://treeherder.mozilla.org/logviewer.html#?job_id=7177597&repo=try

Comment 1

[Edgar Chen [:edgar]]

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 ('[phone]', '00', 'B')","Tue May 05 2015 10:30:54 GMT+0000 (UTC)"],["INFO","  Checking ('[phone]', '00', 'B')","Tue May 05 2015 10:30:55 GMT+0000 (UTC)"],["INFO","  Checking ('[phone]', '41', 'B')","Tue May 05 2015 10:30:56 GMT+0000 (UTC)"],["INFO","  Checking ('[phone]', '41', 'B')","Tue May 05 2015 10:30:57 GMT+0000 (UTC)"],["INFO","  Checking ('[phone]', '42', 'B')","Tue May 05 2015 10:3

Comment 2

[Edgar Chen [:edgar]]

Attachment: Patch, v1

Just increase the time out value.

Comment 3

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

[Tracking Requested - why for this release]:

Comment 4

[Edgar Chen [:edgar]]

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.

Comment 5

[Bevis Tseng[:bevis][:btseng](Exited)]

Comment on attachment 8602631 [details] [diff] [review]
Patch, v1

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

Thanks!

Comment 6

[Edgar Chen [:edgar]]

https://hg.mozilla.org/integration/b2g-inbound/rev/675302681f29

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/675302681f29