Reenable test_modal_dialogs.py

RESOLVED FIXED in Firefox 42

Status

Testing
Marionette
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: adw, Assigned: chmanchester)

Tracking

unspecified
mozilla42
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Bug 959567 was causing test_modal_dialogs.py to consistently time out on tryserver.  We wanted to land that bug ASAP so I disabled the test: https://hg.mozilla.org/mozilla-central/diff/4e2489f7cc7d/testing/marionette/client/marionette/tests/unit/unit-tests.ini

I wanted to fix it instead but couldn't figure out why it was timing out.  It looks like the test has to do with modal dialogs, which bug 959567 shouldn't have caused a problem for.

Henrik, could you look into why it's failing?

https://treeherder.mozilla.org/#/jobs?repo=try&revision=59830f6d8909

http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/dwillcoxon@mozilla.com-59830f6d8909/try-linux-debug/try_ubuntu32_vm-debug_test-marionette-bm03-tests1-linux32-build1307.txt.gz

> ...
> 16:48:09     INFO -  TEST-START | test_with_using_context.py TestSetContext.test_set_same_context_using_with_block
> 16:48:10     INFO -  TEST-PASS | test_with_using_context.py TestSetContext.test_set_same_context_using_with_block | took 714ms
> 16:48:10     INFO -  TEST-START | test_with_using_context.py TestSetContext.test_set_scope_while_in_with_block
> 16:48:11     INFO -  TEST-PASS | test_with_using_context.py TestSetContext.test_set_scope_while_in_with_block | took 1016ms
> 16:48:11     INFO -  TEST-START | test_modal_dialogs.py TestGlobalModals.test_alert_accept
> 16:48:28    ERROR -  TEST-UNEXPECTED-ERROR | test_modal_dialogs.py TestGlobalModals.test_alert_accept | TimeoutException: TimeoutException: Timed out after 5.2 seconds
> 16:48:28     INFO -  Traceback (most recent call last):
> 16:48:28     INFO -    File "/builds/slave/test/build/venv/local/lib/python2.7/site-packages/marionette/marionette_test.py", line 296, in run
> 16:48:28     INFO -      testMethod()
> 16:48:28     INFO -    File "/builds/slave/test/build/tests/marionette/tests/testing/marionette/client/marionette/tests/unit/test_modal_dialogs.py", line 38, in test_alert_accept
> 16:48:28     INFO -      self.wait_for_alert()
> 16:48:28     INFO -    File "/builds/slave/test/build/tests/marionette/tests/testing/marionette/client/marionette/tests/unit/test_modal_dialogs.py", line 30, in wait_for_alert
> 16:48:28     INFO -      Wait(self.marionette).until(lambda _: self.alert_present())
> 16:48:28     INFO -    File "/builds/slave/test/build/venv/local/lib/python2.7/site-packages/marionette_driver/wait.py", line 143, in until
> 16:48:28     INFO -      cause=last_exc)
> 16:48:28     INFO -  TEST-INFO took 16640ms
> 16:48:28     INFO -  TEST-START | test_modal_dialogs.py TestGlobalModals.test_alert_dismiss
> 16:48:30     INFO -  TEST-PASS | test_modal_dialogs.py TestGlobalModals.test_alert_dismiss | took 1533ms
> 16:48:30     INFO -  TEST-START | test_modal_dialogs.py TestGlobalModals.test_alert_text
> 16:48:31     INFO -  TEST-PASS | test_modal_dialogs.py TestGlobalModals.test_alert_text | took 1756ms
> ...
(Assignee)

Comment 1

3 years ago
The dialog/popup added by bug 959567 blocks the alert opened by the test. I think we just need to pref it off. Patch coming up.
Assignee: nobody → cmanchester
(Assignee)

Comment 2

3 years ago
Created attachment 8645098 [details]
MozReview Request: Bug 1192331 - Disable search suggestions UI when running marionette tests and re-enable modal test. r=adw

Bug 1192331 - Disable search suggestions UI when running marionette tests and re-enable modal test. r=adw
Attachment #8645098 - Flags: review?(adw)
(Reporter)

Comment 3

3 years ago
Comment on attachment 8645098 [details]
MozReview Request: Bug 1192331 - Disable search suggestions UI when running marionette tests and re-enable modal test. r=adw

https://reviewboard.mozilla.org/r/15385/#review13767

Setting browser.urlbar.userMadeSearchSuggestionsChoice=true is probably the right thing to do but I don't think I'm "officially" allowed to r+ Marionette patches. :-)
Attachment #8645098 - Flags: review?(adw) → review+
(Reporter)

Comment 4

3 years ago
And thanks for fixing it so quickly.
(Assignee)

Comment 5

3 years ago
Thanks for the review!
https://hg.mozilla.org/mozilla-central/rev/ff5f77b0dc71
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.