Closed
Bug 1116778
Opened 6 years ago
Closed 6 years ago
test_app_permissions.html yields 404, I suspect the test isn't doing what we expect it to
Categories
(Testing :: Talos, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla37
People
(Reporter: jmaher, Assigned: kaustabh93)
Details
Attachments
(1 file)
1.09 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
while debugging a mochitest-chrome failure locally, I ran into test_app_permissions.html yielding a lot of 404 messages. This looked bizarre, so I took a look to ensure that is what we intended to do. From a brief look at the source code, I suspect it is not what we want to do: http://dxr.mozilla.org/mozilla-central/source/dom/devicestorage/test/test_app_permissions.html#131 in fact, I bet the file referenced: var gTestUri = "https://example.com/tests/dom/devicestorage/test/test_app_permissions.html" doesn't exist due to packaging issues. I suspect adding a chrome would help: "https://example.com/chrome/dom/devicestorage/test/test_app_permissions.html" I am sure someone could easily fix this.
Assignee | ||
Comment 1•6 years ago
|
||
Reporter | ||
Comment 2•6 years ago
|
||
Comment on attachment 8546621 [details] [diff] [review] 404.patch looks good to me!
Attachment #8546621 -
Flags: review+
Reporter | ||
Updated•6 years ago
|
Assignee: nobody → kaustabh93
Reporter | ||
Comment 3•6 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5b38afb60d8
Comment 4•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c5b38afb60d8
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in
before you can comment on or make changes to this bug.
Description
•