Closed Bug 1702451 Opened 3 years ago Closed 3 years ago

Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_extension_sideloading.js

Categories

(WebExtensions :: Untriaged, task, P5)

task

Tracking

(firefox89 fixed)

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: prathiksha, Assigned: lesore0789, Mentored)

Details

Attachments

(1 file)

This is a good first bug for newcomers to Firefox development.

BrowserTestUtils.waitForCondition in browser_extension_sideloading.js[0] can be replaced with TestUtils.waitForCondition.

You can run this test with the ./mach test command:
./mach test browser/base/content/test/webextensions/browser_extension_sideloading.js
Feel free to get in touch with me on Matrix if you're stuck.

[0] https://searchfox.org/mozilla-central/rev/4fa18c26fa907f38d56b599571b9846af1506f3c/browser/base/content/test/webextensions/browser_extension_sideloading.js

The Bugbug bot thinks this bug should belong to the 'WebExtensions::Untriaged' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Untriaged
Product: Firefox → WebExtensions

Hi Prathiksha, can I be assigned to this bug?

Flags: needinfo?(prathikshaprasadsuman)
Assignee: nobody → lesore0789
Status: NEW → ASSIGNED
Flags: needinfo?(prathikshaprasadsuman)
Pushed by prathikshaprasadsuman@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/fb14bbb21573
Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition. r=prathiksha
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch

Yay, thank you for the patch, Leslie! 🎉 Your contribution has been added to our recognition wiki at https://wiki.mozilla.org/Add-ons/Contribute/Recognition.

Welcome onboard! We look forward to seeing you around. :)

You need to log in before you can comment on or make changes to this bug.