Closed
Bug 1749284
Opened 3 years ago
Closed 3 years ago
Move test_nsFind.html into `components/find`
Categories
(Core :: Find Backend, enhancement)
Core
Find Backend
Tracking
()
RESOLVED
FIXED
98 Branch
Tracking | Status | |
---|---|---|
firefox98 | --- | fixed |
People
(Reporter: xidorn, Assigned: TYLin)
Details
Attachments
(2 files)
The test is about nsFind
, yet it's very far from it. It should be moved there for better discoverability.
Assignee | ||
Comment 1•3 years ago
|
||
We've removed nsIWordBreaker in
https://bugzilla.mozilla.org/show_bug.cgi?id=1424898.
Assignee | ||
Comment 2•3 years ago
|
||
I follow the suggestion in [1] to put the mochitest under test/mochitest/
.
[1] If you are adding a new test directory, make sure its name matches one of
the patterns in .eslintrc.js.
https://firefox-source-docs.mozilla.org/code-quality/lint/linters/eslint.html?highlight=eslintrc#i-have-valid-code-that-is-failing-the-no-undef-rule-or-can-t-be-parsed
Depends on D136048
Updated•3 years ago
|
Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/d7dc9e5afe91
Remove non-existing nsIWordBreaker interface in nsIFind.idl. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/ebb87165b2e9
Move nsFind tests into toolkit/components/find/test/mochitest. r=jfkthame
Comment 4•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d7dc9e5afe91
https://hg.mozilla.org/mozilla-central/rev/ebb87165b2e9
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox98:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•