public void testCommonUserFlow() {
        givenAtAll();

        add("a", "b", "j");
        assertTasks("a", "b", "j");
        edit("j", "json");
        assertTasks("a", "b", "json");
        //complete
        toggle("b");
        assertTasks("a", "b", "json");
        delete("json");
        assertTasks("a", "b");
        //complete all
        toggleAll();
        clearCompleted();
        assertNoTasks();
    }
/*
    Все равно - этот тест-метод хуже, чем тот, что был в предыдущем задании

    причина - плохо оттестирована операция complete
    лучше просто в этом тесте - не покрывай complete - покроешь в фиче-тесте

    да и для smoke покрытия - так неэкономно создавать таски - однозначно перебор
    когда ты вызываешь add("a", "b", "j") - считай ты трижды покрыл это действие
    подумай, сколько тасок тебе надо, и когда их стоит добавлять
    (добавив сразу все таски - ты сразу лишаешь себя возможности использовать неявные проверки,
    просмотри ревью по заданию - Smoke E2E)
*/
*******************************************

    @Test
    public void testEditDeleteCompleteAtActive(){
/*

    читаем задание
      ...
      provide full coverage according your test plan by
      switching more to feature style tests.
      Move some "coverage" from E2E to FT in order to make E2E smaller,
      more readable and easier to support.
      Leave E2E only for "general" filtering and some basic operations in tasks Life Cycle.
      ....

    в этой работе - у нас была задача - реализовать фиче-тесты
    которые дополнят покрытие до полного
    а это - е2е

    тебе нужны отдельные тест-методы - для каждой фичи

    посмотри работу Smoke E2eE = F https://www.refheap.com/119408
    и ревью к ней

    тут фиче-тесты будут использовать для реализации предварительных действий - гивен-методы
    а все остальное - похоже
*/
**********************************

    public void testEditDeleteAtCompleted(){
/*
    и этот тест - е2е
    почему?
*/
*******************************

    public void testCancelEditTaskAtAll()
    public void testCancelEditTaskAtActive()
    public void testCancelEditAtCompleted()


    public void testReopenAtCompleted() {
    public void testReopenAllAtCompleted() {
/*
    вот эти тест-методы - реализованы как фиче-тесты
    собственно - так и нужно было все остальное не покрытое в е2е реализовать
*/
*************************
    @Test
    public void testSwitchBetweenFilters(){
        given(
                aTask("a", ACTIVE),
                aTask("b", COMPLETED)
        );
        filterActive();
        assertVisibleTasks("a");
        filterCompleted();
        assertVisibleTasks("b");
        filterAll();
        assertTasks("a", "b");
        assertItemsLeft(1);
    }
/*
    кстати, можно было именно такой е2е реализовать
    а остальное - покрыть фиче-тестами

    есть смысл покрыть переходы по фильтрам в рамках е2е
    чтобы убедиться - что при цепочке действий с перехлдами по фильтрам все корректно работает

    а остальные 3 перехода (см тест-план) - надо допокрыть в фиче-тестах
*/

*************************************


http://joxi.ru/823k1x0U67k602

/*
    соответствие обозначенного покрытия реализованному - не проверяла
    т к еще нужно переделывать тесты

    просто анализируем обозначенное покрытие
    где цифры стоят - не хватает покрытия

    add on active - средний приоритет, надо покрывать.
    в рамках теста - покрой и проверку items left

    и далее везде - второй приоритет покрывай

    а в действиях - когда все з варианта низкий приоритет - покрой хотя бы на одном из фильтров

    прямоугольниками обозначила - что можно НЕ покрыть в рамках optimized full coverage
    исходя из приоритетов, которые ты проставил
*/

******************************************

  public static Task[] getListOfTasks(TaskType taskType, String... tasks){
        ...
            listOfTasks[i] = (tasks.length == 1) ? aTask(tasks[i], taskType) : aTask(tasks[i], taskType);
        /*
            не могу понять - зачем тут тернарный оператор

            ListOfTasks - все же говорит о том, что работаем со списком, а не массивом

            тут лучше было бы применить термин
                для имени метода - aTasks (по аналогии с aTask)
                для переменной - tasks, taskArray

            List - вводит в заблуждение
        */