Closed
Bug 1112141
Opened 11 years ago
Closed 10 years ago
Intermittent test_homescreen_delete_app_packaged.py TestDeleteApp.test_delete_app | StaleElementException: StaleElementException: The element reference is stale. Either the element is no longer attached to the DOM or the page has been refreshed.
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Firefox OS Graveyard
Gaia::UI Tests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: RyanVM, Assigned: martijn.martijn)
References
()
Details
(Keywords: intermittent-failure)
Attachments
(2 files)
04:59:15 INFO - TEST-START | test_homescreen_delete_app_packaged.py TestDeleteApp.test_delete_app
04:59:43 ERROR - TEST-UNEXPECTED-ERROR | test_homescreen_delete_app_packaged.py TestDeleteApp.test_delete_app | StaleElementException: StaleElementException: The element reference is stale. Either the element is no longer attached to the DOM or the page has been refreshed.
04:59:43 INFO - Traceback (most recent call last):
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/marionette_test.py", line 268, in run
04:59:43 INFO - testMethod()
04:59:43 INFO - File "/builds/slave/talos-slave/test/gaia/tests/python/gaia-ui-tests/gaiatest/tests/functional/homescreen/test_homescreen_delete_app_packaged.py", line 54, in test_delete_app
04:59:43 INFO - self.homescreen.wait_for_app_icon_not_present(self.test_data['name'])
04:59:43 INFO - File "/builds/slave/talos-slave/test/gaia/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 50, in wait_for_app_icon_not_present
04:59:43 INFO - Wait(self.marionette).until(lambda m: self.installed_app(app_name) is None)
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/wait.py", line 122, in until
04:59:43 INFO - rv = condition(self.marionette)
04:59:43 INFO - File "/builds/slave/talos-slave/test/gaia/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 50, in <lambda>
04:59:43 INFO - Wait(self.marionette).until(lambda m: self.installed_app(app_name) is None)
04:59:43 INFO - File "/builds/slave/talos-slave/test/gaia/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 154, in installed_app
04:59:43 INFO - if root_el.text == app_name:
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/marionette.py", line 96, in text
04:59:43 INFO - return self.marionette._send_message('getElementText', 'value', id=self.id)
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/decorators.py", line 36, in _
04:59:43 INFO - return func(*args, **kwargs)
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/marionette.py", line 631, in _send_message
04:59:43 INFO - self._handle_error(response)
04:59:43 INFO - File "/builds/slave/talos-slave/test/build/venv/lib/python2.7/site-packages/marionette/marionette.py", line 669, in _handle_error
04:59:43 INFO - raise errors.StaleElementException(message=message, status=status, stacktrace=stacktrace)
04:59:43 INFO - TEST-INFO took 28412ms
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•11 years ago
|
QA Whiteboard: [fxosqa-auto-backlog+]
Updated•11 years ago
|
Assignee: nobody → florin.strugariu
QA Whiteboard: [fxosqa-auto-backlog+] → [fxosqa-auto-s8]
Comment 2•11 years ago
|
||
I can't reproduce this issue.
Closing this with works for me
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•11 years ago
|
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Assignee | ||
Comment 4•11 years ago
|
||
I think the last log shows some different issue than the original one.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 15•10 years ago
|
||
TEST-START | test_homescreen_delete_app_packaged.py TestDeleteApp.test_delete_app
TEST-UNEXPECTED-ERROR | test_homescreen_delete_app_packaged.py TestDeleteApp.test_delete_app | StaleElementException: StaleElementException: The element reference is stale. Either the element is no longer attached to the DOM or the page has been refreshed.
Traceback (most recent call last):
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_client-0.11-py2.7.egg/marionette/marionette_test.py", line 296, in run
testMethod()
File "/home/tester/git_checkout/tests/python/gaia-ui-tests/gaiatest/tests/functional/homescreen/test_homescreen_delete_app_packaged.py", line 51, in test_delete_app
self.homescreen.wait_for_app_icon_not_present(self.test_data['name'])
File "/home/tester/git_checkout/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 48, in wait_for_app_icon_not_present
Wait(self.marionette).until(lambda m: self.installed_app(app_name) is None)
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_driver-0.4-py2.7.egg/marionette_driver/wait.py", line 122, in until
rv = condition(self.marionette)
File "/home/tester/git_checkout/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 48, in <lambda>
Wait(self.marionette).until(lambda m: self.installed_app(app_name) is None)
File "/home/tester/git_checkout/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py", line 152, in installed_app
if root_el.text == app_name and (root_el.get_attribute('data-app-state') == 'ready' or
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_driver-0.4-py2.7.egg/marionette_driver/marionette.py", line 87, in text
return self.marionette._send_message('getElementText', 'value', id=self.id)
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_driver-0.4-py2.7.egg/marionette_driver/decorators.py", line 36, in _
return func(*args, **kwargs)
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_driver-0.4-py2.7.egg/marionette_driver/marionette.py", line 715, in _send_message
self._handle_error(response)
File "/home/tester/git_checkout/ci_venv/local/lib/python2.7/site-packages/marionette_driver-0.4-py2.7.egg/marionette_driver/marionette.py", line 751, in _handle_error
raise errors.lookup(status)(message, stacktrace=stacktrace)
TEST-INFO took 39229ms
TEST-START | test_settings_battery_power_save_mode.py TestBatteryPowerSaveMode.test_settings_battery_power_save_mode
TEST-PASS | test_settings_battery_power_save_mode.py TestBatteryPowerSaveMode.test_settings_battery_power_save_mode | took 21951ms
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 17•10 years ago
|
||
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/tests/functional/homescreen/test_homescreen_delete_app_packaged.py#51
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py#47
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/app.py#150
So apparently root_el is stale in some case.
Assignee | ||
Comment 18•10 years ago
|
||
Perhaps, the edit mode is still active for a short while after the confirm button has been tapped, in this case?
![]() |
||
Comment 19•10 years ago
|
||
Assignee | ||
Comment 20•10 years ago
|
||
If this fixes it, it would also address bug 1142060 (which is basically the same bug).
Assignee | ||
Updated•10 years ago
|
Assignee: bebe → martijn.martijn
Assignee | ||
Comment 21•10 years ago
|
||
Retriggered Gip10 a lot of times, which contains this test: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=18d3fd6fb4ac354d33c053288460226f500e7fb5
Assignee | ||
Updated•10 years ago
|
Attachment #8596032 -
Flags: review?(npark)
Attachment #8596032 -
Flags: review?(jlorenzo)
![]() |
||
Comment 22•10 years ago
|
||
Comment on attachment 8596032 [details] [review]
[gaia] mwargers:1112141 > mozilla-b2g:master
Fix looks good to me. I did also notice that the exiting from homescreen edit mode has become tiny bit slower recently.
Attachment #8596032 -
Flags: review?(npark) → review+
Comment 23•10 years ago
|
||
Comment on attachment 8596032 [details] [review]
[gaia] mwargers:1112141 > mozilla-b2g:master
Looks good to me too!
Attachment #8596032 -
Flags: review?(jlorenzo) → review+
Updated•10 years ago
|
Keywords: checkin-needed
![]() |
||
Updated•10 years ago
|
Keywords: checkin-needed
![]() |
||
Comment 24•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/ee4941acf3f4b544aa89871cb46ff306e488c493
![]() |
||
Updated•10 years ago
|
Status: REOPENED → RESOLVED
Closed: 11 years ago → 10 years ago
Resolution: --- → FIXED
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 26•10 years ago
|
||
Ok, an error in fx-team repo. I guess that means this bug is still not fixed then.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 41•10 years ago
|
||
Ok, this test had actually a glaring error, this makes the test correct and I think could fix the intermittent failure.
I tried it 40 times locally and it didn't fail once (although I haven't seen this failure either).
Attachment #8621807 -
Flags: review?(npark)
Attachment #8621807 -
Flags: review?(jlorenzo)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 45•10 years ago
|
||
Comment on attachment 8621807 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/30572
Even though there are many failures in Gip10. I don't see any of them related the StaleElementException seen in delete_app_packaged.
Attachment #8621807 -
Flags: review?(jlorenzo) → review+
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
![]() |
||
Comment 48•10 years ago
|
||
Comment on attachment 8621807 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/30572
Hmm, didn't see that we never actually exited from the edit mode. Tapping the home button does the same thing, which I do in graphics/test_drag_drop.py, but I guess this method is better.
Attachment #8621807 -
Flags: review?(npark) → review+
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 63•10 years ago
|
||
(In reply to No-Jun Park [:njpark] from comment #48)
> Hmm, didn't see that we never actually exited from the edit mode. Tapping
> the home button does the same thing, which I do in
> graphics/test_drag_drop.py, but I guess this method is better.
Oh, I didn't think of that. I don't have a preference, actually. I just wanted to make this code correct.
(In reply to Johan Lorenzo [:jlorenzo] (QA) from comment #45)
> Even though there are many failures in Gip10. I don't see any of them
> related the StaleElementException seen in delete_app_packaged.
Yeah, that looks quite bad :/
At least I hope this failure will be gone with this pull request.
Merged: https://github.com/mozilla-b2g/gaia/commit/29e990805540fa4d1d03f1144bf9c95364ef6b51
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•