Closed
Bug 1873055
Opened 2 years ago
Closed 2 years ago
Fix missing awaits in test_engine_selector_environment.js and test_engine_selector_defaults.js
Categories
(Firefox :: Search, task, P1)
Firefox
Search
Tracking
()
RESOLVED
FIXED
123 Branch
| Tracking | Status | |
|---|---|---|
| firefox123 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
I just noticed that we're not awaiting on the assertActualEnginesEqualsExpected calls in test_engine_selector_environment.js and test_engine_selector_defaults.js.
Although the tests generally pass, they're not checking the results properly, and the first one has a minor test failure due to using the wrong configuration in a test.
I'll put a patch up to fix these in a moment.
|
Assignee | |
Updated•2 years ago
|
Summary: Fix missing awaits in test_engine_selector_environment.js and test_engine_selector_defaults → Fix missing awaits in test_engine_selector_environment.js and test_engine_selector_defaults.js
|
|
Assignee | |
Comment 1•2 years ago
|
||
|
||
Updated•2 years ago
|
Attachment #9371117 -
Attachment description: Bug 1873055 - Fix missing awaits in test_engine_selector_environment.js and test_engine_selector_defaults.js. r?mcheang! → Bug 1873055 - Fix missing awaits in test_engine_selector_*.js in search tests. r?mcheang!
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d4142f269312
Fix missing awaits in test_engine_selector_*.js in search tests. r=mcheang
|
||
Comment 3•2 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox123:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•