Jdlrobson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/109099


Change subject: QA: Fix tests breaking due to I am on the home page step
......................................................................

QA: Fix tests breaking due to I am on the home page step

Make it a given rather than a when as it is not really part of the
test itself

Change-Id: I4675ac5947c1e6e34aa180830f006f7de5716e29
---
M tests/browser/features/language_logged_in.feature
M tests/browser/features/step_definitions/common_steps.rb
2 files changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/99/109099/1

diff --git a/tests/browser/features/language_logged_in.feature 
b/tests/browser/features/language_logged_in.feature
index 2f63e18..64482f9 100644
--- a/tests/browser/features/language_logged_in.feature
+++ b/tests/browser/features/language_logged_in.feature
@@ -3,8 +3,8 @@
 
   Background:
     Given I am logged into the mobile website
+    And I am on the home page
 
   Scenario: Validate Language selection availability
-    When I am on the home page
-      And I click the language button
-    Then I move to the language screen
+    When I click the language button
+    Then I see the language overlay
diff --git a/tests/browser/features/step_definitions/common_steps.rb 
b/tests/browser/features/step_definitions/common_steps.rb
index 79cc704..5051868 100644
--- a/tests/browser/features/step_definitions/common_steps.rb
+++ b/tests/browser/features/step_definitions/common_steps.rb
@@ -61,8 +61,8 @@
   on(NonexistentPage).current_url.should eq NonexistentPage.url
 end
 
-Then(/^I am on the home page$/) do
-  on(HomePage).current_url.should eq HomePage.url
+When(/^I am on the home page$/) do
+  visit(HomePage)
 end
 
 When(/^I click the browser back button$/) do

-- 
To view, visit https://gerrit.wikimedia.org/r/109099
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4675ac5947c1e6e34aa180830f006f7de5716e29
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to