jenkins-bot has submitted this change and it was merged. Change subject: [Browser tests] Wait for save button to be enabled ......................................................................
[Browser tests] Wait for save button to be enabled In some tests the save button won't be available for a few ms until the latest transaction has been pushed to the undo stack. Change-Id: If03f03b033a21a039a322ca32983d0302e44efbb --- M modules/ve-mw/tests/browser/features/anon.feature M modules/ve-mw/tests/browser/features/bullets.feature M modules/ve-mw/tests/browser/features/cite.feature M modules/ve-mw/tests/browser/features/edit_utf8_logged_in.feature M modules/ve-mw/tests/browser/features/general_markup.feature M modules/ve-mw/tests/browser/features/headings.feature M modules/ve-mw/tests/browser/features/links.feature M modules/ve-mw/tests/browser/features/media_general_settings.feature M modules/ve-mw/tests/browser/features/media_interface.feature M modules/ve-mw/tests/browser/features/options.feature 10 files changed, 61 insertions(+), 42 deletions(-) Approvals: Jforrester: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/ve-mw/tests/browser/features/anon.feature b/modules/ve-mw/tests/browser/features/anon.feature index 043a50d..3053df1 100644 --- a/modules/ve-mw/tests/browser/features/anon.feature +++ b/modules/ve-mw/tests/browser/features/anon.feature @@ -5,6 +5,7 @@ Given I go to the "Anonymous Edit Test" page with content "Anonymous Edit Test" When I edit the page with Editing with And I see the IP warning signs + Then Save page should be visible And I click Save page And I do not see This is a minor edit And I click Review your changes diff --git a/modules/ve-mw/tests/browser/features/bullets.feature b/modules/ve-mw/tests/browser/features/bullets.feature index b11f284..ca9f71d 100644 --- a/modules/ve-mw/tests/browser/features/bullets.feature +++ b/modules/ve-mw/tests/browser/features/bullets.feature @@ -7,6 +7,7 @@ Scenario Outline: check strings for bullets and numbering When I click <control> + Then Save page should be visible And I click Save page And I click Review your changes Then a <character> is added in front of input string in the diff view @@ -19,6 +20,7 @@ Scenario Outline: check increase indent for bullets and numbering When I click <control> And I click Increase indentation + Then Save page should be visible And I click Save page And I click Review your changes Then a <character> is added in front of input string in the diff view @@ -33,6 +35,7 @@ Scenario Outline: check decrease indent for bullets and numbering When I click <control> And I click Decrease indentation + Then Save page should be visible And I click Save page And I click Review your changes Then nothing is added in front of input string in the diff view diff --git a/modules/ve-mw/tests/browser/features/cite.feature b/modules/ve-mw/tests/browser/features/cite.feature index b19e68d..852a84e 100644 --- a/modules/ve-mw/tests/browser/features/cite.feature +++ b/modules/ve-mw/tests/browser/features/cite.feature @@ -23,6 +23,7 @@ And I click the new field label And I fill in the new field "New website field contents" And I click Insert Citation + Then Save page should be visible And I click Save page And I click Review your changes Then diff view should show the Website citation added @@ -43,6 +44,7 @@ And I click the new field label And I fill in the new field "New book field contents" And I click Insert Citation + Then Save page should be visible And I click Save page And I click Review your changes Then diff view should show the Book citation added @@ -58,6 +60,7 @@ And I fill in the seventh textarea with "News URL access date" And the News input field titles are in the correct order And I click Insert Citation + Then Save page should be visible And I click Save page And I click Review your changes Then diff view should show the News citation added @@ -74,6 +77,7 @@ And I fill in the eighth textarea with "Journal DOI" And the Journal input field titles are in the correct order And I click Insert Citation + Then Save page should be visible And I click Save page And I click Review your changes Then diff view should show the Journal citation added diff --git a/modules/ve-mw/tests/browser/features/edit_utf8_logged_in.feature b/modules/ve-mw/tests/browser/features/edit_utf8_logged_in.feature index 2ca5082..75a46ef 100644 --- a/modules/ve-mw/tests/browser/features/edit_utf8_logged_in.feature +++ b/modules/ve-mw/tests/browser/features/edit_utf8_logged_in.feature @@ -8,6 +8,7 @@ Scenario Outline: Edit with strings When I edit the page with <input_string> + Then Save page should be visible And I click Save page And I click This is a minor edit And I click Review your changes diff --git a/modules/ve-mw/tests/browser/features/general_markup.feature b/modules/ve-mw/tests/browser/features/general_markup.feature index 6b41309..cb684b9 100644 --- a/modules/ve-mw/tests/browser/features/general_markup.feature +++ b/modules/ve-mw/tests/browser/features/general_markup.feature @@ -7,11 +7,12 @@ Scenario Outline: VisualEditor general markup When I click the text style menu - And I click the <type_of_markup> menu option - And I click Save page - And I click Review your changes + And I click the <type_of_markup> menu option + Then Save page should be visible + And I click Save page + And I click Review your changes Then <expected_markup_text> should appear in the diff view - And I can click the X on the save box + And I can click the X on the save box Examples: | type_of_markup | expected_markup_text | | Bold | '''General Markup VisualEditor Test''' | @@ -21,11 +22,12 @@ Scenario Outline: VisualEditor more general markup When I click the text style menu And I click the More option - And I click the <type_of_markup> menu option - And I click Save page - And I click Review your changes + And I click the <type_of_markup> menu option + Then Save page should be visible + And I click Save page + And I click Review your changes Then <expected_markup_text> should appear in the diff view - And I can click the X on the save box + And I can click the X on the save box Examples: | type_of_markup | expected_markup_text | | Computer Code | <code>General Markup VisualEditor Test</code> | diff --git a/modules/ve-mw/tests/browser/features/headings.feature b/modules/ve-mw/tests/browser/features/headings.feature index 2f3adec..d0ca29f 100644 --- a/modules/ve-mw/tests/browser/features/headings.feature +++ b/modules/ve-mw/tests/browser/features/headings.feature @@ -8,6 +8,7 @@ Scenario Outline: Cycle through headings values When I click the down arrow on the Headings menu And I click <headings_interface_name> + Then Save page should be visible And I click Save page And I click Review your changes Then <headings_string> should appear in the diff view diff --git a/modules/ve-mw/tests/browser/features/links.feature b/modules/ve-mw/tests/browser/features/links.feature index 554ead8..44a56a0 100644 --- a/modules/ve-mw/tests/browser/features/links.feature +++ b/modules/ve-mw/tests/browser/features/links.feature @@ -8,22 +8,25 @@ And I can see the Link User Inteface Scenario: Enter external link - When I enter external link http://www.example.com into link Content box - And I click Done to close Link User Interface - And I click Save page - And I click Review your changes - Then an external link appears in the diff view + When I enter external link http://www.example.com into link Content box + And I click Done to close Link User Interface + Then Save page should be visible + And I click Save page + And I click Review your changes + Then an external link appears in the diff view Scenario: Enter internal link - When I enter internal link Main Page into link Content box - And I click Done to close Link User Interface - And I click Save page - And I click Review your changes - Then an internal link appears in the diff view + When I enter internal link Main Page into link Content box + And I click Done to close Link User Interface + Then Save page should be visible + And I click Save page + And I click Review your changes + Then an internal link appears in the diff view Scenario: Enter non-existing link When I enter non existing link DoesNotExist into link Content box - And I click Done to close Link User Interface - And I click Save page - And I click Review your changes - Then a non-existing link appears in the diff view + And I click Done to close Link User Interface + Then Save page should be visible + And I click Save page + And I click Review your changes + Then a non-existing link appears in the diff view diff --git a/modules/ve-mw/tests/browser/features/media_general_settings.feature b/modules/ve-mw/tests/browser/features/media_general_settings.feature index 5a1ed40..e7c3933 100644 --- a/modules/ve-mw/tests/browser/features/media_general_settings.feature +++ b/modules/ve-mw/tests/browser/features/media_general_settings.feature @@ -3,16 +3,17 @@ Background: Given I go to the "Media Interface VisualEditor Test" page with content "Media Interface VisualEditor Test" - And I click in the editable part + And I click in the editable part Scenario: VisualEditor insert new media Given I click Media - And I enter bug into media Search box - And I select an Image - And I click Use this image - And I fill up the Caption field with "caption" - And I fill up the Alternative text with "alt text" - And I click Insert - And I click Save page - And I click Review your changes + And I enter bug into media Search box + And I select an Image + And I click Use this image + And I fill up the Caption field with "caption" + And I fill up the Alternative text with "alt text" + And I click Insert + Then Save page should be visible + And I click Save page + And I click Review your changes Then diff view should show media file with caption and alt text diff --git a/modules/ve-mw/tests/browser/features/media_interface.feature b/modules/ve-mw/tests/browser/features/media_interface.feature index a83df71..67d1645 100644 --- a/modules/ve-mw/tests/browser/features/media_interface.feature +++ b/modules/ve-mw/tests/browser/features/media_interface.feature @@ -11,6 +11,7 @@ And I select an Image And I click Use this image And I click Insert + Then Save page should be visible And I click Save page And I click Review your changes Then diff view should show correct markup diff --git a/modules/ve-mw/tests/browser/features/options.feature b/modules/ve-mw/tests/browser/features/options.feature index 1531c28..996eb9c 100644 --- a/modules/ve-mw/tests/browser/features/options.feature +++ b/modules/ve-mw/tests/browser/features/options.feature @@ -32,19 +32,21 @@ And I check the option for Enable display title And I type "automated test" for display title textbox And I click Apply Changes button + Then Save page should be visible And I click Save page And I click Review your changes Then the options set in Advanced Settings panel should appear in diff view Scenario: Setting the fields in Page Settings - When I click Page Settings - And I check the option for Redirect this page to - And I type "Linux" for redirect page - And I check the option for Prevent this redirect being updated when target page is moved - And I select the option Always for showing Table of Contents - And I check the option for Disable edit links next to each heading on this page - And I check the option for This is a Disambiguation page - And I click Apply Changes button - And I click Save page - And I click Review your changes - Then the options set in Page Settings panel should appear in diff view + When I click Page Settings + And I check the option for Redirect this page to + And I type "Linux" for redirect page + And I check the option for Prevent this redirect being updated when target page is moved + And I select the option Always for showing Table of Contents + And I check the option for Disable edit links next to each heading on this page + And I check the option for This is a Disambiguation page + And I click Apply Changes button + Then Save page should be visible + And I click Save page + And I click Review your changes + Then the options set in Page Settings panel should appear in diff view -- To view, visit https://gerrit.wikimedia.org/r/246247 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: If03f03b033a21a039a322ca32983d0302e44efbb Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/VisualEditor Gerrit-Branch: master Gerrit-Owner: Esanders <esand...@wikimedia.org> Gerrit-Reviewer: Esanders <esand...@wikimedia.org> Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits