Dduvall has uploaded a new change for review. https://gerrit.wikimedia.org/r/154217
Change subject: QA: Workaround for cookie domain bug in chrome webdriver ...................................................................... QA: Workaround for cookie domain bug in chrome webdriver Added a domain to the add_cookie call used by the "using the mobile site" step to avoid a bug where chrome complains of "optional 'domain' invalid" despite it being optional. Change-Id: I7134bbf3031a348e261a73fa354cd8e19ae11b85 --- M tests/browser/features/step_definitions/common_steps.rb 1 file changed, 5 insertions(+), 3 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend refs/changes/17/154217/1 diff --git a/tests/browser/features/step_definitions/common_steps.rb b/tests/browser/features/step_definitions/common_steps.rb index 79cfeb4..8e10f19 100644 --- a/tests/browser/features/step_definitions/common_steps.rb +++ b/tests/browser/features/step_definitions/common_steps.rb @@ -15,9 +15,11 @@ end Given(/^I am using the mobile site$/) do - visit(MainPage) - @browser.cookies.add 'mf_useformat', 'true' - on(MainPage).refresh + on(MainPage) do |page| + page.goto + page.browser.cookies.add 'mf_useformat', 'true', domain: URI.parse(page.page_url_value).host + page.refresh + end end Given(/^I am on the sign-up page$/) do -- To view, visit https://gerrit.wikimedia.org/r/154217 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7134bbf3031a348e261a73fa354cd8e19ae11b85 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/MobileFrontend Gerrit-Branch: master Gerrit-Owner: Dduvall <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
