Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- /*
- не забывай про форматирование кода
- заведи себе такую привычку - код по окончанию работы
- обязательно реформатируй
- это и тебе даст возможность лучше увидеть свой код
- и тому, с кем ты работаешь - это тоже поможет
- в IntelIJ Idea
- выдели код
- в меню(верхнем) - выбери пункт - Code->Reformat code
- советую завести привычку - в конце работы - обязательно форматировать код
- вероятнее всего, ты будешь работать в команде, и твой код будут смотреть другие специалисты
- им проще смотреть на стандартно отформатированный код (как правило)
- иногда у команды есть свои правила форматирования, которые чуть отличатся от стандартных.
- Если все сделано по уму, то у всех членов команды в IntelIJ Idea именно так и настроено форматирование кода.
- Т е - все равно - привычка реформатировать код - нужна
- https://www.jetbrains.com/help/idea/2016.3/reformatting-source-code.html
- */
- ***************************************************************
- @Test
- public void featureTestEdit(){
- filterAll();
- add("test1");
- edit("test1", "test1 edited");
- assertTasksAre("test1 edited");
- }
- /*
- заметь, перейти на какой-то другой фильтр можно только если есть какие-то таски в списке
- так что - сначала - добавляй таски и только затем переходи на нужный фильтр
- и то, если это нужно
- в данном случае - filterAll() - лишнее действие
- feature - не нужно в имени метода
- testEditAtAll - уже ясно, что проверяем лишь действие Edit, на таком-то фильтре
- букв почти столько же, а пользы больше
- по неймингу - почитай faq, там целый раздел
- усли начинаешь имена тест-методов с test - делай так для всех тест-методов
- будь последовательным)
- имя фиче-теста - что тестим и на каком фильтре
- структура фиче-теста
- предварительные действия
- тестируемое действие
- проверки
- предварительные действия начнем с комментария //given - ...
- чтоб было понятно - что это предварительные действия и что за ситуацию мы в результате их получим
- внутри и в конце блока предварительных действий - проверок не делаем
- (мы это тут не тестируем, а используем для создания тестовой ситуации, ниже будет подробнее)
- после предварительных действий - пропустим строку
- чтоб выделить - вот подготовка, вот - тестируемое действие
- проверки
- сначала - более важные
- затем - менее важные
- (собственно - так ты и реализовал)
- такой порядок - чтобы даже если тест упадет на менее важной проверке - был фидбек о важной проверке
- еще - в фиче-тестах мы можем себе позволить более интересные тестовые ситуации
- например - редактирование второй таски в списке
- и не забывай про проверку items left )
- мы это в прошлой работе обсуждали
- продними ревью
- в итоге - с учетом этих рекомендаций - получим
- */
- @Test
- public void testEditAtAll(){
- //given
- add("test1");
- edit("test1", "test1 edited");
- assertTasksAre("test1 edited");
- assertItemsLeft(1);
- }
- *********************************************
- @Test
- public void featureTestsDelete&CanceEdit(){
- filterActive();
- add("test1","test2");
- delete("test1");
- edit("test3","test2 edited");
- cancelEdit("task2" "task2 edit canceled");
- assertTasksAre("task2 edit canceled");
- }
- /*
- не объединяй это в один тест
- так - это не фиче-тест
- а такой маленький е2е
- разбей на 2 теста
- и учти все выше написанное
- */
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement