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