Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Test
- public void testActivateAll() {
- /*
- лучше уточнить testActivateAllAtCompeted
- смести этот метод ниже остальных фиче-тестов
- и пометь метод комментарием extra coverage
- т к в смоук покрытии - покрывать не высокоприоритетное не надо
- убирать - не стоит
- но пометить - что это экстра - надо
- */
- //given
- /*
- given - completed tasks
- будет чуть понятнее
- */
- page.create("a", "b", "c", "d");
- page.toggleAll();
- page.filterCompleted();
- page.toggleAll();
- /*
- тут - проверь состояние списка и items left
- этого будет достаточно
- т к уже проверяли - что активные таски не отображаются на Completed фильтре
- но они есть (переходили на all и єто проверяли)
- посмотри на свой же е2е
- items left покрываем - потому что это не загромождает код
- и потому что нам по пути
- потом - ради проверки только єтого - городить отдельные фиче-тесты
- с практически таким же кодом - смысла не будет
- */
- page.filterActive();
- page.assertVisibleTasks("a", "b", "c", "d");
- }
- @Test
- public void testEditByTab() {
- /*
- мы убрали из е2е - edit
- вот его и реализуем
- EditByTab - это тоже extra coverage
- т к приоритет у этого действия - ниже
- почему ты не придерживался своего же плана?
- зачем тогда нужен план)
- хочешь - оставляй testEditByTab как extra coverage
- правильно размести метод - после комментария про extra coverage
- но тогда стоит на другом фильтре это покрывать (по сравнению с testEdit)
- да и другую тестовую ситуацию использовать - например редактировать вторую таску в списке
- а хочешь - просто переделай этот метод на testEdit
- советую именно так поступить)
- в имени метода - уточни фильтр
- */
- //given
- page.create("a");
- page.startEdit("a", "a edited").pressTab();
- page.assertVisibleTasks("a edited");
- /*
- добавь проверку items left
- покроем по пути
- */
- }
- @Test
- public void testCancelEditByEsc() {
- /*
- планировалось покрытие - не на
- уточни имя
- и покрой на Active фильтре
- цель - распределить покрытие по разным фильтрам
- и можно разнообразить тестовую ситуацию (по сравнению с edit)
- например можно работать со второй таской из списка
- */
- //given
- page.create("b");
- page.startEdit("b", "b cancel edit").pressEscape();
- page.assertVisibleTasks("b");
- /*
- добавь проверку items left
- покроем по пути
- */
- }
Advertisement
Add Comment
Please, Sign In to add comment