Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public class TodoMVCTest {
- @Test
- public void testCreateTask() {
- /*
- Писала тебе в прошлом ревью
- В следующий раз - в решении должен быть
- тест-класс TodoMVCTest - будет ок (TodoMVC - название приложения тестируемого, далее пока уточнять ничего не надо)
- тест-метод - подумай над названием, учти вот это -
- https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.2gjiy0o1o48q
- Имя тест-класса потправил - это ок
- А вот имя тест-метода - требует доработки
- Что мы тестируем в сценарии
- create
- delete
- complete
- complete all
- clear completed
- Если все это отразить в имени - получится громоздко, и потому - не наглядно
- См линку выше - как решить эту проблему - там подробнее описано
- в таких случаях етсты можно назвать testTasksLifeCycle, testTasksCommonFlow
- что-то общее, указывающее - что работаем с тасками
- */
- open("https://todomvc4tasj.herokuapp.com/");
- /*
- используй пропуски строки - для разделения кода на блоки
- каждый блок = своя цель
- вот тут стоти пропустить строку)
- */
- /*Создаем 4 таска, после чего идёт прверка*/
- $("#new-todo").setValue("Task1").pressEnter();
- $("#new-todo").setValue("Task2").pressEnter();
- $("#new-todo").setValue("Task3").pressEnter();
- $("#new-todo").setValue("Task4").pressEnter();
- $$("#todo-list li").shouldHave(exactTexts("Task1", "Task2", "Task3", "Task4"));
- /*
- Вот - еще один отличный блок
- все ок - и с действиями, и с проверкой
- а вот комментарий /*Создаем 4 таска, после чего идёт прверка*/
- уже лишний
- посмотри на код - уже все достаточно понятно
- максимум - //create
- а после второго видео - даже такие комментарии не понадобятся
- не стоит злоупотреблять комментариями
- будем учиться писать такой код, который не будет в них нуждаться
- ну или почти не будет в них нуждаться)
- */
- /*Удаялем Task 2, после чего идёт проверка */
- $$("#todo-list li").findBy(text("Task2")).hover().$(".destroy").click();
- $$("#todo-list li").shouldHave(exactTexts("task1", "task3", "task4"));
- /*
- еще один блок
- и тут пропусти строку
- с блоком - все ок
- комментарий - сделай лаконичнее
- */
- /*Отмечаем что Task 4 выполнен, и затем удяляем его, после чего идёт
- проверка*/
- $$("#todo-list li").findBy(text("Task4")).hover().$(".toggle").click();
- $$("#todo-list li").findBy(text("Task4")).hover().$(".destroy").click();
- $$("#todo-list li").shouldHave(exactTexts("task1", "task3"));
- /*
- новый блок
- но - под clear completed
- имелось в виду другое - нажатие на кнопку - http://joxi.ru/krDOZldFEMa3KA
- а не удаление таски через кнопку удаления
- осталось с прошлого ревью
- для нажатия на чекбокс .toggle - не нужно выполнять hover()
- ранее - для удаления - мы применяли hover() - т к нам нужно было сделать кнопку удаления видимой
- ведь иначе на нее нажать невозможно
- понаблюдай - она становится видимой лишь после наведения курсора мыши на таску
- а тут - чекбокс видим
- просто находи его и выполняй клик
- не надо в строке - $$("#todo-list li").findBy(text("Task4")).hover().$(".toggle").click();
- использовать hover()
- */
- /*Отмечаем все таски как выполненные, затем удаляем их. В конце проверка */
- $("#toggle-all").click();
- $("#clear-completed").click();
- $$("#todo-list li").shouldHave(size(0));
- /*
- и еще один блок
- тут ты кстати верно используешь $("#clear-completed")
- проверку
- $$("#todo-list li").shouldHave(size(0));
- можно переписать вот так
- $$("#todo-list li").shouldBe(empty);
- технически - это одно и то же
- но второй вариант - лаконичнее и нагляднее
- */
- }
- }
Advertisement
Add Comment
Please, Sign In to add comment