julia_v_iluhina

Untitled

Aug 1st, 2016
74
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 5.21 KB | None | 0 0
  1.    @Test
  2.     public void testTaskDelete() {
  3.         addTasks("a","b");
  4.         filterActive();
  5.  
  6.         deleteTask("a");
  7.         assertTasksAre("b");
  8.         assertItemsLeft(1);
  9.     }
  10. /*
  11.     http://joxi.ru/5md7jYwtvz941r
  12.     не все советы учел
  13.     и то, и то - важно - учти оба момента для всех фиче-тестов
  14.  
  15.         указать в имени метода - на каком фильтре покрыли - полезно
  16.         в реальности - тесты, реализованные для smoke покрытия - это лишь неболшая часть от всего реализованных тестов
  17.         и хорошо бы - чтобы из названий тест-класса и тест-метода - возникало полное понимание - что мы  тестировали
  18.         в рамках Smoke - мы покрыли Delete  - для активного фильтра
  19.         а для полного покрытия, допустим, мы реализовали аналогичный фиче-тест  - и для All,  и для Completed фильтров
  20.         и вот тут важно - чтобы из названий тест-класса и тест-метода - мы для каждого из таких методов понимали - что он делает
  21.  
  22.         про комментарий //given
  23.         тоже важная и полезная вещь
  24.         сейчас предварительные действия реализованы через действия на UI =
  25.         мы просто вызываем методы-действия над тасками, чтобы получить нужную нам тестовую ситуацию
  26.         и из кода без комментариев - не ясно - это мы тестируем эти действия
  27.         или все же только готовим тестовую ситуацию        
  28.         цель - обозначить - вот это = предварительные действия
  29.         а вот это - мы тестируем
  30.        
  31.         когда предварительные действия будут вызываться как given...(...) - так и в таком комментарии не будет потребности
  32.  
  33.     deleteTask vs TaskDelete ?
  34.     правильнее придерживаться одного термина
  35.     действие = deleteTask
  36.     тест этого действия = testDeleteTask
  37.     тест этого действия на Active фильтре = testDeleteTaskAtActive
  38.  
  39.     test+название действия+на каком фильтре
  40. */
  41. ***********************************************
  42.  
  43.     public void testCancelTaskUpdate(){
  44.         ...
  45.         startTaskUpdate("a", "a edit canceled").pressEscape();
  46.         ...
  47.     }
  48.  
  49.     public void testTaskUpdate(){
  50.         ...
  51.         startTaskUpdate("a", "a edit").pressEnter();
  52.         ...
  53.  
  54. /*
  55.     тут вроде бы стройная картина
  56.  
  57.     термин = TaskUpdate
  58.  
  59.     остальное - производные от него
  60.         startTaskUpdate
  61.         testTaskUpdate
  62.         testCancelTaskUpdate
  63.  
  64.     а теперь вспоминаем про testDeleteTaskAt....
  65.     для Delete - писали - глагол-действие+Task (удалить таску = что делаем + над чем делаем)
  66.     для Update - уже Update-уже существительное (редактирование таски = над чем делаем + какой процесс)
  67.  
  68.     и снова не красиво....
  69.  
  70.     советую придерживаться одной линии
  71.     имя вспомогательного метода = начни с описания действия, раз важно для тебя - уточни субъект - над чем выполняется действие
  72.     имя фиче-теста = test + имя метода-действия + описание контекста
  73.  
  74.     startUpdateTask
  75.     testCancelUpdateTaskAtActive(){
  76.     testUpdateTaskAt...
  77.  
  78.     deleteTask
  79.     testDeleteTask
  80.  
  81.     без Task в именах методов(вспомогательных для действий и фиче-тестов) - было бы попроще)
  82.     но и в твоем варианте можно найти мотивы для именно такого порядка слов в именах методов
  83.     если в вопросах нейминга остаешься при своем мнении насчет места слова Task в именах методов - я поставлю accepted
  84.     accepted - значит принято, но есть вопросы, на которые можно закрыть глаза по моему мнению
  85.  
  86. */
  87. *******************************
  88.  
  89. @Test
  90.     public void testTaskUpdate(){
  91.         addTasks("a");
  92.         filterActive();
  93.  
  94. /*
  95.     Вроде бы запланировано покрытие этого на all
  96.     посмотри в тест-плане
  97. */
Advertisement
Add Comment
Please, Sign In to add comment