http://joxi.ru/n2YkKaGUoejgXr
/*
    применяй пропуски строк с какой-то одной логикой
    тут нет точного одного правила
    руководствуйся здравым смыслом
    пропуск строки используй для разделения кода, чтоб подчеркнуть - что это какие-то разные части
    вот это еще почитай
    https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace

    1 - я бы пропустила тут строку, методы друг от друга отделить пропуском строки - помогает код сделать нагляднее

    2 - а тут смысла нет строки пропускать - метод в одну строку

    3 - 2 строки пропускать нет смысла

    4 - а в этом методе, в отличие от случая 2 - другая логика пропусков

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

    как я бы пропуски делала в таком коде
*/

public class TodoMVCTest {

    @After
    public void clearData() {
        executeJavaScript("localStorage.clear()");
    }


    @Before
    public void openPage() {
        open("https://todomvc4tasj.herokuapp.com/");
    }

    @Test
    public void testTasksFlow() {
        open("https://todomvc4tasj.herokuapp.com/");

        add("test1");
        edit("test1", "test1 edited");
        assertItemsLeft(1);
        ....
/*
    сравни
    это конечно мелочь)
    но на восприятие кода это тоже влияет
*/
************************************
    @Test
    public void TestsDelete&

    CanceEdit() {

        //given
        add("test1", "test2");

        delete("test1");
        edit("test3", "test2 edited");
        cancelEdit("test2""test2 edit canceled");
        assertTasksAre("test2 edit canceled");


    }
/*
    ты проверял - этот код работает?
    вообще-то не должен работать)

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

    я в прошлый раз писала - строки 84-104

    вернись к прошлой реализации
    и учти мои комментарии
*/
*********************************************
 @Test
    public void testTasksFlow() {
        open("https://todomvc4tasj.herokuapp.com/");

        ...
        edit("test1", "test1 edited");//<---1
        ...

        filterActive();
        ....
        cancelEdit("test2", "test2 edited");//<---2
        ...

        filterAll();
        ....
        delete("test1 edited");//<---3
        ...
    }
/*
    1, 2 и 3 - это мы планируем покрыть фиче-тестами
    и потому - можем упростить testTasksFlow()
    ведь мы по-прежнему реализуем smoke coverage
    а в рамках смоука - достаточно покрыть единожды все высокоприоритетное
    нам важно остаться эффективными и повторения в данном случае - будут эффективность снижать
    посмотри на ревью прошлой работы, там мы это уже обсуждали
*/
***********************
    public void testEditAtAll() {
/*
    метод ок реализован
    аналогично проработай и 2 остальных запланированных фиче-теста
*/