01tonythomas has uploaded a new change for review.

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


Change subject: Removed sleep variable from TwnMainPage tests
......................................................................

Removed sleep variable from TwnMainPage tests

Removed sleep variable from signed_in_and_approved feature.

Bug: 46887
Change-Id: Ia96d6a34e03ee0f6fd4c941d19ee729582ec8d46
---
M tests/browser/features/step_definitions/signed_in_and_approved_user_steps.rb
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TwnMainPage 
refs/changes/71/109471/1

diff --git 
a/tests/browser/features/step_definitions/signed_in_and_approved_user_steps.rb 
b/tests/browser/features/step_definitions/signed_in_and_approved_user_steps.rb
index bded465..638c8d8 100644
--- 
a/tests/browser/features/step_definitions/signed_in_and_approved_user_steps.rb
+++ 
b/tests/browser/features/step_definitions/signed_in_and_approved_user_steps.rb
@@ -63,8 +63,7 @@
 
 When(/^I click a message group$/) do
        # The message groups load very slow, it should be speed up
-       sleep 5;
-       on(MainPage).message_group_row_element.when_visible.click
+       on(MainPage).message_group_row_element.when_visible(timeout = 5).click
 end
 
 When(/^I search for "(.*?)"$/) do |query|

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia96d6a34e03ee0f6fd4c941d19ee729582ec8d46
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TwnMainPage
Gerrit-Branch: master
Gerrit-Owner: 01tonythomas <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to