Cmcmahon has submitted this change and it was merged.

Change subject: QA: use newest watch locators, simplify
......................................................................


QA: use newest watch locators, simplify

Change-Id: I3a4598246b8aa83667beb12b622c401684250f73
---
M tests/browser/features/step_definitions/reply_steps.rb
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Cmcmahon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/browser/features/step_definitions/reply_steps.rb 
b/tests/browser/features/step_definitions/reply_steps.rb
index 107cea8..857b687 100644
--- a/tests/browser/features/step_definitions/reply_steps.rb
+++ b/tests/browser/features/step_definitions/reply_steps.rb
@@ -1,8 +1,7 @@
 Given(/^I am not watching my new Flow topic$/) do
   on(FlowPage) do |page|
     page.first_topic_unwatch_link_element.when_present.click
-    page.wait_until { page.first_topic_unwatch_link_element.visible? == false }
-    page.wait_until {page.first_topic_watchlist_loading_link_element.visible? 
== false}
+    page.first_topic_unwatch_link_element.when_not_visible
   end
 end
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3a4598246b8aa83667beb12b622c401684250f73
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Cmcmahon <[email protected]>
Gerrit-Reviewer: Cmcmahon <[email protected]>
Gerrit-Reviewer: SG <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to