issues
search
Kate-Volkova
/
Unbiased-Sandbox
Automation Testing
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
Fix2
#24
Kate-Volkova
closed
1 year ago
0
можно более кратко: trueFalse = name.equalsIgnoreCase(ADVISER_NAME);
#23
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
ты попробовала без этой строчки? просто у меня написана в тестовом только одна строчка - Configuration.browserSize = "1920x1080"; и всё работало. такой же вопрос про строчку, которую ты закомментировала. если она не нужна - удали. если вдруг понадобится - гит хранит всю историю изменений, можно будет восстановить
#22
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
названия переменных пишем с маленькой буквы
#21
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
название переменной ни о чем не говорит. можно назвать например isAdviserExsist
#20
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
кажется, программа упадет с ошибкой IndexOutOfBoundsException
#19
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
2
ты когда будешь эти значения переносить в FindAdviserDirectorySearchTest, не тащи константы - ты используешь значения всего один раз, нет необходимости в константах
#18
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
названия переменных пишем с маленькой буквы
#17
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
да, лучше переделать - ты пишешь findAdviser, а я озадачена - ты какого адвайзера ищешь? приходится идти в класс AdvisersServiceFinancialAdviserPage смотреть. а лучше чтобы всё было наглядно
#16
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
смотри, для этого тестового это не ошибка, но вообще в такие методы типа selectAdviseArea, selectIncome и тд стоит передавать параметр. чтобы сразу было в тесте видно, какой доход ты передаешь и чтобы можно было потом метод переиспользовать с другим доходом в другом месте)
#15
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
3
такие штуки делай в одну строку - return resultsList.size(); тебе идея подсвечивает, что здесь проблема?
#14
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
поменяй название. что конкретно делает этот метод? я бы лучше отдельные методы сделала - selectAdviceArea, selectIncome
#13
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
у тебя метод не ипользуется. значит, можно удалить
#12
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
добавь в название Page
#11
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
я бы убрала здесь всё кроме размера окна браузера. а про дженкинс не поняла - тебе придется мне объяснить, что ты задумала. потому что у меня окрывается браузер и твоя программа выполняется в нем
#10
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
когда имена классов, методов, переменных говорят за себя, комментарии излишни. можно почитать по этой теме книгу Чистый код
#9
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
тут используй метод byText(). такие длинные селектры тяжело читать, они очень хрупкие. чем лаконичнее, тем лучше
#8
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
создай пакет page и отправь туда все классы-page
#7
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
=методы твоих классов, ответственных за page, должны называться более конкретно. то есть тут надо сделать вызов mainPage.openBasePage(), а baseUrl разместить в самом классе MainPage
#6
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
вынеси все инициализации объектов в метод init и пометь его аннотацией @Before из Junit
#5
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
я бы изменила имя на более говорящее. это список каких результатов?
#4
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
поля, ответственные за элементы страницы, размещаем в коде класса страницы. это принцип Single Responsibility из SOLID и ещё паттерн Page Object
#3
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1
все private final static поля - константы. их имена пишутся капслоком через нижнее подчеркивание
#2
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
0
Переименуй класс
#1
sofiya-mityukova-unbiased-co-uk
opened
1 year ago
1