Closed
Bug 1224602
Opened 10 years ago
Closed 10 years ago
TEST-UNEXPECTED-ERROR | NoSuchElementException: NoSuchElementException: Unable to locate element: page-proxy-favicon
Categories
(Testing :: Firefox UI Tests, defect, P1)
Tracking
(firefox44 unaffected, firefox45 fixed)
RESOLVED
FIXED
mozilla45
| Tracking | Status | |
|---|---|---|
| firefox44 | --- | unaffected |
| firefox45 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Keywords: intermittent-failure, regression)
Attachments
(1 file)
This fails today across all platforms due to bug 1206244. The page-proxy-favicon reference has been replaced with identity-icon. Should be easy to fix.
|
Assignee | |
Comment 1•10 years ago
|
||
Beside the above reported failure we get another one caused by the same changes:
Traceback (most recent call last):
File "/home/henrik/.virtualenvs/fxtests/local/lib/python2.7/site-packages/marionette_client-1.1.0-py2.7.egg/marionette/marionette_test.py", line 313, in run
testMethod()
File "/mozilla/code/firefox-ui-tests/firefox_ui_harness/decorators.py", line 15, in wrapper
return target(self, *args, **kwargs)
File "/mozilla/code/firefox-ui-tests/firefox_ui_tests/functional/security/test_ev_certificate.py", line 36, in test_ev_certificate
self.locationbar.identity_icon.value_of_css_property('list-style-image'))
AssertionError: 'identity-secure' not found in u'url("chrome://browser/skin/identity-icon.svg#normal")'
The icon has been changed and we should base our test on the #normal.
|
|
Assignee | |
Comment 2•10 years ago
|
||
This changes the favicon to identity_icon and also adds an element for the new connection_icon.
Attachment #8687316 -
Flags: review?(spolk)
|
||
Comment 3•10 years ago
|
||
Travis failed. If that is explained, I will r+ this.
|
|
Assignee | |
Comment 4•10 years ago
|
||
The reason for the failures is an intermittent failure for loading remote web pages. All other 5 builds are passing.
|
|
||
Comment 5•10 years ago
|
||
I am curious as that what server/page is failing, but am convinced that this patch had nothing to do with the Travis failure. r+
|
|
||
Updated•10 years ago
|
Attachment #8687316 -
Flags: review?(spolk) → review+
|
|
Assignee | |
Comment 6•10 years ago
|
||
Merged the PR to mozilla-central:
https://github.com/mozilla/firefox-ui-tests/commit/0c52e899fd81fb659e49a875a54f5f8247368047
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-firefox44:
--- → unaffected
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
| Comment hidden (Intermittent Failures Robot) |
|
||
Updated•9 years ago
|
Product: Mozilla QA → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•