Fix Timeout Ex when open auction page

[OSerhii]

This change is 

[selurvedu]

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

---

op_robot_tests/tests_files/auction.robot, line 97 at r1 (raw file):

Відкрити сторінку аукціону для ${username}
  ${url}=  Run as  ${username}  Отримати посилання на аукціон для глядача  ${TENDER['TENDER_UAID']}  ${TENDER['LOT_ID']}
  Wait Until Keyword Succeeds  10 x  1 s  Run Keywords

Для чого тут Wait Until Keyword Succeeds, та ще й паузою між спробами в одну секунду?

---

op_robot_tests/tests_files/auction.robot, line 99 at r1 (raw file):

  Wait Until Keyword Succeeds  10 x  1 s  Run Keywords
  ...  Create WebDriver  ${USERS.users['${username}'].browser}
  ...  AND  set_custom_page_load_timeout  15

А by default там скільки секунд?

---

Comments from Reviewable

[OSerhii]

Review status: all files reviewed at latest revision, 2 unresolved discussions.

---

op_robot_tests/tests_files/auction.robot, line 97 at r1 (raw file):

Previously, selurvedu wrote…

Для чого тут Wait Until Keyword Succeeds, та ще й паузою між спробами в одну секунду?

Ідея полягає у тому, щоби у разі якщо протягом 15 секунд вебдрайвер не зміг відкрити сторінку із проведенням аукціону, спробувати перевідкрити сторінку за 10 спроб. В середньому та сторінка відкривається за 6 секунд, можна було і менше поставити, але вирішив про всяк випадок поставити вдвічі більший час. Проміжки можна змінити, якщо забажаєш.

---

op_robot_tests/tests_files/auction.robot, line 99 at r1 (raw file):

Previously, selurvedu wrote…

А by default там скільки секунд?

По дефолту 30 секунд.

---

Comments from Reviewable

[selurvedu]

Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

op_robot_tests/tests_files/auction.robot, line 97 at r1 (raw file):

Previously, OSerhii wrote…

Ідея полягає у тому, щоби у разі якщо протягом 15 секунд вебдрайвер не зміг відкрити сторінку із проведенням аукціону, спробувати перевідкрити сторінку за 10 спроб. В середньому та сторінка відкривається за 6 секунд, можна було і менше поставити, але вирішив про всяк випадок поставити вдвічі більший час. Проміжки можна змінити, якщо забажаєш.

1 s малувато, на повільних машинах браузер може не встигнути закритись.
Ще є сумнів щодо того, як зашвидко повільні і/або сильно завантажені машини будуть відкривати ту сторінку.

---

Comments from Reviewable

[OSerhii]

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

op_robot_tests/tests_files/auction.robot, line 97 at r1 (raw file):

Previously, selurvedu wrote…

1 s малувато, на повільних машинах браузер може не встигнути закритись.
Ще є сумнів щодо того, як зашвидко повільні і/або сильно завантажені машини будуть відкривати ту сторінку.

Я не бачу проблеми, оскільки буде створюватися окремий інстанс вебдрайвера і йому буде до лампочки чи встиг закритися браузер)
На рахунок повільних машин - на скільки я розумію, то set_page_load_timeout встановлює час очікування саме завантаження сторінки, і як довго на машині буде відкриватися браузер значення не має.
Знову ж таки - це механізм. Якщо ви вважаєте що ліпше встановити інші часові проміжки - їх завжди можна змінити. Якщо необхідно - скажіть що на які значення поміняти я одразу це зміню і форс пушну)

---

Comments from Reviewable

[selurvedu]

---

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

op_robot_tests/tests_files/auction.robot, line 97 at r1 (raw file):

Previously, OSerhii wrote…

Я не бачу проблеми, оскільки буде створюватися окремий інстанс вебдрайвера і йому буде до лампочки чи встиг закритися браузер)
На рахунок повільних машин - на скільки я розумію, то set_page_load_timeout встановлює час очікування саме завантаження сторінки, і як довго на машині буде відкриватися браузер значення не має.
Знову ж таки - це механізм. Якщо ви вважаєте що ліпше встановити інші часові проміжки - їх завжди можна змінити. Якщо необхідно - скажіть що на які значення поміняти я одразу це зміню і форс пушну)

OK

---

Comments from Reviewable

[selurvedu]

+conflicts