https://bitbucket.org/testmunk/todomvctest/src/e0e21bacd54b?at=master   http://pastebin.com/9Gu9LbWN
 given(new Task(TaskType.ACTIVE, "a"), new Task(TaskType.COMPLETED, "b"));
 //уже можно проще
  given(new Task(ACTIVE, "a"), new Task(COMPLETED, "b"));
/*
    ведь при других вызовах ты так и делаешь - используешь ACTIVE и COMPLETED
    благодаря import static
*/
*************************
 @Test
    public void testClearCompletedAtActive() {

        given(new Task(TaskType.COMPLETED, "a"), new Task(TaskType.ACTIVE, "b"));

/*
    в имени метода  - указали AtActive
    но в самом методе - работаем на All

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

    вот такое же еще тут заметила - testReopenAtCompleted()
    остальное - сам проверь
*/
*******************************
    @Test
    public void testClearCompletedAtCompleted() {

        givenAtCompleted(ACTIVE, "a");

        clearCompletedButton.shouldNotBe(visible);
        assertNoTasks();
        assertItemsLeft(1);
    }
/*
    неожиданно)

    в имени метода - говорим - проверим действие clearCompleted
    а сами что проверяли?

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

    тут - реализуй именно testClearCompletedAtCompleted()

*/
*********************************
    @Test
    public void testReopenAllAtCompleted() {

        givenAtCompleted(ACTIVE, "a", "b");

        toggleAll();
        assertTasks("a", "b");
        assertItemsLeft(0);
    }
/*
    название мтеода - проверяем ReopenAll

    вопрос - применяя toggleAll к активным таскам - мы делаем ReopenAll?

    проверь все методы - чтоб они делали то, что заявляют
*/
**********************************
    public void testSwitchFromAllToCompleted() {
    public void testFromAllToActive() {
    public void testFromActiveToAll() {
    public void testFromActiveToCompleted() {
    public void testFromCompletedToAll() {
    public void testFromCompletedToActive() {
/*
    тянется еще с http://pastebin.com/xXFaxr1c
    строки 75-92

    названия тест-методов - сделай единообразными
    они все - тестят SwitchFrom...To....
    вот пусть все так и называются

    по нюансам реализации - писала - как точнее проверять переходы по фильтрам
    это надо учесть
*/
***********************************
    public void testCancelEditByEscapeAtActive() {

        givenAtActive(ACTIVE, "a");

        cancelEdit("a", "b");
/*
    в имени тест-метода - не надо уточняться до ByEscape
    мы ведь этого е делали даже для вспомогательного метода - cancelEdit("a", "b");

    причина - только так и можно сделать cancelEdit
    нет других вариантов

    в отличие от ConfirmEdit
    там вполне уместно уточниться - ConfirmEditByClickOutside
    т к есть несколько способов выполнить ConfirmEdit
*/
*********************************
SelenideElement placeholder = $("#new-todo");
/*
    почему такое странное имя у переменной?

    почему не newTodo ?
*/
*************************
SelenideElement clearCompletedButton = $("#clear-completed");
/*
    не нужна такая переменная тебе)
    писала выше почему
*/
*****************************
public class TodoMVCTest extends BaseTest {
/*
    наследование подправил
    это ок
    но зачем оставил в проекте класс - AtTodoMVCPageWithClearedDataAfterEachTest
    ( src / test / java / ua / net / itlabs / hw3 /)
    ?

    он не нужен
    более того - там застряли старые ошибки
    удаляй AtTodoMVCPageWithClearedDataAfterEachTest
*/
*********************************
public class BaseTest {
   @Before
    public void openPage() {
        ensureCorrectURL();
    }

....
    @Step
    public void ensureCorrectURL() {
        String todoMvcUrl = "https://todomvc4tasj.herokuapp.com/";
        if (!todoMvcUrl.equals(url()))
            open(todoMvcUrl);
    }
/*
    мы с тобой уже обсуждали это

    встрой вызов метода  ensureCorrectURL() - в гивен-метод
    вот в этот - public void given(Task... tasks)
    этот метод - в остальных гивен-методах переиспользуется
    и значит - такое действие - ensureCorrectURL()
    будет выполняться в начале каждого из гивен-методов

    перенеси метод ensureCorrectURL() из BaseTest
    в тот класс где гивены реализованы

    таким образом - у BaseTest
    останется одна задача - делать скриншоты и их аттачить к репорту

    и это хорошо и правильно
    не надо на один класс - навешивать разные цели и ответственности
    погугли Single Responsibility Principle
*/