diff options
| author | Željko Filipin <zeljko.filipin@gmail.com> | 2024-09-11 10:55:58 +0200 |
|---|---|---|
| committer | Željko Filipin <zeljko.filipin@gmail.com> | 2024-09-11 10:55:58 +0200 |
| commit | bc772ddda093951e7624224af98835ce86847f3c (patch) | |
| tree | 002ca41e0cc3d63c5493316534092f760d1cd20a /tests/selenium/docs/Stack | |
| parent | d38689ae1d7a74cda9df88d9e747b455b66653d6 (diff) | |
| download | mediawikicore-bc772ddda093951e7624224af98835ce86847f3c.tar.gz mediawikicore-bc772ddda093951e7624224af98835ce86847f3c.zip | |
selenium: Main page should have "Log in" link
Make it explicit that there are three separate tests
by expanding test names.
Until now, these three tests were doing the same thing,
but in a different way. All three tests had the same name.
That made it harder to debug if anything went wrong.
For example, all three tests created the screenshot file
with the same name.
Bug: T373125
Change-Id: I53f62186fd4e8ffbe5e93642c933da2493b91896
Diffstat (limited to 'tests/selenium/docs/Stack')
| -rw-r--r-- | tests/selenium/docs/Stack/specs/assert.js | 2 | ||||
| -rw-r--r-- | tests/selenium/docs/Stack/specs/mocha.js | 2 | ||||
| -rw-r--r-- | tests/selenium/docs/Stack/specs/pageobject.js | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/tests/selenium/docs/Stack/specs/assert.js b/tests/selenium/docs/Stack/specs/assert.js index d9cdb2ef1b9d..46052496e5a7 100644 --- a/tests/selenium/docs/Stack/specs/assert.js +++ b/tests/selenium/docs/Stack/specs/assert.js @@ -12,7 +12,7 @@ const baseUrl = `${ process.env.MW_SERVER }${ process.env.MW_SCRIPT_PATH }/index const assert = require( 'assert' ); describe( 'Main page', () => { - it( 'should have "Log in" link', async () => { + it( 'should have "Log in" link when using assert', async () => { await browser.url( `${ baseUrl }/Main_Page` ); const displayed = await $( 'li#pt-login-2 a' ).isDisplayed(); assert( displayed ); diff --git a/tests/selenium/docs/Stack/specs/mocha.js b/tests/selenium/docs/Stack/specs/mocha.js index 18c4c2f60f3b..8c4c586a88b3 100644 --- a/tests/selenium/docs/Stack/specs/mocha.js +++ b/tests/selenium/docs/Stack/specs/mocha.js @@ -10,7 +10,7 @@ const baseUrl = `${ process.env.MW_SERVER }${ process.env.MW_SCRIPT_PATH }/index.php?title=`; describe( 'Main page', () => { - it( 'should have "Log in" link', async () => { + it( 'should have "Log in" link when using mocha', async () => { await browser.url( `${ baseUrl }/Main_Page` ); const displayed = await $( 'li#pt-login-2 a' ).isDisplayed(); if ( displayed === false ) { diff --git a/tests/selenium/docs/Stack/specs/pageobject.js b/tests/selenium/docs/Stack/specs/pageobject.js index 1035209583bb..bd6e65be7067 100644 --- a/tests/selenium/docs/Stack/specs/pageobject.js +++ b/tests/selenium/docs/Stack/specs/pageobject.js @@ -7,7 +7,7 @@ const assert = require( 'assert' ); const MainPage = require( '../pageobjects/main.page' ); describe( 'Main Page', () => { - it( 'should have "Log in" link', async () => { + it( 'should have "Log in" link when using page object', async () => { await MainPage.open(); assert( await MainPage.login.isDisplayed() ); } ); |