ElementsCollection tasks = $$("#todo-list li");
/*
    это уже просто по пути

    "#todo-list li" - можно написать точнее
    вот так - "#todo-list>li"

    в нашем несложном приложении и твой вариант ок
    но вот в более сложных случаях - такая разница может быть критичной
*/
*******************************************
    private void assertEmpty()

/*
    В именах методов-проверок - лучше быть максимально точным

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

    вот например в assertTasksAre - ты ууказал - что работаем со списком Tasks

    и тут тоже нужно бы - а то не ясно - что на пусто проверяется -
    получим assertTasksEmpty
*/
******************************************
    private void assertTasksAre(String... taskText) {
/*
    здесь мы оперируем одним текстом таски, или несколькими тасками ?
    судя по типу параметра - несколькими

    а почему имя параметра - taskText, а не taskTexts?
*/
*****************************************************
    private void toggle(String taskTexts)
/*
    тут - ситуация с точностью до наоборот

    судя по типу параметра и его использованию - это ОДИН текст  - текст таски
    а имя параметра = текстЫ

    разберись)

    точность в нейминге - очень важна
*/