julia_v_iluhina

Untitled

Jan 5th, 2017
83
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.46 KB | None | 0 0
  1.     public enum TaskType{
  2.         ACTIVE("false"),
  3.         COMPLETED("true");
  4.  
  5.         private String task;
  6.         /*
  7.             имя для такого поля - не очень
  8.             isCompleted
  9.             description
  10.         */
  11.         private TaskType(String value) {
  12.         /*
  13.             имя параметра конструктора - назови также
  14.             чтоб уже было единообразно
  15.         */
  16.             this.task = value;
  17.         }
  18.  
  19.         @Override
  20.         public String toString() {
  21.             return task;
  22.         }
  23.         /*
  24.             вспомни алюр-репорт
  25.             репортилось true = http://joxi.ru/vAW36Kgs11P73A
  26.  
  27.             и надо было подумать - что ж это за true
  28.  
  29.             а если на уровне toString для TaskType - собирать
  30.             "\"completed\":true"
  31.             "\"completed\":false"
  32.  
  33.             то и  репортиться это будет
  34.             и код в гивене чуть упростится
  35.  
  36.             это не обязательно учитывать
  37.             но - это для наглядности репорта будет полезно
  38.         */
  39.     }
  40. ******************************************************
  41.     @Test
  42.     public void testTasksLifeCycle() {
  43.  
  44.         //tasks.add("t1");
  45.         given.givenAtAll(PreconditionHelpers.TaskType.ACTIVE, "t1");
  46. /*
  47.     заменив tasks.add("t1"); операцией given.givenAtAll
  48.     ты не покрываешь в е2е - добавление таски на all
  49.     можно в общем-то и так
  50.     только тогда  не забудь эту операцию покрыть фиче-тестами
  51.  
  52.     в PreconditionHelpers метод givenAtAll - объявлен как статический
  53.     раз так - ты можешь его заимпортить (import static)
  54.     и вызывать givenAtAll (а не given.givenAtAll)
  55.     в таком случае - вообще можно не создавать объект given
  56.  
  57.     также можно заимпортировать (import static) - PreconditionHelpers.TaskType.ACTIVE
  58.     и использовать в коде ACTIVE
  59.     аналогично - и с COMPLETED поступи
  60. */
  61. **************************************************
  62.     public static void givenAtAll(TaskType taskType, String... taskTexts)
  63. /*
  64.     с этим методом - все ОК
  65.  
  66.     дальше делаем гивены
  67.     прошлое ревью строки 95-118
  68.     для этого - тут в PreconditionHelpers
  69.  
  70.     реализуй класс Task
  71.     суть класса - быть описателем для одной таски
  72.     у нее есть свойства - ее текст и ее TaskType taskType
  73.  
  74.     советую для класса Task реализовать метод toString
  75.     который вернет сразу нужный кусок команды {......}
  76.  
  77.     так делать не обязательно (toString)
  78.     но это будет полехно и для репортинга, и для наглядности кода
  79.  
  80.     в итоге - будет реализован given(Task... tasks)
  81.  
  82.     дальше - прошлое ревью, начиная со строки 216
  83.  
  84. */
  85. ***************************************************
Advertisement
Add Comment
Please, Sign In to add comment