Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public class ToDoMVCActiveFilterTest {
- pageobjects.pages.ToDoMVCPage page = new ToDoMVCPage();
- /*
- используй import для pageobjects.pages.ToDoMVCPage
- чтобы в коде писать ToDoMVCPage page = new ToDoMVCPage();
- */
- *****************************
- public class ToDoMVCAllFilterTest {
- ToDoMVCPage page = new ToDoMVCPage();
- /*
- а тут уже все красиво)
- старайся одной логики придерживаться -
- что импортишь, что нет)
- тут - правильно импортить)
- */
- @Test
- public void testEditAtAll() {
- //given
- page.givenAtAll(new GivenHelpers.Task(GivenHelpers.TaskStatus.ACTIVE, "1"));
- /*
- и тут используй import static для GivenHelpers.Task и GivenHelpers.TaskStatus.ACTIVE
- и сможешь написать
- page.givenAtAll(new Task(ACTIVE, "1"));
- правда - код стал понятнее?
- */
- *******************************
- public class ToDoMVCCompletedFilterTest {
- /*
- класс - для тестов на Completed фильтре
- */
- ...
- @Test
- public void testCancelEditAtActive() {
- /*
- а тут в имени метода - мы уже пишем про Active фильтр...
- )
- и про отмену редактирования ...
- */
- //given
- page.givenAtCompleted(GivenHelpers.TaskStatus.COMPLETED, "1", "2");
- page.filterCompleted();
- /*
- при этом - работаем на Completed фильтре
- лишний раз переходим на нужный фильтр
- */
- page.delete("2");
- /*
- и потом - вместо заявленной отмены редактирования - удаляем
- */
- /*
- ...
- Я не поняла твоей идеи)
- Исправь все несоответствия
- реализуй сначала full coverage полностью
- уже в той работе - не делай в фиче-тестах лишнего
- (например, лишних переходов на фильтр, лишних комментариев) - в ревью к full coverage все это есть),
- и уже для этих работ - оттолкнешься от приведенного в порядок full coverage
- теперь - когда в имени класса мы указываем - на каком фильтре мы работаем
- можно будет из имен тест-методов убрать упоминания про фильтры
- и так достаточно точно будет
- */
Advertisement
Add Comment
Please, Sign In to add comment