Closed Bug 867548 Opened 6 years ago Closed 6 years ago

Intermittent B2G emulator Marionette test_emulator.py TestEmulatorContent.test_emulator_cmd | ScriptTimeoutException: timed out

Categories

(Testing :: Marionette, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox21 wontfix, firefox22 wontfix, firefox23 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 fixed)

RESOLVED FIXED
mozilla23
Tracking Status
firefox21 --- wontfix
firefox22 --- wontfix
firefox23 --- fixed
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- fixed

People

(Reporter: RyanVM, Assigned: jgriffin)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=22432588&tree=Mozilla-Inbound

b2g_ics_armv7a_gecko_emulator_vm mozilla-inbound opt test marionette-webapi on 2013-04-30 15:39:19 PDT for push d963b65538e1
slave: tst-linux64-ec2-303

15:45:09     INFO -  TEST-START test_emulator.py
15:45:10     INFO -  test_emulator_cmd (test_emulator.TestEmulatorChrome) ... ok
15:45:31     INFO -  test_emulator_cmd (test_emulator.TestEmulatorContent) ... ERROR
15:45:37     INFO -  test_emulator_orientation (test_emulator.TestEmulatorScreen) ... ok
15:45:37     INFO -  ======================================================================
15:45:37     INFO -  ERROR: test_emulator_cmd (test_emulator.TestEmulatorContent)
15:45:37     INFO -  ----------------------------------------------------------------------
15:45:37     INFO -  Traceback (most recent call last):
15:45:37     INFO -    File "/builds/slave/test/build/tests/marionette/tests/testing/marionette/client/marionette/tests/unit/test_emulator.py", line 17, in test_emulator_cmd
15:45:37     INFO -      """);
15:45:37     INFO -    File "/builds/slave/test/build/tests/marionette/marionette/marionette.py", line 616, in execute_async_script
15:45:37     INFO -      scriptTimeout=script_timeout)
15:45:37     INFO -    File "/builds/slave/test/build/tests/marionette/marionette/marionette.py", line 342, in _send_message
15:45:37     INFO -      self._handle_error(response)
15:45:37     INFO -    File "/builds/slave/test/build/tests/marionette/marionette/marionette.py", line 391, in _handle_error
15:45:37    ERROR -      raise ScriptTimeoutException(message=message, status=status, stacktrace=stacktrace)
15:45:37    ERROR -  TEST-UNEXPECTED-FAIL | test_emulator.py TestEmulatorContent.test_emulator_cmd | ScriptTimeoutException: timed out
15:45:37     INFO -  ----------------------------------------------------------------------
15:45:37     INFO -  Ran 3 tests in 27.910s
15:45:37  WARNING -  FAILED (errors=1)
This test happened to time out immediately after the emulator checked for OTA updates.  When running emulator tests, we should disable these checks by setting the appropriate pref.
Assignee: nobody → jgriffin
Attachment #744255 - Flags: review?(ahalberstadt)
Comment on attachment 744255 [details] [diff] [review]
Disable updates when launching emulator,

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

Isn't there already a marionette session running? Why do we have to start a new one?

Also, I'd much rather the marionette test runner use mozprofile to create a new profile with these prefs in them, but I guess that is outside the scope of this bug. r+ for now
Attachment #744255 - Flags: review?(ahalberstadt) → review+
(In reply to Andrew Halberstadt [:ahal] from comment #3)
> Comment on attachment 744255 [details] [diff] [review]
> Disable updates when launching emulator,
> 
> Review of attachment 744255 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Isn't there already a marionette session running? Why do we have to start a
> new one?

There isn't; otherwise this would cause an error about there already being a session.
 
> Also, I'd much rather the marionette test runner use mozprofile to create a
> new profile with these prefs in them, but I guess that is outside the scope
> of this bug. r+ for now

The "bad" thing about doing it that way would be that we'd need to restart B2G to pick up the new profile, which would add 15+ seconds to every test run.  Not a big deal, but possibly a pain when running/debugging tests locally.
https://hg.mozilla.org/mozilla-central/rev/f9a5c9ba7d7a
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.