jenkins-bot has submitted this change and it was merged.
Change subject: Revert "[Browser tests] Wait for save button to be enabled"
......................................................................
Revert "[Browser tests] Wait for save button to be enabled"
This reverts commit 122be9c95ae1864656a7805186028072062c1b0b.
Change-Id: Ia79636d5fdcf3f1a224c93688c164bf1030c3bee
---
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, 0 insertions(+), 19 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 3053df1..043a50d 100644
--- a/modules/ve-mw/tests/browser/features/anon.feature
+++ b/modules/ve-mw/tests/browser/features/anon.feature
@@ -5,7 +5,6 @@
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 ca9f71d..b11f284 100644
--- a/modules/ve-mw/tests/browser/features/bullets.feature
+++ b/modules/ve-mw/tests/browser/features/bullets.feature
@@ -7,7 +7,6 @@
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
@@ -20,7 +19,6 @@
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
@@ -35,7 +33,6 @@
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 852a84e..b19e68d 100644
--- a/modules/ve-mw/tests/browser/features/cite.feature
+++ b/modules/ve-mw/tests/browser/features/cite.feature
@@ -23,7 +23,6 @@
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
@@ -44,7 +43,6 @@
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
@@ -60,7 +58,6 @@
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
@@ -77,7 +74,6 @@
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 75a46ef..2ca5082 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,7 +8,6 @@
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 cb684b9..7db0e2b 100644
--- a/modules/ve-mw/tests/browser/features/general_markup.feature
+++ b/modules/ve-mw/tests/browser/features/general_markup.feature
@@ -8,7 +8,6 @@
Scenario Outline: VisualEditor general markup
When I click the text style menu
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
@@ -23,7 +22,6 @@
When I click the text style menu
And I click the More option
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
diff --git a/modules/ve-mw/tests/browser/features/headings.feature
b/modules/ve-mw/tests/browser/features/headings.feature
index d0ca29f..2f3adec 100644
--- a/modules/ve-mw/tests/browser/features/headings.feature
+++ b/modules/ve-mw/tests/browser/features/headings.feature
@@ -8,7 +8,6 @@
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 44a56a0..b69673f 100644
--- a/modules/ve-mw/tests/browser/features/links.feature
+++ b/modules/ve-mw/tests/browser/features/links.feature
@@ -10,7 +10,6 @@
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
- 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
@@ -18,7 +17,6 @@
Scenario: Enter internal link
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
@@ -26,7 +24,6 @@
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
- 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 e7c3933..edbfcbe 100644
--- a/modules/ve-mw/tests/browser/features/media_general_settings.feature
+++ b/modules/ve-mw/tests/browser/features/media_general_settings.feature
@@ -13,7 +13,6 @@
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 67d1645..a83df71 100644
--- a/modules/ve-mw/tests/browser/features/media_interface.feature
+++ b/modules/ve-mw/tests/browser/features/media_interface.feature
@@ -11,7 +11,6 @@
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 996eb9c..136b2c1 100644
--- a/modules/ve-mw/tests/browser/features/options.feature
+++ b/modules/ve-mw/tests/browser/features/options.feature
@@ -32,7 +32,6 @@
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
@@ -46,7 +45,6 @@
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/246397
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia79636d5fdcf3f1a224c93688c164bf1030c3bee
Gerrit-PatchSet: 3
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
