Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public void assertTasksVisible(String... taskTexts) {
- tasks.filter(visible).shouldHave(exactTexts(taskTexts));
- }
- public void assertTasksInvisible() {
- tasks.filter(visible).shouldBe(empty);
- }
- public void assertTasksAre(String... taskTexts) {
- tasks.shouldHave(exactTexts(taskTexts));
- }
- public void assertNoTasks() {
- tasks.shouldBe(empty);
- }
- /*
- для проверки текстов тасок - у нас есть методы
- assertTasksAre
- assertTasksVisible
- а для проверки - что список тасок пуст (или список видимых тасок пуст)
- assertNoTasks()
- assertTasksInvisible()
- разброд и шатание)
- ранее у нас была пара
- assertTasksAre
- assertNoTasks()
- В общем-то - хорошие имена для методов, достаточно точно отражают суть
- появляется вторая пара
- в которой все, что добавляется
- это то, что работаем именно с видимыми тасками
- т е не с Tasks, а с VisibleTasks
- напрашивается логичное решение - и следующую пару методов - назвать - руководствуясь той же логикой
- assertVisibleTasksAre
- assertNoVisibleTasks()
- получается - что мы останемся в тех же терминах и одной логике в нейминге
- название assertTasksInvisible() - вообще может запутать
- мы не проверяем не видимых тасок (их может вообще не быть)
- мы проверяем - видимые таски
- а именно - факт, что их нету
- это подравняй обязательно
- */
- **********************************************
- public SelenideElement startEdit(String oldTaskText, String newTaskText){
- ...
- SelenideElement editedTask = ...
- return editedTask;
- }
- /*
- сравни с вариантом
- */
- public SelenideElement startEdit(String oldTaskText, String newTaskText){
- ...
- return ...;
- }
- /*
- второй вариант - лаконичнее
- и не сказать, что сложнее
- разве что - используя такую переменную - ты с помощью удачно подобранного имени
- можешь получше объяснить - что это такое
- только вот это название editedTask - не идеальное
- edited - это и редактируемая, и отредактированная (а это очень разные вещи, в нашем случае - отличительные)
- по мне, так название только путает)
- идеального варианта не вижу
- для меня, кстати, это тоже индикатор
- если я нормального названия не могу подобрать для локальной переменной
- то я смотрю - а могу я от нее избавиться без потери наглядности
- как правило - могу)
- не настаиваю на переписывании в 2 строки
- но советую)
- */
- *****************************
- /*
- про переходы по фильтрам
- http://pastebin.com/svZarY1D, строки 166-174
- да, тоже не настаиваю
- я надеялась, что после просмотра видео ты изменишь свое мнение)
- */
Advertisement
Add Comment
Please, Sign In to add comment