Closed Bug 1175080 Opened 9 years ago Closed 7 years ago

AssertionError: expected [Function] to not throw an error but 'Error: Polling socket recv() timeout!' was thrown can happen if you wait for an element to disappear

Categories

(Testing Graveyard :: JSMarionette, defect)

Product:
Testing Graveyard
Component:
JSMarionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: jlorenzo, Unassigned)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

[gaia] JohanLorenzo:bug-1175080 > mozilla-b2g:master
46 bytes, text/x-github-pull-request
azasypkin
: review+
Details | Review 
Found in bug 1170151. We have an intermittent failure, where Marionette looses the connection to b2g desktop. This happens when you wait for an element to disappear (thanks to the helper) while this element is being discarded.
The exception is thrown at [1] and the exception gets caught at [2]. As you can see per the comments in the code, we're in a case where the window is on the edge to be unloaded.

[1] https://github.com/mozilla-b2g/gaia/commit/1fe8bc6266f2873798fc803cbea33b2dbc44f9b7#diff-93f450be0058b99f97a02c6657b29880R241
[2] https://github.com/mozilla-b2g/gaia/commit/1fe8bc6266f2873798fc803cbea33b2dbc44f9b7#diff-ff53a9d7d2f4aba9ed3ff83dc2d380afR75
Keywords: intermittent-failure
Summary: "Error: Polling socket recv() timeout!" can happen if you wait for an element to disappear → AssertionError: expected [Function] to not throw an error but 'Error: Polling socket recv() timeout!' was thrown can happen if you wait for an element to disappear
This is an issue that doesn't exist with Marionette Python. An equivalent test ran for multiple months against b2gdesktop without being disconnected. Aus, Gareth, how is it possible to debug the connection between MarionetteJS and the device?
Flags: needinfo?(gaye)
Flags: needinfo?(aus)
We've been chasing down similar issues for a while. You can look at the progress that's happening there in bug 1159200.
Flags: needinfo?(aus)
Side note: TH has stopped to report intermittent failures, because bug 1175575 deactivated the test (missing feature in MJS). Let's wait the test to be reactivated before closing this bug as WFM.
Depends on: 1175575
See Also: → 1207453
Depends on: 1140344
There's a spike in failures matching this signature today (orangefactor bot will likely comment with them tonight), but it probably belongs in a new bug. They appear to have started happening with this bumperbot push: https://hg.mozilla.org/integration/b2g-inbound/rev/96a094ab2225

Needinfoing the three people who had patches in that push.
Flags: needinfo?(jlorenzo)
Flags: needinfo?(jdarcangelo)
Flags: needinfo?(eitan)
My patch in this push did not touch any test code. Looks like it may be caused by the patch from Bug 1140344.
Flags: needinfo?(jdarcangelo)
I turned the tests back on in bug 1140344. The signature is the same and is related to this current bug.

Out of curiosity, is there any orange to report? In bug1140344, I didn't get any oranges out of 2x30 Gij35 runs.
Flags: needinfo?(jlorenzo)
Flags: needinfo?(eitan)
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #63)
> I turned the tests back on in bug 1140344. The signature is the same and is
> related to this current bug.
> 
> Out of curiosity, is there any orange to report? In bug1140344, I didn't get
> any oranges out of 2x30 Gij35 runs.

Here's a list of the failures in OrangeFactor: http://brasstacks.mozilla.com/orangefactor/?display=Bug&tree=trunk&startday=2015-10-07&endday=2015-10-14&bugid=1175080

You can scroll down below the graph and click the "log" links to view the logs.
I see. I thought a job was re-triggered if it failed, like in Gaia PRs. Wes, do you want these tests to be turned off again? Fixing this intermittent failure will require some Marionette knowledge I'm not familiar with.
Flags: needinfo?(wkocher)
It's frequently but not permanently failing. I'd lean towards disabling it unless you can track someone down to deal with it sometime in the next week.
Flags: needinfo?(wkocher)
Comment on attachment 8674453 [details] [review]
[gaia] JohanLorenzo:bug-1175080 > mozilla-b2g:master

In this case, it's better to skip these tests, until this Marionette issue is fixed. The Marionette guys are currently working on MJS on device. Oleg, would you mind reviewing these skips? 

Here's a TH job showing no Gij35 error due to that signature: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=43daca2a212e1aa45859baedee5b9c23dc842ea3
Attachment #8674453 - Flags: review?(azasypkin)
Comment on attachment 8674453 [details] [review]
[gaia] JohanLorenzo:bug-1175080 > mozilla-b2g:master

lgtm
Attachment #8674453 - Flags: review?(azasypkin) → review+
Marionette JS was mainly implemented for B2G. The latter being discontinued, we can close this bug. Feel free to reopen if I'm mistaken.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(gaye)
Resolution: --- → WONTFIX
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: