Closed Bug 1112141 Opened 10 years ago Closed 9 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)

defect
Not set
normal

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
QA Whiteboard: [fxosqa-auto-backlog+]
Assignee: nobody → florin.strugariu
QA Whiteboard: [fxosqa-auto-backlog+] → [fxosqa-auto-s8]
I can't reproduce this issue.

Closing this with works for me
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
I think the last log shows some different issue than the original one.
See Also: → 1142060
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
Perhaps, the edit mode is still active for a short while after the confirm button has been tapped, in this case?
If this fixes it, it would also address bug 1142060 (which is basically the same bug).
Assignee: bebe → martijn.martijn
Retriggered Gip10 a lot of times, which contains this test: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=18d3fd6fb4ac354d33c053288460226f500e7fb5
Attachment #8596032 - Flags: review?(npark)
Attachment #8596032 - Flags: review?(jlorenzo)
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 on attachment 8596032 [details] [review]
[gaia] mwargers:1112141 > mozilla-b2g:master

Looks good to me too!
Attachment #8596032 - Flags: review?(jlorenzo) → review+
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Ok, an error in fx-team repo. I guess that means this bug is still not fixed then.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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 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 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+
(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: 9 years ago9 years ago
Resolution: --- → FIXED
Blocks: 1142060
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: