bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.
52 bytes, text/plain
In Bug 1032745 - Update test_homescreen_delete_app to use local content. We need the ability to install webapps. For this the server must serve content with the header "application/x-web-app-manifest+json" Can we update the web server to contain this header?
This is all done from Marionette so I'll see if the Marionette team can do it.
Component: Gaia::UI Tests → Marionette
Product: Firefox OS → Testing
Summary: Update the gaia ui tests webserver to work with manifest files → Update the Marionette webserver to work with manifest files
Priority: -- → P2
Marionette uses mozhttpd as a web server. It should be possible to add the appropriate content type for manifest files, however I'm unsure if this should go into mozhttpd itself, the Marionette test runner, or even the Gaia test runner. As it's related to the tests, I suspect the Gaia test runner (gaiatest) would be the most appropriate place, however we may need changes in Marionette test runner to allow this. Will: I'm guessing you're most familiar with mozhttpd, do you have any suggestions?
Yes, it should be pretty easy to use mozhttpd for this, just define a handler function with the appropriate headers. See the json response decorator for an example of this: http://hg.mozilla.org/mozilla-central/file/f93c0ef45597/testing/mozbase/mozhttpd/mozhttpd/handlers.py I suspect it would make the most sense if the logic for doing this was in the gaia test runner, but I'm not very familiar with either its structure or that of that marionette test runner so can't give much guidance there.
4 years ago
Assignee: nobody → dave.hunt
Status: NEW → ASSIGNED
Summary: Update the Marionette webserver to work with manifest files → Update the gaiatest webserver to serve app manifest files with the correct content-type
4 years ago
Component: Marionette → Gaia::UI Tests
Product: Testing → Firefox OS
Created attachment 8456069 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/21749/files
Comment on attachment 8456069 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/21749/files r+, works very well. Surprised, it didn't look too painful in the end. (tested in the context of the other pull request)
Attachment #8456069 - Flags: review?(zcampbell) → review+
Comment on attachment 8456069 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/21749/files Nice!
Attachment #8456069 - Flags: review?(wlachance) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.