SelenideElement stringForNewTask = $("#new-todo");
/*
    молодец, с типом переменной сообразил)

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

    посмотри на раздел по неймингу в faq
    там есть что почитать и про имена переменных
*/
***************************
    private void ensureUrlOpened() {
        if (url().equals("https://todomvc4tasj.herokuapp.com")) {
        } else {
            open("https://todomvc4tasj.herokuapp.com/");
        }
    }
/*
    можно проще
*/
  private void ensureUrlOpened() {
        if (!url().equals("https://todomvc4tasj.herokuapp.com")) {
            open("https://todomvc4tasj.herokuapp.com/");
        }
    }
/*
    http://www.java-made-easy.com/logical-operators.html

    http://www.cs.columbia.edu/~lennox/3101-03/class1-slides/logical.html
    https://docs.oracle.com/javase/tutorial/java/nutsandbolts/opsummary.html

*/
*******************************
    @Test
    public void taskWorkFlow() {
        given();
        add("a");
        /*
            как и в остальных тест-методах - отступи строку после вызова гивен-метода
            так будет четче отделена - логика предварительных действий от логики тестовых действий
        */
**************************
    @Test
    public void reopenAtAllFilter() {
        given(COMPLETED, "a");

        //reopen
        toggle("a");

        assertItemsLeft(1);
    }
/*
    смело прибивай комментарий //reopen

    и так ясно все
    четко ясно - где тестируемое действие
    и название тест-метода - тоже ок

    а вот с проверками - не все ок)
    одной не хватает
    проверить assertItemsLeft(1); - недостаточно
*/
***********************
    public void submitEditByClickOutside() {
    public void submitEditByPressTab() {
    public void deleteByEmptyingText() {
/*
    сверься с тест-планом - что на каком фильтре покрыть
    и в названии тест-методов - тоже отрази это
*/
**************************
    @Test
    public void switchToAllFromActive() {
        givenAtActive(COMPLETED, "a");

        filterAll();

        assertTasksAre("a");
        assertItemsLeft(0);
    }
/*
    когда мы хорошо и точно проверим фильтеринг

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

    эти условия выполняются - это ок

    как еще можно уточнить
    так мы проверили - как при переходе на all отображаются закомпличеные таски
    а как себя ведут активные - не проверили

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

*/