Closed
Bug 1906047
Opened 1 year ago
Closed 1 year ago
Move more test files to be under directories named test
Categories
(Toolkit :: General, task, P3)
Toolkit
General
Tracking
()
RESOLVED
FIXED
130 Branch
| Tracking | Status | |
|---|---|---|
| firefox130 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(4 files)
I've just noticed that we have a few directories that are containing tests, but are not under a /test/ or /tests/ directory.
This has a couple of effects:
- ESLint can't apply the
general-testconfiguration. - Searchfox can't identify these files as tests.
The affected directories are:
devtools/client/shared/redux/middleware/xpcshell/intl/benchmarks/toolkit/profile/xpcshell/
| Assignee | ||
Comment 1•1 year ago
|
||
| Assignee | ||
Comment 2•1 year ago
|
||
| Assignee | ||
Comment 3•1 year ago
|
||
| Assignee | ||
Comment 4•1 year ago
|
||
The general-test configuration applies this rule for all tests.
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e67c8cb4c716
Move devtools/client/shared/redux/middleware/xpcshell/ to be in a test sub-directory. r=devtools-reviewers,frontend-codestyle-reviewers,bomsy
https://hg.mozilla.org/integration/autoland/rev/89d9a9a60b82
Move intl/benchmarks/ to be in a test sub-directory. r=hsivonen,frontend-codestyle-reviewers
https://hg.mozilla.org/integration/autoland/rev/87f1ceadf115
Move toolkit/profile/ tests into a test sub-directory. r=mossop
https://hg.mozilla.org/integration/autoland/rev/b94db0e16ac7
Remove ESLint workaround for importing headjs globals now that the test directories have been fixed. r=frontend-codestyle-reviewers,Gijs
Comment 6•1 year ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/e67c8cb4c716
https://hg.mozilla.org/mozilla-central/rev/89d9a9a60b82
https://hg.mozilla.org/mozilla-central/rev/87f1ceadf115
https://hg.mozilla.org/mozilla-central/rev/b94db0e16ac7
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox130:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•