From 95540a313c7ba4757b2c0bdd6565da6c4b86498f Mon Sep 17 00:00:00 2001 From: WMDE-Fisch Date: Fri, 2 Feb 2024 16:53:02 +0100 Subject: [PATCH] Merge tooltip browser tests No need to build up two test here with refreshing the page. Change-Id: I9bc5b44e2b92704b896edaaa03affac216b9dbce --- tests/selenium/specs/tooltips.js | 33 ++++++++++---------------------- 1 file changed, 10 insertions(+), 23 deletions(-) diff --git a/tests/selenium/specs/tooltips.js b/tests/selenium/specs/tooltips.js index 46d56880..3ffb19fe 100644 --- a/tests/selenium/specs/tooltips.js +++ b/tests/selenium/specs/tooltips.js @@ -7,47 +7,34 @@ describe( 'RevisionSlider revision tooltips', function () { before( async function () { await DiffPage.prepareSimpleTests( 2 ); - } ); - - beforeEach( async function () { DiffPage.ready(); await DiffPage.openSlider(); } ); - afterEach( async function () { - await browser.refresh(); - } ); - - it( 'should appear on hover', async function () { + it( 'appears and remains on hovering it', async function () { await DiffPage.dwellRevision( 1 ); + assert( + await DiffPage.getTooltip( 1 ).isDisplayed(), 'tooltip 1 should appear' + ); + assert( + !await DiffPage.getTooltip( 2 ).isDisplayed(), 'tooltip 2 should not appear' + ); + await DiffPage.dwellRevision( 2 ); assert( await DiffPage.getTooltip( 2 ).isDisplayed(), 'tooltip 2 should appear' ); - assert( - !await DiffPage.getTooltip( 1 ).isDisplayed(), 'tooltip 1 should not appear' + !await DiffPage.getTooltip( 1 ).isDisplayed(), 'tooltip 1 should vanish' ); - } ); - - it( 'appears and remains on hover', async function () { - - await DiffPage.dwellRevision( 1 ); - await DiffPage.getTooltip( 1 ).moveTo(); - - await DiffPage.dwellRevision( 2 ); await DiffPage.getTooltip( 2 ).moveTo(); assert( - await DiffPage.getTooltip( 2 ).isDisplayed(), 'tooltip 2 should appear' - ); - - assert( - !await DiffPage.getTooltip( 1 ).isDisplayed(), 'tooltip 1 should not appear' + await DiffPage.getTooltip( 2 ).isDisplayed(), 'tooltip 2 should still be visible' ); } );