Merge "selenium: Ensure .waitForDisplayed() before .click()"

This commit is contained in:
jenkins-bot 2022-02-04 23:03:28 +00:00 committed by Gerrit Code Review
commit a7ea088346
6 changed files with 6 additions and 0 deletions

View file

@ -103,6 +103,7 @@ const iClickTheBrowserBackButton = () => {
const iClickTheOverlayCloseButton = () => {
waitForPropagation( 2000 );
ArticlePageWithOverlay.overlay_close_element.waitForDisplayed();
ArticlePageWithOverlay.overlay_close_element.click();
};

View file

@ -84,6 +84,7 @@ const watch = ( title ) => {
// So we run the non-js workflow.
const page = new Page();
page.openTitle( title, { action: 'watch' } );
$( '#mw-content-text button[type="submit"]' ).waitForDisplayed();
$( '#mw-content-text button[type="submit"]' ).click();
waitForPropagation( 10000 );
page.openTitle( title );

View file

@ -8,6 +8,7 @@ const iSeeALinkToAboutPage = () => {
};
const iClickOnTheMainNavigationButton = () => {
ArticlePage.menu_button_element.waitForDisplayed();
ArticlePage.menu_button_element.click();
};

View file

@ -12,6 +12,7 @@ const iHaveNoNotifications = () => {
const iClickOnTheNotificationIcon = () => {
ArticlePage.waitUntilResourceLoaderModuleReady( 'skins.minerva.scripts' );
ArticlePage.notifications_button_element.waitForDisplayed();
ArticlePage.notifications_button_element.click();
};

View file

@ -12,6 +12,7 @@ const iShouldSeeNotTheReferenceDrawer = () => {
};
const iClickOnAReference = () => {
ArticlePage.reference_element.waitForDisplayed();
ArticlePage.reference_element.click();
};

View file

@ -5,6 +5,7 @@ const { ArticlePage } = require( '../support/world.js' );
const ArticlePageWithOverlay = require( '../support/pages/article_page_with_overlay' );
const iClickTheSearchIcon = () => {
ArticlePage.search_icon_element.waitForDisplayed();
ArticlePage.search_icon_element.click();
};