selenium: Replace ES5 one-var assignments with const/let per line

[lhc/web/wiklou.git] / tests / selenium / specs / page.js

diff --git a/tests/selenium/specs/page.js b/tests/selenium/specs/page.js
index 24e1d65..464fbb2 100644 (file)
--- a/tests/selenium/specs/page.js
+++ b/tests/selenium/specs/page.js
@@ -1,22 +1,18 @@
-const assert = require( 'assert' ),
-       Api = require( 'wdio-mediawiki/Api' ),
-       BlankPage = require( 'wdio-mediawiki/BlankPage' ),
-       DeletePage = require( '../pageobjects/delete.page' ),
-       RestorePage = require( '../pageobjects/restore.page' ),
-       EditPage = require( '../pageobjects/edit.page' ),
-       HistoryPage = require( '../pageobjects/history.page' ),
-       UndoPage = require( '../pageobjects/undo.page' ),
-       UserLoginPage = require( 'wdio-mediawiki/LoginPage' ),
-       Util = require( 'wdio-mediawiki/Util' );
+const assert = require( 'assert' );
+const Api = require( 'wdio-mediawiki/Api' );
+const DeletePage = require( '../pageobjects/delete.page' );
+const RestorePage = require( '../pageobjects/restore.page' );
+const EditPage = require( '../pageobjects/edit.page' );
+const HistoryPage = require( '../pageobjects/history.page' );
+const UndoPage = require( '../pageobjects/undo.page' );
+const UserLoginPage = require( 'wdio-mediawiki/LoginPage' );
+const Util = require( 'wdio-mediawiki/Util' );
 
 describe( 'Page', function () {
-       var content,
-               name;
+       let content, name, bot;
 
-       before( function () {
-               // disable VisualEditor welcome dialog
-               BlankPage.open();
-               browser.setLocalStorage( 've-beta-welcome-dialog', '1' );
+       before( async function () {
+               bot = await Api.bot();
        } );
 
        beforeEach( function () {
@@ -46,17 +42,13 @@ describe( 'Page', function () {
        it( 'should be re-creatable', function () {
                const initialContent = Util.getTestString( 'initialContent-' );
 
-               // create
-               browser.call( function () {
-                       return Api.edit( name, initialContent );
-               } );
-
-               // delete
-               browser.call( function () {
-                       return Api.delete( name, 'delete prior to recreate' );
+               // create and delete
+               browser.call( async () => {
+                       await bot.edit( name, initialContent, 'create for delete' );
+                       await bot.delete( name, 'delete prior to recreate' );
                } );
 
-               // create
+               // re-create
                EditPage.edit( name, content );
 
                // check
@@ -66,8 +58,8 @@ describe( 'Page', function () {
 
        it( 'should be editable @daily', function () {
                // create
-               browser.call( function () {
-                       return Api.edit( name, content );
+               browser.call( async () => {
+                       await bot.edit( name, content, 'create for edit' );
                } );
 
                // edit
@@ -81,26 +73,26 @@ describe( 'Page', function () {
 
        it( 'should have history @daily', function () {
                // create
-               browser.call( function () {
-                       return Api.edit( name, content );
+               browser.call( async () => {
+                       await bot.edit( name, content, `created with "${content}"` );
                } );
 
                // check
                HistoryPage.open( name );
-               assert.strictEqual( HistoryPage.comment.getText(), `Created or updated page with "${content}"` );
+               assert.strictEqual( HistoryPage.comment.getText(), `created with "${content}"` );
        } );
 
        it( 'should be deletable', function () {
-               // login
-               UserLoginPage.loginAdmin();
-
                // create
-               browser.call( function () {
-                       return Api.edit( name, content );
+               browser.call( async () => {
+                       await bot.edit( name, content, 'create for delete' );
                } );
 
+               // login
+               UserLoginPage.loginAdmin();
+
                // delete
-               DeletePage.delete( name, content + '-deletereason' );
+               DeletePage.delete( name, 'delete reason' );
 
                // check
                assert.strictEqual(
@@ -110,40 +102,32 @@ describe( 'Page', function () {
        } );
 
        it( 'should be restorable', function () {
-               // login
-               UserLoginPage.loginAdmin();
-
-               // create
-               browser.call( function () {
-                       return Api.edit( name, content );
+               // create and delete
+               browser.call( async () => {
+                       await bot.edit( name, content, 'create for delete' );
+                       await bot.delete( name, 'delete for restore' );
                } );
 
-               // delete
-               browser.call( function () {
-                       return Api.delete( name, content + '-deletereason' );
-               } );
+               // login
+               UserLoginPage.loginAdmin();
 
                // restore
-               RestorePage.restore( name, content + '-restorereason' );
+               RestorePage.restore( name, 'restore reason' );
 
                // check
                assert.strictEqual( RestorePage.displayedContent.getText(), name + ' has been restored\nConsult the deletion log for a record of recent deletions and restorations.' );
        } );
 
        it( 'should be undoable', function () {
-               // create
-               browser.call( function () {
-                       return Api.edit( name, content );
-               } );
-
-               // edit
                let previousRev, undoRev;
-               browser.call( function () {
-                       return Api.edit( name, Util.getTestString( 'editContent-' ) )
-                               .then( ( response ) => {
-                                       previousRev = response.edit.oldrevid;
-                                       undoRev = response.edit.newrevid;
-                               } );
+               browser.call( async () => {
+                       // create
+                       await bot.edit( name, content, 'create to edit and undo' );
+
+                       // edit
+                       const response = await bot.edit( name, Util.getTestString( 'editContent-' ) );
+                       previousRev = response.edit.oldrevid;
+                       undoRev = response.edit.newrevid;
                } );
 
                UndoPage.undo( name, previousRev, undoRev );