jenkins-bot has submitted this change and it was merged. Change subject: QA: alphabetize common_article_steps.rb ......................................................................
QA: alphabetize common_article_steps.rb No change to function or text Change-Id: Id7c91373ea6172dfba5a3f62caf42abd627c8aab --- M tests/browser/features/step_definitions/common_article_steps.rb 1 file changed, 14 insertions(+), 14 deletions(-) Approvals: Cmcmahon: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/browser/features/step_definitions/common_article_steps.rb b/tests/browser/features/step_definitions/common_article_steps.rb index 69612ef..ec0f7ff 100644 --- a/tests/browser/features/step_definitions/common_article_steps.rb +++ b/tests/browser/features/step_definitions/common_article_steps.rb @@ -49,8 +49,16 @@ expect(on(ArticlePage).toast_element.when_present.class_name).to match "error" end -Then /^the watch star should be selected$/ do - expect(on(ArticlePage).watch_link_element.parent.class_name).to match "watched" +Then /^I should see a drawer with message "(.+)"$/ do |text| + expect(on(ArticlePage).drawer_element.when_present.text).to match text +end + +Then(/^I should see the error box message "(.+)"$/) do |error_message| + expect(on(ArticlePage).error_message).to match (error_message) +end + +Then(/^I should see the watch star$/) do + expect(on(ArticlePage).watch_link_element).to be_visible end Then(/^the text of the first heading should be "(.*)"$/) do |title| @@ -62,18 +70,10 @@ end end +Then /^the watch star should be selected$/ do + expect(on(ArticlePage).watch_link_element.parent.class_name).to match "watched" +end + Then /^the watch star should not be selected$/ do expect(on(ArticlePage).watch_link_element).to be_visible -end - -Then /^I should see a drawer with message "(.+)"$/ do |text| - expect(on(ArticlePage).drawer_element.when_present.text).to match text -end - -Then(/^I should see the watch star$/) do - expect(on(ArticlePage).watch_link_element).to be_visible -end - -Then(/^I should see the error box message "(.+)"$/) do |error_message| - expect(on(ArticlePage).error_message).to match (error_message) end -- To view, visit https://gerrit.wikimedia.org/r/176938 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7c91373ea6172dfba5a3f62caf42abd627c8aab Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/MobileFrontend Gerrit-Branch: master Gerrit-Owner: Cmcmahon <[email protected]> Gerrit-Reviewer: Cmcmahon <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
