package ua.stobf.study;


import com.codeborne.selenide.Condition;
import org.junit.Test;
import org.openqa.selenium.By;

import static com.codeborne.selenide.CollectionCondition.exactTexts;
import static com.codeborne.selenide.CollectionCondition.size;

import static com.codeborne.selenide.Condition.name;
import static com.codeborne.selenide.Condition.text;
import static com.codeborne.selenide.Selectors.byName;
import static com.codeborne.selenide.Selectors.byText;
import static com.codeborne.selenide.Selenide.*;

/**
 * Created by Stobf on 30.01.2017.
 */
public class DemoTest { // почитай в FAQ о именовании... в данном случае - обрати внимание на советы по именам к тест классам
//здесь обычно ставят пустую строчку "для феншуя", но это не обязательно конечно...
    @Test
    public void testCreateTask(){ // а здесь обрати внимание в ФАК на советы по именам к тест методам
        open("https://todomvc4tasj.herokuapp.com/");
        $("#new-todo").setValue("task1").pressEnter();
        $("#new-todo").setValue("task2").pressEnter();
        $("#new-todo").setValue("task3").pressEnter();
        $("#new-todo").setValue("task4").pressEnter();
        $("#todo-list li:nth-of-type(2) .destroy").click();
        $("#todo-list li:nth-of-type(4) .toggle").click();
        $("#clear-completed").click();
        $("#toggle-all").click();
        $("#clear-completed").click();

// зачем сдесь эти пустые строчки? :)
    }
}

// в целом твой код сильно "сбит в кучу" - его сложно читать.
// Если ты добавишь пустые строки что бы выделить в нем основные блоки - появится структура
// и код станет проще читать и понимать

// также в коде отсутствую какие либо проверки. Разве тест - это только шаги?
// это ведь и проверка актуальных результатов на предмет соответствия ожидаемым.
// в задании было указано именно написать ТЕСТ на основе "сценария из шагов" ;)
// то есть шаги то у тебя были, да, но на их основе нужно было написать полноценный тест:)
// а тест без проверок - это не тест ;)