Kate-Volkova / Unbiased-Sandbox

Automation Testing
0 stars 0 forks source link

смотри, для этого тестового это не ошибка, но вообще в такие методы типа selectAdviseArea, selectIncome и тд стоит передавать параметр. чтобы сразу было в тесте видно, какой доход ты передаешь и чтобы можно было потом метод переиспользовать с другим доходом в другом месте) #15

Open sofiya-mityukova-unbiased-co-uk opened 1 year ago

sofiya-mityukova-unbiased-co-uk commented 1 year ago

https://github.com/Kate-Volkova/Unbiased-Sandbox/blob/618eaa426394577d5ca56ad63771ca81265647a9/src/test/java/unbiased_1/FindAdviserDirectorySearchTest.java#L26

sofiya-mityukova-unbiased-co-uk commented 1 year ago

advisersServiceFinancialAdviserPage.selectIncome("£501,000+"); - вот так бы я хотела видеть в использовании подобных методов

Kate-Volkova commented 1 year ago

Соня, я сначала задавала изменяемые параметры для тестов в основном файле FindAdviserDirectorySearchTest. Потом после твоего комментария о том, что "поля, ответственные за элементы страницы, размещаем в коде класса страницы" разнесла всё по страницам, хотя мне и было странно убирать из теста переменные для него. Но они ведь отвечали за элементы страницы, поэтому перенесла. Теперь, если я правильно понимаю, мне надо вернуть их обратно в тест. В целом согласна. Однако я больше привыкла (меня так учили) передавать параметры через переменные, а не напрямую в метод. Мне кажется удобнее собрать все вводимые параметры в начале теста и оперировать уже переменными. Что скажешь? Если это приципиально задавать параметры напрямую в тексте теста, поясни, пожалуйста, в чём преимущество. Возможно, мне не хватает опыта в том, чтобы его увидеть.

Kate-Volkova commented 1 year ago

увидела ответ в другом комментарии ) вопрос снимаю!