1716982 - Replace waitForEvent use in preferences tests with BrowserTestUtils.waitForEvent

Status: RESOLVED FIXED

(Firefox :: Settings UI, task, P3)

Description

[Mark Banner (:standard8)]

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

waitForEvent in the browser/components/preferences test directory can be replaced by the BrowserTestUtils.waitForEvent utility function.

The code in question is here: https://searchfox.org/mozilla-central/search?q=+waitForEvent&path=preferences&case=false&regexp=false

Once replaced, the function definition in the head.js file can be removed. In addition to removing the function, the import for Promise should also be able to be removed.

For instructions on how to get your local build of Firefox up and running and submit your patch, see https://developer.mozilla.org/en-US/docs/Introduction.

You can run the tests with the ./mach mochitest command:

./mach mochitest browser/components/preferences

This will run all the tests in the directory, to make sure that the head.js changes don't adversely affect anything else.

You should also make sure that ESLint passed:

./mach eslint browser/components/preferences

Note this bug will be auto-assigned when the first patch is attached.

Comment 1

[Mathew Hodson]

Attachment: Bug 1716982 - Use BrowserTestUtils.waitForEvent in preferences tests. r=standard8

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/31fe68d72de8
Use BrowserTestUtils.waitForEvent in preferences tests. r=Standard8,preferences-reviewers

Comment 3

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/31fe68d72de8