test_app_permissions.html yields 404, I suspect the test isn't doing what we expect it to

RESOLVED FIXED in mozilla37

Status

Testing
Talos
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jmaher, Assigned: Kaustabh Datta Choudhury)

Tracking

Trunk
mozilla37
x86
Linux
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
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

3 years ago
Created attachment 8546621 [details] [diff] [review]
404.patch
(Reporter)

Comment 2

3 years ago
Comment on attachment 8546621 [details] [diff] [review]
404.patch

looks good to me!
Attachment #8546621 - Flags: review+
(Reporter)

Updated

3 years ago
Assignee: nobody → kaustabh93
https://hg.mozilla.org/mozilla-central/rev/c5b38afb60d8
Status: NEW → RESOLVED
Last Resolved: 3 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.