Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public enum TaskType{
- ACTIVE("false"),
- COMPLETED("true");
- private String task;
- /*
- имя для такого поля - не очень
- isCompleted
- description
- */
- private TaskType(String value) {
- /*
- имя параметра конструктора - назови также
- чтоб уже было единообразно
- */
- this.task = value;
- }
- @Override
- public String toString() {
- return task;
- }
- /*
- вспомни алюр-репорт
- репортилось true = http://joxi.ru/vAW36Kgs11P73A
- и надо было подумать - что ж это за true
- а если на уровне toString для TaskType - собирать
- "\"completed\":true"
- "\"completed\":false"
- то и репортиться это будет
- и код в гивене чуть упростится
- это не обязательно учитывать
- но - это для наглядности репорта будет полезно
- */
- }
- ******************************************************
- @Test
- public void testTasksLifeCycle() {
- //tasks.add("t1");
- given.givenAtAll(PreconditionHelpers.TaskType.ACTIVE, "t1");
- /*
- заменив tasks.add("t1"); операцией given.givenAtAll
- ты не покрываешь в е2е - добавление таски на all
- можно в общем-то и так
- только тогда не забудь эту операцию покрыть фиче-тестами
- в PreconditionHelpers метод givenAtAll - объявлен как статический
- раз так - ты можешь его заимпортить (import static)
- и вызывать givenAtAll (а не given.givenAtAll)
- в таком случае - вообще можно не создавать объект given
- также можно заимпортировать (import static) - PreconditionHelpers.TaskType.ACTIVE
- и использовать в коде ACTIVE
- аналогично - и с COMPLETED поступи
- */
- **************************************************
- public static void givenAtAll(TaskType taskType, String... taskTexts)
- /*
- с этим методом - все ОК
- дальше делаем гивены
- прошлое ревью строки 95-118
- для этого - тут в PreconditionHelpers
- реализуй класс Task
- суть класса - быть описателем для одной таски
- у нее есть свойства - ее текст и ее TaskType taskType
- советую для класса Task реализовать метод toString
- который вернет сразу нужный кусок команды {......}
- так делать не обязательно (toString)
- но это будет полехно и для репортинга, и для наглядности кода
- в итоге - будет реализован given(Task... tasks)
- дальше - прошлое ревью, начиная со строки 216
- */
- ***************************************************
Advertisement
Add Comment
Please, Sign In to add comment