julia_v_iluhina

Untitled

Dec 11th, 2016
91
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 5.87 KB | None | 0 0
  1. public class TodoMVCTest {
  2.  
  3.     @Test
  4.       public void testCreateTask() {
  5. /*
  6.     Писала тебе в прошлом ревью
  7.  
  8.      В следующий раз - в решении должен быть
  9.           тест-класс TodoMVCTest - будет ок (TodoMVC - название приложения тестируемого, далее пока уточнять ничего не надо)
  10.           тест-метод - подумай над названием, учти вот это -
  11.           https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.2gjiy0o1o48q
  12.  
  13.      Имя тест-класса потправил - это ок
  14.      А вот имя тест-метода - требует доработки
  15.  
  16.     Что мы тестируем в сценарии
  17.         create
  18.         delete
  19.         complete
  20.         complete all
  21.         clear completed
  22.     Если все это отразить в имени - получится громоздко, и потому - не наглядно
  23.     См линку выше - как решить эту проблему - там подробнее описано
  24.     в таких случаях етсты можно назвать testTasksLifeCycle, testTasksCommonFlow
  25.     что-то общее, указывающее - что работаем с тасками
  26. */
  27.         open("https://todomvc4tasj.herokuapp.com/");
  28.         /*
  29.             используй пропуски строки - для разделения кода на блоки
  30.             каждый блок = своя цель
  31.  
  32.             вот тут стоти пропустить строку)
  33.         */
  34.        /*Создаем 4 таска, после чего идёт прверка*/
  35.         $("#new-todo").setValue("Task1").pressEnter();
  36.         $("#new-todo").setValue("Task2").pressEnter();
  37.         $("#new-todo").setValue("Task3").pressEnter();
  38.         $("#new-todo").setValue("Task4").pressEnter();
  39.         $$("#todo-list li").shouldHave(exactTexts("Task1", "Task2", "Task3", "Task4"));
  40.         /*
  41.             Вот - еще один отличный блок
  42.             все ок - и с действиями, и с проверкой
  43.  
  44.             а вот комментарий /*Создаем 4 таска, после чего идёт прверка*/
  45.             уже лишний
  46.             посмотри на код - уже все достаточно понятно
  47.  
  48.             максимум - //create
  49.             а после второго видео - даже такие комментарии не понадобятся
  50.  
  51.             не стоит злоупотреблять комментариями
  52.             будем учиться писать такой код, который не будет в них нуждаться
  53.             ну или почти не будет в них нуждаться)
  54.         */
  55.        /*Удаялем Task 2, после чего идёт проверка */
  56.         $$("#todo-list li").findBy(text("Task2")).hover().$(".destroy").click();
  57.         $$("#todo-list li").shouldHave(exactTexts("task1", "task3", "task4"));
  58.         /*
  59.             еще один блок
  60.             и тут пропусти строку
  61.  
  62.             с блоком - все ок
  63.             комментарий - сделай лаконичнее
  64.         */
  65.         /*Отмечаем что Task 4 выполнен, и затем удяляем его, после чего идёт
  66.         проверка*/
  67.         $$("#todo-list li").findBy(text("Task4")).hover().$(".toggle").click();
  68.         $$("#todo-list li").findBy(text("Task4")).hover().$(".destroy").click();
  69.         $$("#todo-list li").shouldHave(exactTexts("task1", "task3"));
  70.         /*
  71.             новый блок
  72.  
  73.             но - под  clear completed
  74.             имелось в виду другое - нажатие на кнопку - http://joxi.ru/krDOZldFEMa3KA
  75.             а не удаление таски через кнопку удаления
  76.            
  77.             осталось с прошлого ревью
  78.             для нажатия на чекбокс .toggle - не нужно выполнять hover()
  79.                         ранее - для удаления - мы применяли hover() - т к нам нужно было сделать кнопку удаления видимой
  80.                         ведь иначе на нее нажать невозможно
  81.                         понаблюдай - она становится видимой лишь после наведения курсора мыши на таску
  82.              
  83.                         а тут - чекбокс видим
  84.                         просто находи его и выполняй клик
  85.              не надо в строке - $$("#todo-list li").findBy(text("Task4")).hover().$(".toggle").click();
  86.              использовать hover()
  87.         */
  88.         /*Отмечаем все таски как выполненные, затем удаляем их. В конце проверка */
  89.         $("#toggle-all").click();
  90.         $("#clear-completed").click();
  91.         $$("#todo-list li").shouldHave(size(0));
  92.         /*
  93.             и еще один блок
  94.             тут ты кстати верно используешь $("#clear-completed")
  95.            
  96.             проверку
  97.             $$("#todo-list li").shouldHave(size(0));
  98.            
  99.             можно переписать вот так
  100.             $$("#todo-list li").shouldBe(empty);
  101.            
  102.             технически - это одно и то же
  103.             но второй вариант - лаконичнее и нагляднее
  104.         */
  105.  
  106.  
  107.  
  108.     }
  109.  
  110.  
  111. }
Advertisement
Add Comment
Please, Sign In to add comment