Closed
Bug 1448235
Opened 7 years ago
Closed 7 years ago
chrome url-classifier tests fail when run individually
Categories
(Toolkit :: Safe Browsing, defect)
Toolkit
Safe Browsing
Tracking
()
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
References
Details
Attachments
(1 file, 1 obsolete file)
2.95 KB,
patch
|
francois
:
review+
|
Details | Diff | Splinter Review |
They use files they don't declare in support-files. Just try running "mach mochitest toolkit/components/url-classifier/tests/mochitest/test_donottrack.html" and watch the 404 you get.
![]() |
Assignee | |
Updated•7 years ago
|
Summary: DNT url-classifier tests fail when run individually → chrome url-classifier tests fail when run individually
![]() |
Assignee | |
Comment 1•7 years ago
|
||
Attachment #8961645 -
Flags: review?(francois)
![]() |
Assignee | |
Updated•7 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
![]() |
Assignee | |
Comment 2•7 years ago
|
||
Attachment #8961657 -
Flags: review?(francois)
![]() |
Assignee | |
Updated•7 years ago
|
Attachment #8961645 -
Attachment is obsolete: true
Attachment #8961645 -
Flags: review?(francois)
Updated•7 years ago
|
Attachment #8961657 -
Flags: review?(francois) → review+
Comment 3•7 years ago
|
||
Did these tests ever work individually or has something changed recently which makes them fail in this way?
![]() |
Assignee | |
Comment 4•7 years ago
|
||
I expect they never worked.
![]() |
Assignee | |
Comment 5•7 years ago
|
||
Well, depends on how recently "recently" is. It's been a while since we started shipping out only the support files for the one test being run into the temp space we run them from, but "a while" is order of a year or two. So it's possible they worked right when initially checked in...
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/89c4b23da971
List the right support-files for the url-classifier chrome tests. r=francois
Comment 7•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in
before you can comment on or make changes to this bug.
Description
•