Closed Bug 1214439 Opened 6 years ago Closed 2 years ago

Intermittent browser_dbg_split-console-keypress.js | Test 4: DebugPos at line 19 -

Categories

(DevTools :: Debugger, defect, P5)

Product:
DevTools
Component:
Debugger
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: KWierso, Assigned: RyanVM)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure, Whiteboard: [test disabled on debug and Linux][stockwell disabled]) 

Linux e10s failure that's exceedingly easy to hit on ASAN runs w/ e10s forced on. Brian, can you please take a look?
Blocks: e10s-tests
tracking-e10s: --- → ?
Flags: needinfo?(bgrinstead)
(In reply to Ryan VanderMeulen [:RyanVM] from comment #6)
> Linux e10s failure that's exceedingly easy to hit on ASAN runs w/ e10s
> forced on. Brian, can you please take a look?

Looks like it's a timeout based on them failing at different stages in the test:

https://treeherder.mozilla.org/logviewer.html#?job_id=7184588&repo=fx-team
https://treeherder.mozilla.org/logviewer.html#?job_id=7245766&repo=fx-team

Seems like a good case for requestLongerTimeout since the test is repeating the same assertions over a series of steps.  Splitting it up would require extra code and the setup/teardown time for the tests would be constant so total test time would increase in that case.
Flags: needinfo?(bgrinstead)
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/5c3790398ff5
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Duplicate of this bug: 1255981
> 233 INFO TEST-OK | devtools/client/debugger/test/mochitest/browser_dbg_split-console-keypress.js | took 9206ms

This test is running in 10-20sec from what I can tell. Not sure what telling the harness to wait extra long for the test to finish is going to accomplish, since the default timeout for the harness is 30s.
Status: RESOLVED → REOPENED
status-firefox48: fixed → affected
Flags: needinfo?(bgrinstead)
Resolution: FIXED → ---
Target Milestone: Firefox 48 → ---
Maybe Eddy has cycles to look into this before I skip it on Linux?
Flags: needinfo?(ejpbruel)
Sorry Ryan, I already have several P1s for the debugger on my plate :-(

This looks like a frontend test for the console, neither of which are particular forte's of mine. Usually, I'd ask bgrins for help on this, but it seems like he's too busy as well?
Flags: needinfo?(ejpbruel)
Turns out we're already skipping this test on debug e10s anyway (along with the most of the rest of the directory), so it doesn't seem like a huge loss to add the rest of Linux to the mix too.
Flags: needinfo?(bgrinstead)
Keywords: leave-open
Whiteboard: [test disabled on Linux e10s]
Intermittent e10s test failure
Priority: -- → P5
Assignee: bgrinstead → nobody
This is currently skipped on: e10s && (debug || os == "linux")

Now we are getting additional failures on linux64-debug and linux64-asan and some osx-debug and win7-debug. Any objections to broadening to (debug || os == "linux"), or perhaps just skip-if = true?
Flags: needinfo?(gbrown)
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/82f925709dd1
Skip test browser_dbg_split-console-keypress.js on non-e10s too; r=me,test-only
Flags: needinfo?(gbrown)
Whiteboard: [test disabled on Linux e10s] → [test disabled on debug and Linux]
Whiteboard: [test disabled on debug and Linux] → [test disabled on debug and Linux][stockwell disabled]
Product: Firefox → DevTools
Status: REOPENED → RESOLVED
Closed: 5 years ago → 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.