julia_v_iluhina

Untitled

Oct 4th, 2016
74
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.92 KB | None | 0 0
  1.     @Test
  2.     public void testActivateAll() {
  3.     /*
  4.         лучше уточнить testActivateAllAtCompeted
  5.  
  6.         смести этот метод ниже остальных фиче-тестов
  7.         и пометь метод комментарием extra coverage
  8.         т к в смоук покрытии - покрывать не высокоприоритетное не надо
  9.  
  10.         убирать  - не стоит
  11.         но пометить - что это экстра - надо
  12.     */
  13.  
  14.         //given
  15.         /*
  16.             given - completed tasks
  17.             будет чуть понятнее
  18.         */
  19.         page.create("a", "b", "c", "d");
  20.         page.toggleAll();
  21.         page.filterCompleted();
  22.  
  23.         page.toggleAll();
  24.         /*
  25.             тут - проверь состояние списка и items left
  26.             этого будет достаточно
  27.  
  28.             т к уже проверяли - что активные таски не отображаются на Completed фильтре
  29.             но они есть (переходили на all и єто проверяли)
  30.             посмотри на свой же е2е
  31.  
  32.             items left покрываем - потому что это не загромождает код
  33.             и потому что нам по пути
  34.             потом - ради проверки только єтого - городить отдельные фиче-тесты
  35.             с практически таким же кодом - смысла не будет
  36.         */
  37.         page.filterActive();
  38.         page.assertVisibleTasks("a", "b", "c", "d");
  39.     }
  40.  
  41.     @Test
  42.     public void testEditByTab() {
  43.     /*
  44.         мы убрали из е2е  - edit
  45.         вот его и реализуем
  46.  
  47.         EditByTab - это тоже extra coverage
  48.         т к приоритет у этого действия - ниже
  49.  
  50.         почему ты не придерживался своего же плана?
  51.         зачем тогда нужен план)
  52.  
  53.         хочешь - оставляй testEditByTab как extra coverage
  54.         правильно размести метод - после комментария про extra coverage
  55.         но тогда стоит на другом фильтре это покрывать (по сравнению с testEdit)
  56.         да и другую тестовую ситуацию использовать - например редактировать вторую таску в списке
  57.  
  58.         а хочешь - просто переделай этот метод на testEdit
  59.         советую именно так поступить)
  60.  
  61.         в имени метода - уточни фильтр
  62.     */
  63.  
  64.         //given
  65.         page.create("a");
  66.  
  67.         page.startEdit("a", "a edited").pressTab();
  68.         page.assertVisibleTasks("a edited");
  69.         /*
  70.             добавь проверку items left
  71.             покроем по пути
  72.         */
  73.     }
  74.  
  75.     @Test
  76.     public void testCancelEditByEsc() {
  77.     /*
  78.         планировалось покрытие - не на
  79.         уточни имя
  80.         и покрой на Active фильтре
  81.        
  82.         цель - распределить покрытие по разным фильтрам
  83.        
  84.         и  можно разнообразить тестовую ситуацию (по сравнению с edit)        
  85.         например можно работать со второй таской из списка
  86.     */
  87.  
  88.         //given
  89.         page.create("b");
  90.  
  91.         page.startEdit("b", "b cancel edit").pressEscape();
  92.         page.assertVisibleTasks("b");
  93.          /*
  94.             добавь проверку items left
  95.             покроем по пути
  96.          */
  97.     }
Advertisement
Add Comment
Please, Sign In to add comment