Closed Bug 1370223 Opened 3 years ago Closed 3 years ago
Create Window Maximising Tests
We need to have Window Maximising wdspec tests
Shouldn't we add those as wpt tests now that geckodriver is in tree?
(In reply to Henrik Skupin (:whimboo) from comment #1) > Shouldn't we add those as wpt tests now that geckodriver is in tree? I think that’s exactly what David said in the description.
(In reply to Henrik Skupin (:whimboo) from comment #1) > Shouldn't we add those as wpt tests now that geckodriver is in tree? Yup, thats why I said we need wdspec tests :)
So moving the bug into the right component.
Component: Marionette → web-platform-tests
As talked in our meeting yesterday we will keep wdspec test writing in the Marionette component.
Component: web-platform-tests → Marionette
Comment on attachment 8881449 [details] Bug 1370223: Add Window Maximize wdspec tests https://reviewboard.mozilla.org/r/152584/#review157706 ::: testing/web-platform/tests/webdriver/window_resizing_and_positioning.py:1 (Diff revision 1) > +from support.inline import inline I think we should rename this file according to the command’s name, so maximize_window.py. The reason is that working on the other tests has shown we are quite likely to hit the maximum timeout quite easily with wdspec, so in anticipation of this—knowing how expensive it is to run tests for Set Window Rect—we should have a separate file testing the Maximize Window command, I think. ::: testing/web-platform/tests/webdriver/window_resizing_and_positioning.py:16 (Diff revision 1) > +def test_maximize_rect_alert_prompt(session): > + # Step 2 > + session.url = alert_doc > + > + result = session.transport.send("POST", "session/%s/window/maximize" % session.session_id) > + > + assert_error(result, "unexpected alert open") This is not very extensive compared to our other tests, but I have a patch for this locally. I’m happy to land this first and integrate my changes into this file later. Sound OK?
Attachment #8881449 - Flags: review?(ato) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/25ee2110abb7 Add Window Maximize wdspec tests r=ato
You need to log in before you can comment on or make changes to this bug.