Default steps refactored #78
Conversation
• Unit tests added
…Sequence # Conflicts: # src/main/java/ru/alfabank/steps/DefaultSteps.java # src/test/java/ru/alfabank/steps/DefaultStepsTest.java
Fork update
• Public access for string generator method
* Unit tests added
Feature/list elements count
# Conflicts: # src/main/java/ru/alfabank/steps/DefaultSteps.java # src/test/java/ru/alfabank/steps/DefaultStepsTest.java
Default steps rebased
DefaultStepsTest class divided into four classes Imports optimized
| /** | ||
| * Шаги для тестирования элеменотв страницы, доступные по умолчанию в каждом новом проекте. | ||
| * | ||
| * В akitaScenario используется хранилище переменных. Для сохранения/изъятия переменных используются методы setVar/getVar |
There was a problem hiding this comment.
это описание не относится непосредственно к классу ElementSteps. В InputFieldSteps тоже.
Лучше в readme вынести
| /** | ||
| * Проверка, что кнопка/ссылка недоступна для нажатия | ||
| */ | ||
| @Тогда("^(?:ссылка|кнопка) \"([^\"]*)\" недоступна для нажатия$") |
There was a problem hiding this comment.
кажется этот и следующий шаги идентичны)
There was a problem hiding this comment.
И действительно. Подумаю, как это по-русски склеить в один.
| */ | ||
|
|
||
| @Slf4j | ||
| public class ElementSteps { |
There was a problem hiding this comment.
может еще разбить, на ElementInteractionSteps и ElementVerificationSteps
There was a problem hiding this comment.
Логика тут есть, только нужно ли. Сейчас получается четыре примерно равных класса, а так будет три больших и два совсем куцых.
There was a problem hiding this comment.
много мелких - более правильный подход. в них искать/добавлять легче. есть если дублирующиеся шаги - быстрее обнаружить получиться, Single Responsibility типа) я бы разбила вообще на InputInteractionSteps, InputVerificationSteps, ElementsInteractionSteps, ElementsVerificationSteps, ListInteractionSteps и т.д
Кажется так будет проще поддерживать все это многообразие степдефов
…remove # Conflicts: # src/main/java/ru/alfabank/steps/DefaultSteps.java # src/test/java/ru/alfabank/other/AkitaPageTest.java # src/test/java/ru/alfabank/steps/DefaultStepsTest.java
…nteractionSteps, InputFieldSteps, ListSteps and WebPageSteps * DefaultStepsTest class splitted (analogically) * ReadMe updated
DefaultSteps class divided into four classes
DefaultStepsTest class divided into four classes
Imports optimized