julia_v_iluhina

Untitled

Feb 7th, 2017
101
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 5.67 KB | None | 0 0
  1.    @Test
  2.     public void testTasksFlow() {
  3.  
  4. ...
  5.         filterAll();
  6.  
  7.         assertTasksAre("test1");
  8.         assertTasksAre("test1");
  9.     }
  10.     /*
  11.         зачем 2 одинаковые проверки подряд ?
  12.         внимательность)
  13.     */
  14. *********************************
  15.     @Test
  16.     public void testDeleteAtActiveAndtestCancelEditAtActive() {
  17.  
  18.         //given
  19.         add("test1");
  20.  
  21.         filterActive();
  22.         delete("test1");
  23.  
  24.         add("test2");
  25.  
  26.         filterActive();
  27.         cancelEdit("test2", "test2 edited");
  28.         assertTasksAre("test2");
  29.         assertItemsLeft(1);
  30.     }
  31. /*
  32.     хм...
  33.     я не пойму, почему реализуешь один тест-метод, а не 2
  34.    
  35.     ведь мы планировали - именно фиче-тесты
  36.     когда каждый тест-метод - проверяет лишь одно действие
  37.    
  38.     вот это я имела в виду
  39.     см ниже
  40. */  
  41.     @Test
  42.     public void testDeleteAtActive() {
  43.  
  44.         //given
  45.         add("test1", "test2");
  46.         filterActive();
  47.        
  48.         delete("test2");
  49.         assertTasksAre("test1");
  50.         assertItemsLeft(1);
  51.     }  
  52.     /*
  53.         обрати внимание
  54.        
  55.         пропустили строку после filterActive(); - т к это - все еще предварительное действие
  56.         в предварительных действиях - добавили 2 таски
  57.         чтобы иметь возможность после удаления - проверить и assertItemsLeft
  58.         это и покрытие улучшит - мы же еще и assertItemsLeft покрыли
  59.         и тестовая ситуация более интересная - т к работам мы с второй=не единственной таской
  60.     */
  61.        
  62.     @Test
  63.     public void testCancelEditAtActive() {
  64.  
  65.         //given
  66.         add("test1");
  67.         filterActive();
  68.        
  69.         cancelEdit("test1", "test1 edited");
  70.         assertTasksAre("test1");
  71.         assertItemsLeft(1);
  72.     }    
  73.     /*
  74.         тут - логика похожая
  75.     */
  76.  
  77.  *************************************
  78.  /*
  79.     а если бы была цель делать второ е2е
  80.     то он бы был тоже - другой
  81.  */
  82.      @Test
  83.      public void testDeleteAndCancelEditAtActive() {
  84.      /*
  85.         сравни имя метода                         с testDeleteAtActiveAndtestCancelEditAtActive()
  86.         вопрос - мы скрыли что-то важное в варианте testDeleteAndCancelEditAtActive()?
  87.        
  88.         повторять что-либо в имени - нет смысла
  89.      */  
  90.          //given
  91.          add("test1, test2");  
  92.          /*
  93.             если тут добавить лишь одну таску - то мы еще и add покрываем как тестовое действие
  94.             а мы этого не планировали
  95.          */
  96.          filterActive();
  97.          
  98.          delete("test1");
  99.          assertTasksAre("test2");
  100.          /*
  101.             обрати внимание - после каждой операции - проверяем результаты
  102.             если следующая операция не проверяет предыдущую
  103.             то - используй явные проверки вместо неявных через действие
  104.          */                      
  105.          cancelEdit("test2", "test2 edited");
  106.          assertTasksAre("test2");
  107.          assertItemsLeft(1);
  108.      }
  109. /*
  110.     этой задачи = реализовать еще один е2е - не стояло
  111.    
  112.     нам нужно было - реализовать именно фиче-тесты
  113.     выше - я привела их
  114.    
  115.     поскольку это мы уже обсуждали несколько раз
  116.     по этой работе я ставлю accepted = можно двигаться дальше, но к єтой работе остались вопросы
  117.    
  118.     переделывать и снова отдавать эту работу на ревью - не обязательно
  119.     но - судя по реализации
  120.     у тебя или какое-то свое твердое мнение - как должны выглядеть тесты для этой работы
  121.     или все же предыдущие материалы ты недостаточно хорошо усвоил
  122.    
  123.     если с чем-то не согласен - давай планировать, обсудим
  124.     то ли в переписке в слеке
  125.     то ли в разговоре голосом по скайпу
  126.    
  127.     если что-то не ясно - советую разобраться
  128.     и задать все вопросы, которые остались
  129.    
  130.     советую вернуться в видео после работы Smoke test
  131.     и все видео - еще раз последовательно посмотреть
  132.    
  133.     эту работу уже не буду заставлять переделывать
  134.     но если эти же ошибки будут в следующих работах -
  135.     я снова буду настаивать на переделке
  136.    
  137.     советую проработать материал и все для себя выяснить
  138.     все что не понятно
  139.     все с чем не согласен)
  140. */
Advertisement
Add Comment
Please, Sign In to add comment