Jdlrobson has uploaded a new change for review.

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

Change subject: QA: Search overlay gives false positives
......................................................................

QA: Search overlay gives false positives

Scenario "Closing search (overlay button)" will pass if the search
overlay never loads. Same goes for
"Closing search (browser button)"

To avoid this add a background check "I see the search overlay"
to ensure we've loaded it first.

Change-Id: Ib2d4883073adeb88028ac84277d691b321bafc39
---
M tests/browser/features/search.feature
1 file changed, 5 insertions(+), 9 deletions(-)


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

diff --git a/tests/browser/features/search.feature 
b/tests/browser/features/search.feature
index 9fc5d4d..5a1b2b8 100644
--- a/tests/browser/features/search.feature
+++ b/tests/browser/features/search.feature
@@ -5,7 +5,8 @@
     Given I am using the mobile site
       And the page "Selenium search test" exists
       And I am on the "Main Page" page
-    When I click the placeholder search box
+      And I click the placeholder search box
+      And I see the search overlay
 
   Scenario: Closing search (overlay button)
     When I click the search overlay close button
@@ -21,15 +22,13 @@
     Then search results should contain "Selenium search test"
 
   Scenario: Search with search in pages button
-      And I see the search overlay
-      And I type into search box "Test is used by Selenium web driver"
+    When I type into search box "Test is used by Selenium web driver"
       And I see the search in pages button
       And I click the search in pages button
     Then I should see a list of search results
 
   Scenario: Search with enter key
-      And I see the search overlay
-      And I type into search box "Test is used by Selenium web driver"
+    When I type into search box "Test is used by Selenium web driver"
       And I press the enter key
     Then I should see a list of search results
 
@@ -50,9 +49,6 @@
 
   @login
   Scenario: Clicking on a watchstar toggles the watchstar
-    Given I am logged into the mobile website
-      And the page "Selenium search test" exists
-    When I click the placeholder search box
-      And I type into search box "Selenium search tes"
+    When I type into search box "Selenium search tes"
       And I click a search watch star
     Then I should see a toast

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib2d4883073adeb88028ac84277d691b321bafc39
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to