jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/370667 )

Change subject: QA: Relax toast notification QA steps
......................................................................


QA: Relax toast notification QA steps

It seems trying to test both the steps can cause
false positives. Relaxing these checks seems to make
our Jenkins job happy without breaking the tests themselves

Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
---
M tests/browser/features/step_definitions/common_article_steps.rb
1 file changed, 2 insertions(+), 3 deletions(-)

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



diff --git a/tests/browser/features/step_definitions/common_article_steps.rb 
b/tests/browser/features/step_definitions/common_article_steps.rb
index df5d4fd..c08ee3a 100644
--- a/tests/browser/features/step_definitions/common_article_steps.rb
+++ b/tests/browser/features/step_definitions/common_article_steps.rb
@@ -32,14 +32,13 @@
     # Minerva loads mediawiki.notify at startup which defers the loading of 
this module
     # We must wait until the lazy loading has happened before checking for the 
toast (T170890)
     page.wait_until_rl_module_ready('mediawiki.notification')
+    expect(page.notification_area_element.when_visible).to be_visible
   end
-  expect(on(ArticlePage).notification_area_element.when_visible).to be_visible
-  expect(on(ArticlePage).toast_element.when_visible).to be_visible
 end
 
 Then(/^I should see a toast with message "(.+)"$/) do |msg|
-  step "I should see a toast notification"
   on(ArticlePage) do |page|
+    page.wait_until_rl_module_ready('mediawiki.notification')
     page.wait_until do
       page.toast_element.when_present.text.include? msg
     end

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/MinervaNeue
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: Pmiazga <pmia...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to