Сообщений в теме: 17

[marinel]

Всем добрый день.

Подскажите, пожалуйста, как написать цикл для следующего теста.

 

приложение позволяет создать не более 10 полей при помощи кнопки.

Думаю о том, что бы написать цикл, где будет нажиматься кнопка до тех пор, пока она не исчезнет.

В доступных примерах дается инициализация в виде счетчика, как это можно применить к кнопке?

 

Заранее спасибо.

[Vad1m198]

Ну если нажимать пока не исчезнет, то как то так

 

WebElement el = driver.findElementBy....

 

while(el.isDisplayed())

   el.click();

[user12]

Думаю о том, что бы написать цикл, где будет нажиматься кнопка до тех пор, пока она не исчезнет.

 

 

while(el.isDisplayed())

   el.click();

 

 

 

Вот только никогда нельзя делать так.

А если кнопка никогда не исчезнет и ты получишь бесконечный цикл !

[user12]

private static final int MAX_COUMT_CLICK_BUTTON = 10;



-----

    int i = 0;
        while(el.isDisplayed()) {
            if (MAX_COUMT_CLICK_BUTTON == i) {
                log.error("Click " + MAX_COUMT_CLICK_BUTTON+ " and the button is visible");
                break;
            }
            i++;
              el.click();
        }

[vkazar]

private static final int MAX_COUNT_CLICK_BUTTON = 10;

for (var i=1; i<=MAX_COUNT_CLICK_BUTTON ; i++)
{
    if (el.isDisplayed()) el.Click();
    Assert.IsFalse(el.isDisplayed(), "Кнопка исчезла после совершения " + i + " кликов");
}
Assert.IsTrue(el.isDisplayed(), "Кнопка не исчезла после совершения " + MAX_COUNT_CLICK_BUTTON + " кликов");

[marinel]

Ребята, спасибо вам огромное 

[elvis]

извините за глупый вопрос - а зачем вам это? Я так понимаю кнопка не нажимается сразу и вы жёстко жмёте её до посинения, пока не отработает onclick ?

[breakmt]

private static final int MAX_COUNT_CLICK_BUTTON = 10;

for (var i=1; i<=MAX_COUNT_CLICK_BUTTON ; i++)
{
    if (el.isDisplayed()) el.Click();
    Assert.IsFalse(el.isDisplayed(), "Кнопка исчезла после совершения " + i + " кликов");
}
Assert.IsTrue(el.isDisplayed(), "Кнопка не исчезла после совершения " + MAX_COUNT_CLICK_BUTTON + " кликов");

Помоему у вас после первой же итерации цикла вылетит ошибка проверки, т.к. кнопка исчезает только после 10го клика, а вы проверяете сразу после первого

[marinel]

Все бы хорошо. Да приложение на аяксе и дело усложняют ожидание.

я так понимаю, что цикл не успевает дождаться появление кнопки, решает, что ее нет и заканчивается.

Попробовала добавить ожидание перед тем, как нажимать на кнопку, но видимо это неправильное решение. Помогите, пожалуйста, дальше.

 

for (int second = 0;; second++) {

if (second >= 60) fail("timeout");

try { if ("Disable administration".equals(driver.findElement(By.id("administration")).getText())) break; } catch (Exception e) {}

Thread.sleep(1000);

}

 

Заранее спасибо

[vkazar]

Помоему у вас после первой же итерации цикла вылетит ошибка проверки, т.к. кнопка исчезает только после 10го клика, а вы проверяете сразу после первого

Вы правы, перепутал местами IsTrue и IsFalse :)

private static final int MAX_COUNT_CLICK_BUTTON = 10;

for (var i=1; i<=MAX_COUNT_CLICK_BUTTON ; i++)
{
    if (el.isDisplayed()) el.Click();
    Assert.IsTrue(el.isDisplayed(), "Кнопка исчезла после совершения " + i + " кликов");
}
Assert.IsFalse(el.isDisplayed(), "Кнопка не исчезла после совершения " + MAX_COUNT_CLICK_BUTTON + " кликов");

[breakmt]

 

Все бы хорошо. Да приложение на аяксе и дело усложняют ожидание.

я так понимаю, что цикл не успевает дождаться появление кнопки, решает, что ее нет и заканчивается.

Попробовала добавить ожидание перед тем, как нажимать на кнопку, но видимо это неправильное решение. Помогите, пожалуйста, дальше.

 

for (int second = 0;; second++) {

if (second >= 60) fail("timeout");

try { if ("Disable administration".equals(driver.findElement(By.id("administration")).getText())) break; } catch (Exception e) {}

Thread.sleep(1000);

}

 

Заранее спасибо

 

 

Это неправильное решение, потому что вы используете sleep, что считается плохой практикой. Почитайте про явные и неявные ожидания, например, вот здесь

Еще рекомендую почитать про selenide - в нем многие проблемы и вопросы (в том числе с ожиданиями) уже решены и вам не нужно будет тратить силы и нервы на изобретение велосипеда.

[user12]

2 marinel

Я перестал понимать, как работает ваше приложение.

Думал что

 

Думаю о том, что бы написать цикл, где будет нажиматься кнопка до тех пор, пока она не исчезнет.

 

 просто кликешь на кнопку N раз и она исчезает

Теперь же

цикл не успевает дождаться появление кнопки, решает, что ее нет и заканчивается.

 

Т..е кликнули на кнопку она исчезла, когда появилась - кликнули еще раз (опять исчезла и появилась), на N-ый клик - они исчезла и не появилась ?

Или же открыли страницу, а кнопка еще не появилась, т.е. мы в ПЕРВЫЙ раз хотим нажать на кнопку, а её нет ?

По коду:

1)if (second >= 60)

60 сделайте константой

2) Не используйте пустые

catch (Exception e) {
    }

[marinel]

Виктор, приложение работает так:

есть кнопка "Новое поле" при нажатии на нее, появляется новое поле.

Полей может быть всего 10, тоесть нажать можно 9 раз, т.к. одно поле по умолчанию уже есть.

[Snap]

 

Думаю о том, что бы написать цикл, где будет нажиматься кнопка до тех пор, пока она не исчезнет.

 

while(el.isDisplayed())

   el.click();

 

Вот только никогда нельзя делать так.

А если кнопка никогда не исчезнет и ты получишь бесконечный цикл !

Так это же был просто шаблон, остальное автор додумает (счетчики и т.п.).

[elvis]

1. нажимаем кнопку
2. ждём пока не появится новое поле (ожидание ExpectedConditions)
3. повторяем цикл 9 раз
4. проверяем что кнопка исчезла.
 

[BabyRoot]

Предвижу следующий вопрос: а где вы взяли такой полезный и интересный метод .isDisplayed() ? )))

[marinel]

Всем огромное спасибо!

Получилось решить задачу таким образом:

 

public void createNewMeasure() throws InterruptedException {

for (int x = 0; x < 9; x++){

driver.findElement(By.name("newMeasure")).click();

waitFor();

}

}

 

[vkazar]

Всем огромное спасибо!
Получилось решить задачу таким образом:
 
public void createNewMeasure() throws InterruptedException {
for (int x = 0; x < 9; x++){
driver.findElement(By.name("newMeasure")).click();
waitFor();
}
}
 

а в waitFor() что находится?) опять sleep?)