jenkins-bot has submitted this change and it was merged.
Change subject: [BrowserTest] comments for classes
......................................................................
[BrowserTest] comments for classes
Change-Id: I3d5e063ee4dd36632ab9c2a433660567b70777ea
---
M .rubocop_todo.yml
M modules/ve-mw/tests/browser/features/support/pages/dummy_page.rb
D modules/ve-mw/tests/browser/features/support/pages/user_page.rb
M modules/ve-mw/tests/browser/features/support/pages/visual_editor_page.rb
M modules/ve-mw/tests/browser/features/support/pages/zarticlepage.rb
5 files changed, 3 insertions(+), 12 deletions(-)
Approvals:
Zfilipin: Looks good to me, approved
jenkins-bot: Verified
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index cf265f1..a45077f 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -27,10 +27,6 @@
Metrics/MethodLength:
Max: 18
-# Offense count: 4
-Style/Documentation:
- Enabled: false
-
# Offense count: 3
# Configuration parameters: AllowedVariables.
Style/GlobalVars:
diff --git a/modules/ve-mw/tests/browser/features/support/pages/dummy_page.rb
b/modules/ve-mw/tests/browser/features/support/pages/dummy_page.rb
index 11f099a..54e4cdd 100644
--- a/modules/ve-mw/tests/browser/features/support/pages/dummy_page.rb
+++ b/modules/ve-mw/tests/browser/features/support/pages/dummy_page.rb
@@ -1,3 +1,4 @@
+# This is used in the verify_production_wikis test
class DummyPage
include PageObject
diff --git a/modules/ve-mw/tests/browser/features/support/pages/user_page.rb
b/modules/ve-mw/tests/browser/features/support/pages/user_page.rb
deleted file mode 100644
index a7c80be..0000000
--- a/modules/ve-mw/tests/browser/features/support/pages/user_page.rb
+++ /dev/null
@@ -1,8 +0,0 @@
-class UserPage
- include PageObject
-
- include URL
- page_url URL.url('<%=params[:page_title]%>')
-
- div(:ve_editing_surface, class: 've-ui-surface
ve-init-mw-viewPageTarget-surface')
-end
diff --git
a/modules/ve-mw/tests/browser/features/support/pages/visual_editor_page.rb
b/modules/ve-mw/tests/browser/features/support/pages/visual_editor_page.rb
index 9b08946..b0b2b97 100644
--- a/modules/ve-mw/tests/browser/features/support/pages/visual_editor_page.rb
+++ b/modules/ve-mw/tests/browser/features/support/pages/visual_editor_page.rb
@@ -1,3 +1,4 @@
+# This is the main page for VisualEditor
class VisualEditorPage
include PageObject
include URL
diff --git a/modules/ve-mw/tests/browser/features/support/pages/zarticlepage.rb
b/modules/ve-mw/tests/browser/features/support/pages/zarticlepage.rb
index ed379ff..732dd2d 100644
--- a/modules/ve-mw/tests/browser/features/support/pages/zarticlepage.rb
+++ b/modules/ve-mw/tests/browser/features/support/pages/zarticlepage.rb
@@ -1,3 +1,4 @@
+# This page object exists because logged in users should not see these URL
params
class ZtargetPage < VisualEditorPage
include URL
page_url
URL.url('<%=params[:article_name]%>?vehidebetadialog=true&veaction=edit')
--
To view, visit https://gerrit.wikimedia.org/r/174975
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3d5e063ee4dd36632ab9c2a433660567b70777ea
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Cmcmahon <[email protected]>
Gerrit-Reviewer: Zfilipin <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
