http://joxi.ru/J2b1MVvsXxglYm

/*
    вот этот комментарий только не учтен
*/
********************************
    public void testFilteringOnActive() {
 // testSwitchFromCompletedToActive ?

    public void testFilteringOnCompleted() {
 // testSwitchFromAllToCompleted ?

    public void testFilteringOnAll() {
 //testSwitchFromActiveToAll ?
/*
    в именах этих тестов лучше уточнить - с какого на какой фильтр переходим

    нам недостаточно в имени уточнить - на каком фильтре мы выполняем действие
    нам нужно еще и описать само действие = на какой фильтр переходим

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

***************************
    public void testSaveChangesByClick() {
    public void testSaveChangesByTab() {
/*
    в этих именах - нужно уточнить фильтр
*/
************************************

    public void testSaveChangesByClick() {
        given(ACTIVE, "1");

        startEdit("1", "1 edit click");
        //outside click
        filterAll();
        assertTasksAre("1 edit click");
        assertItemsLeft(1);
    }
/*
    действие outside click - реализованное как filterAll() - не лучший выбор

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

    а вот плохо то, что при кликах на линках для фильтеринга - происходит не только потеря фокуса у редактируемого поля
    а и некая логика связанная с работой самих линок для фильтеринга
    согласна, ты мне можешь возразить - что раз мы будучи на  All фильтре - кликаем по линке All   - то ничего и не будет
    я бы сказала - не должно быть) ЕСЛИ -  filterAll() - будет работать правильно)

    и чтобы потом не думать - из-за чего в этом тесте возникла ошибка
    из-за неверной работы filterAll();
    или неверной работы SaveChangesByOutsideClick (Outside - в имени тоже стоит отразить)

    лучше - кликать на чем-то уже используемом, но таком - при клике а котором произойдет только смена фокуса
    я пока лучшего варианта чем  $("#new-todo") - не вижу

    если будешь использовать  $("#new-todo")
    то -  $("#new-todo") будет использоваться дважды - в разных методах
    значит - нужна переменная
    (cм в faq про это - в разделе о DRY)
*/