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 <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
