Closed
Bug 1823797
Opened 2 years ago
Closed 2 years ago
Convert browser_JsonSchemaValidator.js and browser_ClientEnvironment.js to xpcshell tests
Categories
(Toolkit :: General, task, P3)
Toolkit
General
Tracking
()
RESOLVED
FIXED
113 Branch
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(2 files)
browser_JsonSchemaValidator.js
currently runs as a mochitest, but it is only testing the JsonSchemaValidator
module and that isn't doing anything above what xpcshell is capable of.
We should move it across to an xpcshell-test to make it simpler to run/develop.
Assignee | ||
Updated•2 years ago
|
Severity: -- → N/A
Priority: -- → P3
Assignee | ||
Comment 1•2 years ago
|
||
I just noticed that we can do the same with browser_ClientEnvironment.js
- it says it relies on XULAppInfo, but we can make that available in an xpcshell-test.
Hence we can remove both mochitests in toolkit/components/utils/
.
Summary: Convert browser_JsonSchemaValidator.js to a xpcshell-test → Convert browser_JsonSchemaValidator.js and browser_ClientEnvironment.js to a xpcshell-test
Assignee | ||
Updated•2 years ago
|
Summary: Convert browser_JsonSchemaValidator.js and browser_ClientEnvironment.js to a xpcshell-test → Convert browser_JsonSchemaValidator.js and browser_ClientEnvironment.js to xpcshell tests
Assignee | ||
Comment 2•2 years ago
|
||
Assignee | ||
Comment 3•2 years ago
|
||
Depends on D173365
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0f449a95cc7b
Convert browser_JsonSchemaValidator.js to a xpcshell-test. r=mossop
https://hg.mozilla.org/integration/autoland/rev/64604190f082
Remove browser_ClientEnvironment.js; merge its content into the xpcshell-test. r=mossop
Comment 5•2 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0f449a95cc7b
https://hg.mozilla.org/mozilla-central/rev/64604190f082
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox113:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•