Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1470659] Move platformName test to separate file. #12260

Merged
merged 3 commits into from Aug 3, 2018

Conversation

moz-wptsync-bot
Copy link
Collaborator

Specific tests for the platformName capability do not belong in
the test for the response body structure.

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1470659
gecko-commit: a3d141d56e531d882111ccf569c76427091ed088
gecko-integration-branch: mozilla-inbound
gecko-reviewers: whimboo

Specific tests for the platformName capability do not belong in
the test for the response body structure.

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1470659
gecko-commit: a3d141d56e531d882111ccf569c76427091ed088
gecko-integration-branch: mozilla-inbound
gecko-reviewers: whimboo
For similar reasons as for platformName, tests for configuring the
timeouts object do not belong in the same parent test as those for
response body structure.

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1470659
gecko-commit: 0ad9872c27e07e5f9dbbb62ec41e5be180e98f05
gecko-integration-branch: mozilla-inbound
gecko-reviewers: automatedtester, whimboo
To be able to ignore tests for individual capabilities we need to
parametrize these tests.  geckodriver now supports setWindowRect,
but fails the proxy capability test because it is for some reason
not propagated back.

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1470659
gecko-commit: b5d9ec944fe0e33bdea383a1f0114ada1dcabdc3
gecko-integration-branch: mozilla-inbound
gecko-reviewers: whimboo
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already reviewed downstream.

@jgraham
Copy link
Contributor

jgraham commented Aug 3, 2018

Travis timeouts for wdspec tests.

@jgraham jgraham merged commit d0018e6 into master Aug 3, 2018
@sideshowbarker sideshowbarker deleted the gecko/1470659 branch November 22, 2018 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants