Skip to content

Commit c55d448

Browse files
author
Vitaliy Honcharenko
committed
MTA-2395: Tax module functional tests maintenance
- fixed random fails on bamboo
1 parent eb6ea83 commit c55d448

File tree

1 file changed

+9
-8
lines changed
  • dev/tests/functional/tests/app/Magento/Tax/Test/Block/Adminhtml/Rule/Edit

1 file changed

+9
-8
lines changed

dev/tests/functional/tests/app/Magento/Tax/Test/Block/Adminhtml/Rule/Edit/Form.php

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -212,12 +212,14 @@ protected function addNewTaxClass(array $taxClasses, SimpleElement $element)
212212
foreach ($taxClasses as $taxClass) {
213213
$option = $element->find(sprintf($this->optionMaskElement, $taxClass), Locator::SELECTOR_XPATH);
214214
if (!$option->isVisible()) {
215-
$this->clickAddNewButton($element);
216-
$inputSelector = $this->addNewInput;
217215
$element->waitUntil(
218-
function () use ($element, $inputSelector) {
219-
$input = $element->find($inputSelector);
220-
return $input->isVisible() ? true : null;
216+
function () use ($element) {
217+
if ($element->find($this->addNewInput)->isVisible()) {
218+
return true;
219+
} else {
220+
$this->clickAddNewButton($element);
221+
return null;
222+
}
221223
}
222224
);
223225
$element->find($this->addNewInput)->keys([$taxClass]);
@@ -269,10 +271,9 @@ function () use ($browser) {
269271
*/
270272
protected function clickAddNewButton(SimpleElement $element)
271273
{
272-
$addNewButton = $this->addNewButton;
273274
$element->waitUntil(
274-
function () use ($element, $addNewButton) {
275-
return $element->find($addNewButton)->isVisible() ? true : null;
275+
function () use ($element) {
276+
return $element->find($this->addNewButton)->isVisible() ? true : null;
276277
}
277278
);
278279
$element->find($this->addNewButton)->click();

0 commit comments

Comments
 (0)