Create a class to abstract away the interactions with the status bar icons to easily create and maintain marionette tests. See apps/system/test/marionette/lib/system.js for a model.
Created attachment 8511109 [details] [review] Github PR Here's the beginning of the testing class. It comes with a set of files performing basic feature tests. The idea is to add more tests later and more helpers if needed. Etienne, what do you think of it? Any suggestions on how to improve it?
Attachment #8511109 - Flags: review?(etienne)
Comment on attachment 8511109 [details] [review] Github PR This is brilliant. r=me with the few comments on github addressed. But we should wait for Kevin's go (see github) regarding the timeout extension for the geolocation test before landing. Do we already have a bug file to "Cover icon prioritization with integration tests"? :)
Attachment #8511109 - Flags: review?(etienne) → review+
Nice work. I left some comments on github, but I think we're ok here? Let me know if that's not the case.
Target Milestone: --- → 2.1 S7 (24Oct)
Thanks Kevin and Etienne for your input, I addressed your comments on an updated PR and replied on Github. Currently waiting on TBPL before landing.
Landed in master as of https://github.com/mozilla-b2g/gaia/commit/8b619f8496adc3e7fafcdd1d7008066d795388c6
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.