Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix flaky test

Browse Source 
'StaleElementReferenceException' seems to happen because the selector gets destroyed and created again.
While the placeholder is visible the querySelector finds an element. This commit waits now until the
placeholder DOM structure is removed before trying to access the values using the selecors.

[MAILPOET-5526]
This commit is contained in:
David Remer
2023-08-11 09:57:29 +03:00
committed by Oluwaseun Olorunsola
parent 1096921da7
commit b15065f2a6
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
mailpoet/tests/acceptance/Automation/AnalyticsCest.php
View File

@@ -54,7 +54,8 @@ class AnalyticsCest {
$i->amOnPage("/wp-admin/admin.php?page=mailpoet-automation-analytics&id=" . $this->automation->getId() . "&tab=automation-emails");
$i->see('Overview');
$i->waitForText('Opened');
$i->waitForElement('.woocommerce-summary:not(.is-placeholder)');
$i->see('Opened', '.woocommerce-summary > .woocommerce-summary__item-container');
$i->see('0%', '.woocommerce-summary > .woocommerce-summary__item-container');
$i->see('Clicked', '.woocommerce-summary > .woocommerce-summary__item-container:nth-child(2n)');
@@ -96,7 +97,8 @@ class AnalyticsCest {
$click2 = $this->createClickForNewsletter($this->newsletter1, $subscriber2);
$i->amOnPage("/wp-admin/admin.php?page=mailpoet-automation-analytics&id=" . $this->automation->getId() . "&tab=automation-emails");
$i->waitForText('Opened');
$i->waitForElement('.woocommerce-summary:not(.is-placeholder)');
$i->see('Opened', '.woocommerce-summary > .woocommerce-summary__item-container');
$i->see('1.5%', '.woocommerce-summary > .woocommerce-summary__item-container');
$i->see('Clicked', '.woocommerce-summary > .woocommerce-summary__item-container:nth-child(2n)');
@@ -126,8 +128,8 @@ class AnalyticsCest {
$this->alterCreateDateForOpen($open2, $date);
$i->amOnPage("/wp-admin/admin.php?page=mailpoet-automation-analytics&id=" . $this->automation->getId() . "&tab=automation-emails");
$i->waitForText('Opened');
$i->waitForElement('.woocommerce-summary:not(.is-placeholder)');
$i->see('Opened', '.woocommerce-summary > .woocommerce-summary__item-container:first-child');
$i->see('1%', '.woocommerce-summary > .woocommerce-summary__item-container:first-child .woocommerce-summary__item-value');
$i->see('-80%', '.woocommerce-summary > .woocommerce-summary__item-container:first-child .woocommerce-summary__item-delta');