Closed Bug 995093 Opened 5 years ago Closed 5 years ago
Add a marionette test to check Keyboard app could show up correctly
Add a marionette test to check Keyboard app could show up correctly.
We have gaia-ui-tests that already cover this.
Hi Jan, Yes, but we also need this test in js-marionette. And this bug is blocking Bug 993247.
Hi Evan, Please help to review this patch, thanks.
Hi Gary, Just left comments for your patch. Nice work! :)
Please review this patch again. thanks.
Comment on attachment 8405294 [details] [review] pull request: https://github.com/mozilla-b2g/gaia/pull/18219 Hi Gary, The https://travis-ci.org/mozilla-b2g/gaia/jobs/22761579 travis job wasn't passed.
Hi Gary, Have some nits. Please refer to https://github.com/evanxd/gaia/blob/mpizza-bug-995093/apps/keyboard/test/marionette/lib/keyboard_tests.js https://github.com/evanxd/gaia/blob/mpizza-bug-995093/apps/keyboard/test/marionette/keyboardtests/index.html https://github.com/evanxd/gaia/blob/mpizza-bug-995093/apps/keyboard/test/marionette/keyboard_launch_test.js Thanks.
Hi Evan, I can not see any comments what you list links on comment #7, could you left you comment again? Thanks
Attachment #8405841 - Attachment is obsolete: true
Rudy is helping to debug the intermittent issues.
Comment on attachment 8405842 [details] [review] pull request: https://github.com/mozilla-b2g/gaia/pull/18241 Hi Evan, I just clean up this patch, please review it again. I'll re-run travis after bug 997045 is landed. thanks.
Attachment #8405842 - Flags: review?(evanxd)
We could review the patch after Bug 997045 is done. Thanks.
Please be informed that I've sent another pull request for stability check, which already contains the patches in Bug 997045. - https://github.com/mozilla-b2g/gaia/pull/18443 For the keyboard marionette-js test itself, it still needs some changes as the commit shows, https://github.com/RudyLu/gaia/commit/71d3202684d91b086080eed5a350f0f3a59a2dad
Hi Rudy and Evan, I just addressed your comments on my patch please help to take a look. https://github.com/mozilla-b2g/gaia/pull/18241 Thanks.
Thanks for incorporating my changes into the patch. BTW, the tests are not going to pass since my patches into marionette-apps is not merged yet, and we're working on that.
Comment on attachment 8405842 [details] [review] pull request: https://github.com/mozilla-b2g/gaia/pull/18241 Just comment on the pull request with some suggestions/nits. Thanks.
Attachment #8405842 - Flags: feedback+
We should just wait for Bug 997045 is fixed, then we could review the patch. Thanks.
Hi Evan and Rudy, Thanks for your wonderful works on bug 997045. I just update my code base and trigger travis again. Please help to reive my patch thanks. https://travis-ci.org/mozilla-b2g/gaia/builds/24083761
Yeah, I also did the stability test again with the latest Gaia change, https://github.com/mozilla-b2g/gaia/pull/18443 Need Evan's help to review it again and then we could merge this change!
Left comments in the GitHub page.
Hi Evan: Travis looks good can I get r+? https://travis-ci.org/mozilla-b2g/gaia/builds/24092483 https://travis-ci.org/mozilla-b2g/gaia/builds/24093269 Thanks~
Comment on attachment 8405842 [details] [review] pull request: https://github.com/mozilla-b2g/gaia/pull/18241 Hi Gary, r+, nice work!
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Hi Gary and Rudy, Do you guys have badg.us accounts? You got the Marionette Starter awards. http://badg.us/en-US/badges/badge/Marionette-Starter
We finally have marionette tests for keyboard app. It will let us have less regression. Nice!
(In reply to Evan Tseng [:evanxd][:愛聞插低] from comment #24) > Hi Gary and Rudy, > > Do you guys have badg.us accounts? > You got the Marionette Starter awards. > http://badg.us/en-US/badges/badge/Marionette-Starter Sure, my account is email@example.com. Thanks again.
Evan, Gary, Thanks for making this happen. -- I'll send Evan my account info to get the badge. :)
Attachment #8405842 - Flags: review?(evanxd) → review+
You need to log in before you can comment on or make changes to this bug.