Closed Bug 1258106 Opened 8 years ago Closed 7 years ago

Intermittent testEventDispatcher | Should have completed event before timeout -

Categories

(Firefox for Android Graveyard :: General, defect, P3)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect

Tracking

(fennec+, firefox56 wontfix, firefox57 fixed, firefox58 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 58
Tracking Flags:
Tracking Status
fennec + ---
firefox56 --- wontfix
firefox57 --- fixed
firefox58 --- fixed

People

(Reporter: aryx, Assigned: jchen)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Extend testEventDispatcher timeout (v1)
2.63 KB, patch
jchen
: review+
Details | Diff | Splinter Review 
+++ This bug was initially created as a clone of Bug #1213030 +++

https://treeherder.mozilla.org/logviewer.html#?job_id=24172513&repo=mozilla-inbound

05:14:53     INFO -  TEST-PASS | testEventDispatcher | testEventDispatcher.js - [send_message_for_response/< : 38] success == success
05:14:53     INFO -  TEST-PASS | testEventDispatcher | Given message occurred for registered event: {"innerType":"sync-reply","type":"Robocop:JS","args":[],"method":"send_message_for_response"} - Robocop:JS should equal Robocop:JS
05:14:53     INFO -  TEST-PASS | testEventDispatcher | UI event should be on UI thread -
05:14:53     INFO -  TEST-PASS | testEventDispatcher | Given message occurred for registered event: {"innerType":"sync-reply","type":"Robocop:JS","args":[],"method":"send_test_message"} - Robocop:JS should equal Robocop:JS
05:14:53     INFO -  TEST-PASS | testEventDispatcher | Given message occurred for registered event: {"message":"\nTEST-PASS | testEventDispatcher.js | [send_message_for_response\/< : 39] error == error\n","innerType":"progress","type":"Robocop:JS"} - Robocop:JS should equal Robocop:JS
05:14:53  WARNING -  TEST-UNEXPECTED-FAIL | testEventDispatcher | Should have completed event before timeout -
05:14:53     INFO -  0 ERROR Exception caught during test! - junit.framework.AssertionFailedError: TEST-UNEXPECTED-FAIL | testEventDispatcher | Should have completed event before timeout -
05:14:53     INFO -  	at junit.framework.Assert.fail(Assert.java:50)
05:14:53     INFO -  	at org.mozilla.gecko.FennecMochitestAssert._logMochitestResult(FennecMochitestAssert.java:128)
05:14:53     INFO -  	at org.mozilla.gecko.FennecMochitestAssert.ok(FennecMochitestAssert.java:150)
05:14:53     INFO -  	at org.mozilla.gecko.tests.helpers.AssertionHelper.fFail(AssertionHelper.java:110)
05:14:53     INFO -  	at org.mozilla.gecko.tests.testEventDispatcher.waitForAsyncEvent(testEventDispatcher.java:84)
05:14:53     INFO -  	at org.mozilla.gecko.tests.testEventDispatcher.testEventDispatcher(testEventDispatcher.java:133)
05:14:53     INFO -  	at java.lang.reflect.Method.invokeNative(Native Method)
05:14:53     INFO -  	at java.lang.reflect.Method.invoke(Method.java:525)
05:14:53     INFO -  	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
05:14:53     INFO -  	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
05:14:53     INFO -  	at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
05:14:53     INFO -  	at org.mozilla.gecko.tests.BaseRobocopTest.runTest(BaseRobocopTest.java:188)
05:14:53     INFO -  	at junit.framework.TestCase.runBare(TestCase.java:134)
05:14:53     INFO -  	at junit.framework.TestResult$1.protect(TestResult.java:115)
05:14:53     INFO -  	at junit.framework.TestResult.runProtected(TestResult.java:133)
05:14:53     INFO -  	at junit.framework.TestResult.run(TestResult.java:118)
05:14:53     INFO -  	at junit.framework.TestCase.run(TestCase.java:124)
05:14:53     INFO -  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
05:14:53     INFO -  	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
05:14:53     INFO -  	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
05:14:53     INFO -  	at org.mozilla.gecko.FennecInstrumentationTestRunner.onStart(FennecInstrumentationTestRunner.java:64)
05:14:53     INFO -  	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)
05:14:53  WARNING -  TEST-UNEXPECTED-FAIL | testEventDispatcher | Exception caught - junit.framework.AssertionFailedError: TEST-UNEXPECTED-FAIL | testEventDispatcher | Should have completed event before timeout -
05:14:53     INFO -  EventExpecter: no longer listening for Robocop:JS
05:14:53     INFO -  TEST-OK | testEventDispatcher | took 181754ms
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Nevin, might you be able to help with this longstanding (and frequent on Beta) intermittent?
Flags: needinfo?(cnevinchen)
ni tech leads and set tracking-fennec? to bring this up in triage meeting.
tracking-fennec: --- → ?
Flags: needinfo?(snorp)
Flags: needinfo?(s.kaspari)
Flags: needinfo?(max)
Flags: needinfo?(cnevinchen)
I'll consult with Max first
Flags: needinfo?(snorp)
Flags: needinfo?(s.kaspari)
tracking-fennec: ? → +
The timeouts happen at different times, so I think we just need to wait longer to prevent the timeout.
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Extend timeout for testEventDispatcher to 40 seconds and fix a bug where
the wrong mode is used for event callback tests. r=me for trivial
test-only fix.
Attachment #8913434 - Flags: review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8597af836f8
Extend testEventDispatcher timeout; r=me
https://hg.mozilla.org/mozilla-central/rev/a8597af836f8
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Flags: needinfo?(max)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: