This bug will cover the necessary work to convert the following test into Marionette. http://hg.mozilla.org/qa/mozmill-tests/file/default/firefox/tests/remote/testSecurity/testUntrustedConnectionErrorPage.js
Some things to note here: * When loading the given URL we expect a MarionetteException given that it is an about:error page * The "getMeOutOfHereButton" can be retrieved via the content scope * Before you are able to click on the button, you will have to wait about 1s. That's something magic with about:error pages, which we never figured out exactly in Mozmill: https://github.com/mozilla/mozmill/blob/master/mozmill/mozmill/extension/resource/modules/windows.js#L194
I will work on converting this test to Marionette
Thanks a lot! Let me know if you need more information in how to get started.
Created attachment 8565500 [details] [review] patch for bugfix I have written the code and created a pull request. Could you review this for me?
Comment on attachment 8565500 [details] [review] patch for bugfix I added a couple of comments to the code on the PR. Please have a look at those and ask if something is unclear.
Hi Om, I see a pull request for this: https://github.com/mozilla/firefox-ui-tests/pull/89 It looks like it has a small tweak so it can be landed. Om, can you look at fixing this or let us know if you can't get to it so we can work on resolving this issue!
No reply from Om, given that is a P1 and we have to get this finished soon, we should tackle the remaining bits ourselves. I'm pulling this off from mentored bugs.
Comment on attachment 8579989 [details] [review] https://github.com/mozilla/firefox-ui-tests/pull/133 I made a couple of comments on the PR, which need to be fixed. Let me know if something is unclear.
Created attachment 8586514 [details] [review] github_pull_request.txt This PR simply removes the now, I believe, unnecessary edits to test_security_notification.py present in PR 133.
Comment on attachment 8586514 [details] [review] github_pull_request.txt This looks great. One small clean up noted in github.
Thanks for your review, Chris! I've updated the PR.