private void assertTasksAre(String... taskTexts) {
  private void assertTasksListIsEmpty() {
...
    private void assertActiveListIsEmpty() {
        tasks.filterBy(visible).shouldBe(empty);
    }

    private void assertCompletedListIsEmpty() {
        tasks.filterBy(visible).shouldBe(empty);
    }
 /*
    посмотри на реализацию assertActiveListIsEmpty() и assertCompletedListIsEmpty()
    я не вижу отличий)
    так нужны ли нам 2 метода?

    почитай в faq - про DRY principle
    https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.lzxge2s5ilgv
    (там несколько вопросов-ответов на эту тему, пролистывай)

    что касается названий методов assertActiveListIsEmpty() и assertCompletedListIsEmpty()
    они не отражают того, что методы выполняют - лучше быть максимально точным в нейминге

    assertTasksListIsEmpty() - так мы проверяли не отфильтрванный по visible список
    assertVisibleTasksListIsEmpty() - логично было бы уточнить имя и остаться в тех же терминах, что ранее использовал

    измени название и оставь лишь один метод из вот этих двух assertActiveListIsEmpty() и assertCompletedListIsEmpty()

    еще такой момент
    ты используешь на комлитед фильтре - assertTasksAre
    если сценарий где-то выше изменится так, что в это месте (строка 41 твоего кода)
    будет 2 видимых таски и одна не видимая - проверка assertTasksAre не пройдет
    и можно не сразу додуматься - из-за чего
    потому - разумно и тексты тасок - на Active & Completed фильтрах проверять не у всего списка тасок,
    а у отфильтрованного по visible

    итого получим - у нас будет 4 проверки
    2 - для проверки не отфильтрованного списка тасок
    2 - для проверки  отфильтрованного списка тасок

    посмотри вот это видео и определись окончательно - какие проверки будешь использовать
    https://drive.google.com/file/d/0B8hgIBw8-V-AdGxxU1R3enl1RzQ/view?ts=567ab8d7
*/
*************************************************
 private void filterAll() {
 private void filterActive() {
 private void showCompleted() {
/*
    Посмотри на имена методов
    ничего не хочешь изменить ? )
    чтобы переименовать класс/метод/переменную/параметр
    https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.vwuqi54t6fyg

*/
**************************************

 private SelenideElement editTask(String taskText, String newText) {
/*
    не вижу изменений - согласно прошлого ревью, строки 233-276
*/
****************************************
    private void assertItemsLeftCounterEquals(String counter) {
        $("#todo-count").shouldBe(text(counter + " item"));
    }
/*
    Для имени метода - достаточно assertItemsLeftCounter, а еще лучше - assertItemsLeft

    параметр метода = количество = число, а не строка
    измени тип параметра, и поправь его имя
    проверяем не счетчик, а ожидаемое количество активных тасок

    нам достаточно проанализировать только выводимое количество
    а кусок фразы анализировать - точно не нужно
    понимаю, почему так реализовал)

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

        //Activate
        toggleTask("2");
        clearCompleted();
/*
    осталась ошибка
    нужна проверка после toggleTask("2");
*/