Closed Bug 1405128 Opened 7 years ago Closed 7 years ago

Intermittent devtools/client/framework/test/browser_source_map-reload.js | Test timed out -

Categories

(DevTools :: Framework, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: intermittent-bug-filer, Assigned: jlast)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Attached patch remove-dbg.patchSplinter Review
Attachment #8914583 - Flags: review?(jdescottes)
Assignee: nobody → jlaster
Status: NEW → ASSIGNED
Attachment #8914583 - Flags: review?(jdescottes) → review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c1d08a0deae
Remove debugger statement. r=jdescottes
Keywords: checkin-needed
TV shows that the test still can fail, but no action desired:
jdescottes: So the patch reverted the change from Bug 1404837 that touched the test that the TV suite found to be intermittent
jdescottes: I would say we should keep the patch that landed and leave open to investigate
jdescottes: (in any case the patch that landed for 1405128 is good because it fixes a stupid mistake from the other Bug)
Keywords: leave-open
We tried running locally ./mach test devtools/client/framework/test/browser_source_map-reload.js --repeat 5

and the test failed on the second iteration, even without the patches from Bug 1404837 (which we thought made the test intermittent). This means that the test would have failed TV also before Bug 1404837, it just happened to never have been run in a TV suite.

Here the test is probably badly designed and just fails when it is executed several times in a row, but not when part of a bigger test suite. We will close this bug since browser_source_map-reload.js is not really intermittent, but just failing when executed several times in a row.
See Also: → 1405338
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → INVALID
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: