Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Test
- public void testTasksFlow() {
- /*
- выше - пропущены 2 строки
- одной достаточно
- */
- add("test1");
- toggle("test1");
- assertTasksAre("test1");
- filterActive();
- assertTasksEmpty();
- add("test2");
- /*
- следующая операция - toggleAll() - не проверяет предыдущую add("test2")
- нужна проверка
- */
- toggleAll();
- assertTasksEmpty();
- filterCompleted();
- assertTasksAre("test1", "test2");
- toggle("test1");
- clearCompleted();
- assertTasksEmpty();
- filterAll();
- assertTasksAre("test1");
- delete("test1");
- /*
- delete - запланировано покрыть в фиче-тесте
- */
- assertTasksEmpty();
- }
- /*
- возвращаемся к планам
- http://joxi.ru/Y2LXgYnf9E84d2
- планировали - 1 2 и 3 - покрыть в фиче-тестах
- Edit - запланировано покрыть для All фильтре
- а CancelEdit и Delete - на Active фильтре
- а в этой версии
- CancelEdit и Delete - покрыли на All фильтре, и не в фиче-тестах, а в еще одном е2е тесте (т к в рамках одного теста - 2 фичи тестим)
- Delete - остался покрытым и в testTasksFlow в том числе
- Так что - получаем, что наша реализация отличается от планов, и значительно
- Можно конечно вернуться к планам и на это еще раз потратить время
- Но запланированное - достаточно хорошо сделано)
- Я бы все же реализацию подправляла )
- */
- **************************************************
- @Test
- public void testEditAtAll() {
- /*
- и реализация, и имя метода = ок
- */
- ********************************************
- @Test
- public void testDeleteAndCancelEdit() {
- /*
- а вот тут - и реализция, и имя - есть вопросы
- что мы собирались делать
- мы собирались реализовать фиче-тесты CancelEdit и Delete - на Active фильтре
- т е - еще 2 тест-метода
- testDeleteAtActive
- testCancelEditAtActive
- и в реализации учти
- что в гивен-действиях - не только таску нужно добавить
- но и перейти на нужный фильтр
- */
- //given
- add("test1");
- delete("test1");
- add("test2");
- cancelEdit("test2", "test2 edited");
- assertTasksAre("test2");
- assertItemsLeft(1);
- }
- /*
- сейчас - реализован маленький е2е
- причем - в нем не хватает проверок
- вернись к работе по smoke тесту - мы обсуждали проверки
- реализуй именно фиче-тесты
- если нужно - давай обсудим
- */
Advertisement
Add Comment
Please, Sign In to add comment