Advertisement
julia_v_iluhina

Untitled

Jan 25th, 2017
109
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 5.50 KB | None | 0 0
  1. /*
  2.     не забывай про форматирование кода
  3.     заведи себе такую привычку - код по окончанию работы
  4.     обязательно реформатируй
  5.     это и тебе даст возможность лучше увидеть свой код
  6.     и тому, с кем ты работаешь - это тоже поможет
  7.  
  8.        в IntelIJ Idea
  9.         выдели код
  10.         в меню(верхнем) - выбери пункт  - Code->Reformat code
  11.  
  12.         советую завести привычку - в конце работы - обязательно форматировать код
  13.         вероятнее всего, ты будешь работать в команде, и твой код будут смотреть другие специалисты
  14.         им проще смотреть на стандартно отформатированный код (как правило)
  15.         иногда у команды есть свои правила форматирования, которые чуть отличатся от стандартных.
  16.         Если все сделано по уму, то у всех членов команды в IntelIJ Idea именно так и настроено форматирование кода.
  17.         Т е - все равно - привычка реформатировать код - нужна
  18.  
  19.         https://www.jetbrains.com/help/idea/2016.3/reformatting-source-code.html
  20. */
  21. ***************************************************************
  22.      @Test
  23.      public void featureTestEdit(){
  24.  
  25.         filterAll();
  26.         add("test1");
  27.         edit("test1", "test1 edited");
  28.         assertTasksAre("test1 edited");
  29.  }
  30. /*
  31.     заметь, перейти на какой-то другой фильтр можно только если есть какие-то таски в списке
  32.     так что - сначала - добавляй таски и только затем переходи на нужный фильтр
  33.  
  34.     и то, если это нужно
  35.     в данном случае - filterAll() - лишнее действие
  36.     feature - не нужно в имени метода
  37.     testEditAtAll - уже ясно, что проверяем лишь действие Edit, на таком-то фильтре
  38.     букв почти столько же, а пользы больше
  39.  
  40.     по неймингу - почитай faq, там целый раздел
  41.     усли начинаешь имена тест-методов с test - делай так для всех тест-методов
  42.     будь последовательным)
  43.  
  44.       имя фиче-теста - что тестим и на каком фильтре
  45.  
  46.          структура фиче-теста
  47.            предварительные действия
  48.            тестируемое действие
  49.            проверки
  50.  
  51.          предварительные действия начнем с комментария //given - ...
  52.          чтоб было понятно - что это предварительные действия и что за ситуацию мы в результате их получим
  53.          внутри и в конце блока предварительных действий - проверок не делаем
  54.          (мы это тут не тестируем, а используем для создания тестовой ситуации, ниже будет подробнее)
  55.  
  56.          после предварительных действий - пропустим строку
  57.          чтоб выделить - вот подготовка, вот - тестируемое действие
  58.  
  59.          проверки
  60.          сначала - более важные
  61.          затем - менее важные
  62.          (собственно - так ты и реализовал)
  63.          такой порядок - чтобы даже если тест упадет на менее важной проверке - был фидбек о важной проверке
  64.  
  65.          еще - в фиче-тестах мы можем себе позволить более интересные тестовые ситуации
  66.          например - редактирование второй таски в списке
  67.  
  68.          и не забывай про проверку items left )
  69.          мы это в прошлой работе обсуждали
  70.          продними ревью
  71.          
  72.          в итоге - с учетом этих рекомендаций - получим
  73. */
  74.      @Test
  75.      public void testEditAtAll(){
  76.         //given
  77.         add("test1");
  78.  
  79.         edit("test1", "test1 edited");
  80.         assertTasksAre("test1 edited");
  81.         assertItemsLeft(1);
  82.  }
  83. *********************************************
  84.       @Test
  85.      public void featureTestsDelete&CanceEdit(){
  86.  
  87.          filterActive();
  88.         add("test1","test2");
  89.         delete("test1");
  90.         edit("test3","test2 edited");
  91.         cancelEdit("task2" "task2 edit canceled");
  92.         assertTasksAre("task2 edit canceled");
  93.  
  94.  
  95. }
  96. /*
  97.     не объединяй это в один тест
  98.     так - это не фиче-тест
  99.     а такой маленький е2е
  100.    
  101.     разбей на 2 теста
  102.    
  103.     и учти все выше написанное
  104. */
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement