Closed Bug 1621466 Opened 5 years ago Closed 5 years ago

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

Categories

(Firefox :: General, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 76
Tracking Status
firefox76 --- fixed

People

(Reporter: prathiksha, Assigned: krystle.salazar2, Mentored)

Details

Attachments

(1 file)

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

BrowserTestUtils.waitForCondition in browser_popupNotification_checkbox.js[0] can be replaced by TestUtils.waitForCondition.

You can run this test with the ./mach test command:
./mach test browser/base/content/test/popupNotifications/browser_popupNotification_checkbox.js

Please leave a comment if you would like to be assigned to this bug. Feel free to contact us via Matrix in the #osfcep channel or leave a comment here if you're stuck.

[0] https://searchfox.org/mozilla-central/source/browser/base/content/test/popupNotifications/browser_popupNotification_checkbox.js

Hi! I would like to work on this! Can it be assigned to me?

Assignee: nobody → krystle.salazar2
Status: NEW → ASSIGNED
Attachment #9133048 - Attachment description: Bug 1621466 - Replace Test Class in browser_popupNotification_checkbox.js r=prathiksha → Bug 1621466 - Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_popupNotification_checkbox.js r=prathiksha
Pushed by prathikshaprasadsuman@gmail.com: https://hg.mozilla.org/integration/autoland/rev/4a5224c856a0 Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_popupNotification_checkbox.js r=prathiksha
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
Attachment #9133048 - Attachment description: Bug 1621466 - Replace BrowserTestUtils.waitForCondition with TestUtils.waitForCondition in browser_popupNotification_checkbox.js r=prathiksha → Bug 1621466 - Replace Test Class in browser_popupNotification_checkbox.js r=prathiksha
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: