julia_v_iluhina

Untitled

Sep 28th, 2016
85
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.06 KB | None | 0 0
  1.     @Test
  2.     public void deleteTaskAll() {
  3.     /*
  4.         At -  напрасно пропустил
  5.         не очень понятно теперь - All - это что
  6.  
  7.         deleteTaskAtAll - поточнее будет
  8.  
  9.         я бы слово Task из имени тест-метода - убрала
  10.         оперировала - тем же названием, что мы использовали для действия
  11.  
  12.         для такого маленького и простого приложения - ясно что удаляем
  13.  
  14.         ну или уже чтоб совсем однозначно
  15.         deleteTaskAtAllFilter
  16.  
  17.         так тоже можно)
  18.     */
  19.         //Given
  20.         add("a");
  21.         add("b");
  22.         /*
  23.             ну можно и полаконичнее
  24.             add принимает как параметры - и несколько тасок )
  25.         */
  26. *************************
  27.     @Test
  28.     public void editCancelTaskAll() {
  29.         ...
  30.         cancelEdit("a", "a edited");
  31. /*
  32.     действие = отменить редиктирование = cancelEdit
  33.  
  34.     имя тест-метода - аналогично строим
  35.      cancelEditAtAll
  36.      cancelEditTaskAtAllFilter
  37.  
  38.     только - выбери одну схему построения имен
  39.     и ее используй
  40.     иначе - обратного эффекта добьешься
  41.     вроде бы каждое имя - ок
  42.     а вместе - каша...
  43. */
  44. **********************
  45.     @Test
  46.     public void taskWorkFlow() {
  47.  
  48.         ...
  49.         filterActive();
  50.         assertVisibleTasksEmpty();
  51.  
  52.         add("b");
  53.         assertVisibleTasksAre("b");
  54.  
  55.         toggle("b");
  56.         assertVisibleTasksEmpty();
  57. /*
  58.     между toggle("b") и add("b") можно убрать проверку
  59.     т к в этот момент "b" - единственная таска в списке
  60.  
  61.     то toggle("b") - неявно проверит состояние списка
  62. */
Advertisement
Add Comment
Please, Sign In to add comment