Open Bug 1421662 Opened 2 years ago Updated 2 years ago

Consider removing BrowserTestUtils.addTab

Categories

(Testing :: Mochitest, enhancement, minor)

enhancement
Not set
minor

Tracking

(Not tracked)

People

(Reporter: arai, Unassigned)

References

Details

BrowserTestUtils.addTab adds new tab and returns instantly before waiting for the load finishes.
each testcase needs to wait for load just after that,
otherwise the testcase runs in unstable state, that may result in intermittent failure (see 1421653).

BrowserTestUtils already provides BrowserTestUtils.openNewForegroundTab, that waits for load event by default,
so most callsite could be rewritten with it, or we can add similar function for background tab if really necessary.

also, other testcases that directly calls browser.addTab should also be rewritten with such function.
Component: BrowserTest → Mochitest
You need to log in before you can comment on or make changes to this bug.