julia_v_iluhina

Untitled

Oct 22nd, 2016
74
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 2.82 KB | None | 0 0
  1. public class ToDoMVCActiveFilterTest {
  2.     pageobjects.pages.ToDoMVCPage page = new ToDoMVCPage();
  3.  
  4. /*
  5.     используй import для pageobjects.pages.ToDoMVCPage
  6.     чтобы  в коде писать ToDoMVCPage page = new ToDoMVCPage();
  7.  
  8. */
  9. *****************************
  10. public class ToDoMVCAllFilterTest {
  11.     ToDoMVCPage page = new ToDoMVCPage();
  12.     /*
  13.         а тут уже все красиво)
  14.         старайся одной логики придерживаться -
  15.         что импортишь, что нет)
  16.  
  17.         тут - правильно импортить)
  18.     */
  19.     @Test
  20.     public void testEditAtAll() {
  21.         //given
  22.         page.givenAtAll(new GivenHelpers.Task(GivenHelpers.TaskStatus.ACTIVE, "1"));
  23.         /*
  24.             и тут используй import static для GivenHelpers.Task и GivenHelpers.TaskStatus.ACTIVE
  25.             и сможешь написать
  26.              page.givenAtAll(new Task(ACTIVE, "1"));
  27.             правда - код стал понятнее?
  28.         */
  29. *******************************
  30. public class ToDoMVCCompletedFilterTest {
  31. /*
  32.     класс - для тестов на Completed фильтре
  33. */
  34.     ...
  35.  
  36.     @Test
  37.     public void testCancelEditAtActive() {
  38.     /*
  39.         а тут в имени метода - мы уже пишем про Active фильтр...
  40.         )
  41.         и про отмену редактирования ...
  42.     */
  43.         //given
  44.         page.givenAtCompleted(GivenHelpers.TaskStatus.COMPLETED, "1", "2");
  45.         page.filterCompleted();
  46.         /*
  47.             при этом - работаем на Completed фильтре
  48.  
  49.             лишний раз переходим на нужный фильтр
  50.         */
  51.         page.delete("2");
  52.         /*
  53.             и потом - вместо заявленной отмены редактирования - удаляем
  54.         */
  55. /*
  56.  
  57.     ...
  58.     Я не поняла твоей идеи)
  59.     Исправь все несоответствия
  60.  
  61.     реализуй сначала full coverage полностью
  62.     уже в той работе - не делай в фиче-тестах лишнего
  63.     (например, лишних переходов на фильтр, лишних комментариев) - в ревью к full coverage все это есть),
  64.    
  65.     и уже для этих работ - оттолкнешься от приведенного в порядок  full coverage
  66.  
  67.     теперь - когда в имени класса мы указываем  - на каком фильтре мы работаем
  68.     можно будет из имен тест-методов убрать упоминания про фильтры
  69.     и так достаточно точно будет
  70. */
Advertisement
Add Comment
Please, Sign In to add comment