From cb2634a78b2fb296a1f081aca6ff11c85a1b924b Mon Sep 17 00:00:00 2001 From: jdlrobson Date: Tue, 8 Jan 2019 11:52:36 -0800 Subject: [PATCH] Constraint page issues CSS selector The 2 selector approach is flawed as the .ambox element matches a hidden element in the old treatment which is not clickable. I suspect this change will fix the failing browser test on the beta cluster. Integration tests will continue to test the new selector in the mean time. Change-Id: I44a873b2e89069c4a47a428c528592159520568c --- tests/browser/features/support/pages/article_page.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/browser/features/support/pages/article_page.rb b/tests/browser/features/support/pages/article_page.rb index b66ea60..004efb3 100644 --- a/tests/browser/features/support/pages/article_page.rb +++ b/tests/browser/features/support/pages/article_page.rb @@ -179,9 +179,9 @@ class ArticlePage # issues # We use 2 selectors here - the first relates to the old treatment (A) and - # the 2nd relates to the new treatment (.ambox) + # the 2nd relates to the new treatment (.issues-group-B .ambox) # see https://phabricator.wikimedia.org/T206647 - a(:issues_stamp, css: '.mw-mf-cleanup, .ambox') + a(:issues_stamp, css: '.mw-mf-cleanup, .issues-group-B .ambox') # page info (action=info) td(:edit_count, css: '#mw-pageinfo-edits td', index: 1)