jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/404077 )

Change subject: selenium: add deletion test to page spec
......................................................................


selenium: add deletion test to page spec

Change-Id: Ia17b52958778a09378eeb10c9f24ebe9229bd9f5
---
A tests/selenium/pageobjects/delete.page.js
M tests/selenium/pageobjects/userlogin.page.js
M tests/selenium/specs/page.js
3 files changed, 48 insertions(+), 0 deletions(-)

Approvals:
  Zfilipin: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/selenium/pageobjects/delete.page.js 
b/tests/selenium/pageobjects/delete.page.js
new file mode 100644
index 0000000..bb095bb
--- /dev/null
+++ b/tests/selenium/pageobjects/delete.page.js
@@ -0,0 +1,22 @@
+'use strict';
+const Page = require( './page' );
+
+class DeletePage extends Page {
+
+       get reason() { return browser.element( '#wpReason' ); }
+       get watch() { return browser.element( '#wpWatch' ); }
+       get submit() { return browser.element( '#wpConfirmB' ); }
+       get displayedContent() { return browser.element( '#mw-content-text' ); }
+
+       open( name ) {
+               super.open( name + '&action=delete' );
+       }
+
+       delete( name, reason ) {
+               this.open( name );
+               this.reason.setValue( reason );
+               this.submit.click();
+       }
+
+}
+module.exports = new DeletePage();
diff --git a/tests/selenium/pageobjects/userlogin.page.js 
b/tests/selenium/pageobjects/userlogin.page.js
index bdbd41b..0061d0c 100644
--- a/tests/selenium/pageobjects/userlogin.page.js
+++ b/tests/selenium/pageobjects/userlogin.page.js
@@ -19,5 +19,9 @@
                this.loginButton.click();
        }
 
+       loginAdmin() {
+               this.login( browser.options.username, browser.options.password 
);
+       }
+
 }
 module.exports = new UserLoginPage();
diff --git a/tests/selenium/specs/page.js b/tests/selenium/specs/page.js
index e59da78..4ec94eb 100644
--- a/tests/selenium/specs/page.js
+++ b/tests/selenium/specs/page.js
@@ -1,5 +1,6 @@
 'use strict';
 const assert = require( 'assert' ),
+       DeletePage = require( '../pageobjects/delete.page' ),
        EditPage = require( '../pageobjects/edit.page' ),
        HistoryPage = require( '../pageobjects/history.page' ),
        UserLoginPage = require( '../pageobjects/userlogin.page' );
@@ -65,4 +66,25 @@
 
        } );
 
+       it( 'should be deletable', function () {
+
+               // login
+               UserLoginPage.loginAdmin();
+
+               // create
+               browser.call( function () {
+                       return EditPage.apiEdit( name, content );
+               } );
+
+               // delete
+               DeletePage.delete( name, content + '-deletereason' );
+
+               // check
+               assert.equal(
+                       DeletePage.displayedContent.getText(),
+                       '"' + name + '" has been deleted. See deletion log for 
a record of recent deletions.\nReturn to Main Page.'
+               );
+
+       } );
+
 } );

-- 
To view, visit https://gerrit.wikimedia.org/r/404077
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia17b52958778a09378eeb10c9f24ebe9229bd9f5
Gerrit-PatchSet: 6
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Daniel Kinzler <daniel.kinz...@wikimedia.de>
Gerrit-Reviewer: WMDE-Fisch <christoph.jau...@wikimedia.de>
Gerrit-Reviewer: Zfilipin <zfili...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to