/*
    К рализации сценария - вопросов нет)
    Оптимальный сценарий, оптимальное покрытие, оптимальное распределение действий по фильтрам

    есть вопросы к оформлению
    http://joxi.ru/KAg8voQsga1y5r
    комментарии
        поясняешь справа
        не поясняешь (в аналогичной ситуации)
        поясняешь сверху

        выбери обин стиль - как ты будешь комментировать
        и его придерживайся

    пропуски строк
        пропуск одной строки - разумно использовать как разделитель
        несколько строк пропускать - уже перебор

        посмотри conventions про это
        https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
        и вообще полюбопытствуй - в документе много интересного

        по пропускам строк - весь код проанализируй. Тут есть что поправить
*/
***************

public class ToDoMVCSmoke {
//имя класса - не по conventions

    @Test
    //что тут пропуск строки тебе дает?
    public void testTasksCycle() {
    //не стоит сокращать LifeCycle до Cycle - уходит точность и однозначность

/*
    про использование Smoke как части имени тест-класса или метода

    не надо)

    да, это важно
    круто, что ты эту информацию выделила как важную

    но - ее обычно по-другому показывают
    дальше на курсе будет

    а сейчас - Smoke - убери из имени
*/

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

private SelenideElement startEdit(String oldText, String newText) {
/*
    В качестве параметров других методов ты используешь термин taskText - для обозначения текста таски

    тут тоже ничего нового не будем придумывать

    oldTaskText & newTaskText

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

    так мы код сделаем понятнее, однозначнее
*/

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

    private void assertTasks(String... taskTexts) {
        tasks.shouldHave(exactTexts(taskTexts));
    }

    private void assertNoTasks() {
        tasks.filter(visible).shouldBe(empty);
    }

/*
    ага
    ты таки заметила
    что active & completed фильтрах надо проверять список тасок в отфильрованном состоянии

    только тут надо чуть доработать

    посмотри видео про этот момент
    и прими окончательное решение

    https://drive.google.com/file/d/0B8hgIBw8-V-AdGxxU1R3enl1RzQ/view?usp=sharing
*/

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

 private void assertItemsLeft(int itemsLeft) {

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

    вызови раз в рамках сценария и эту проверку

    одного раза для е2е сценария будет достаточно ( т к это штука не высокоприоритетная)
*/