сравни версию

Алексей Алалам
21:05 19 сент.
http://pastebin.com/0J9HZR0m

и


Алексей Алалам
21:03 вчера
http://pastebin.com/3tbbRtXV

В новой версии - ты опять вперед убежал )

в версии http://pastebin.com/0J9HZR0m - было практически все ок
нужно было лишь подправить http://pastebin.com/ME6M8KvG

далее - разберу новые ошибки

но - задачу реши ту, которая поставлена
https://docs.google.com/document/d/1yvUML7eXyEyDh5asUIL7M88RStlE1RZmgUOJZSXjMVo/edit?usp=sharing
от тестового сценария - не уходи

отдай доработанное решение первого задания

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

public class ToDoMvcTest {

    @Test
    public void taskWorkFlow() {

        open("https://todomvc4tasj.herokuapp.com/");

        add("1", "2", "3", "4");
        /*
            каждая операция должна быть проверена
            после add - нужна проверка

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

            так что - нужна проверка
        */
        edit("2", "333");
        /*
            и тут нужна проверка
        */
        delete("1");
        assertTasksAre("333", "3", "4");

        toggle("4");
        assertTasksCompleted("4");
        assertTasksActive("333", "3");
        /*
            про такие проверки была информация в видео
            https://drive.google.com/file/d/0B8hgIBw8-V-AUDhxWDg1YmYxM3c/view
            примерно с 58-ой минуты смотри

            не нужно делать такие проверки

            в данном случае -
                вариант 1 - можно отложить проверку на после clearCompleted
                вариант 2 -
                    после закомпличивания - проверить тексты тасок
                    (проверим - что после закомпличивания таски по-прежнему отображаются в списке - что тоже важно)

                    после clearCompleted - проверить тексты тасок еще раз
                    (проверим - что ранее закомпличеная таска удалилась - так допроверим закомпличивание и проверим clearCompleted)
        */
        clearCompleted();
        assertTasksAre("333", "3");

        toggleAll();
        assertTasksCompleted("333", "3");
        /*
            про такого рода проверку  - писала выше
        */
        toggleAll();
        assertTasksAre("333", "3");
        /*
            reopen all - лучше проверять на другом фильтре

            мало того, в рамках smoke coverage -  его лучше вообще не покрывать
            т к это - не высокоприоритетная операция
        */

        toggleAll();
        clearCompleted();
        tasks.shouldBe(empty);
        /*
            писала в прошлом ревью - для tasks.shouldBe(empty);
            реализуй метод

            ты реализовал )
            assertTasksEmpty()

            а почему не используешь?
        */

 ....

    private void edit(String taskText, String taskNew) {
/*
    круто, что смог реализовать )

    для сценария из первого задания - этот метод тебе еще не нужен
    но - пригодится в следующем задании

    нужно будет подправить вторую строку
    ты оперируешь таской в режиме редактирования

    доступайся к ней так
        список тасок
        получить таску в режиме редактирования
        получить внутренний элемент для ввода нового текста таски

    т е выражение будет -
        tasks.findBy(...).$(...)
        или
        tasks.findBy(...).find(...)
*/

**************************
  private void assertTasksEmpty() {
        tasks.shouldHaveSize(0);
  }
  /*
    да, можно и так проверить

    а можно и так
    tasks.shouldBe(empty);

    технически - это одно и то же

    вариант tasks.shouldBe(empty);
    более нагляден

    но это все несколько субъективно
    не настаиваю на этих изменениях

  */
***************************************
  private void assertTasksActive(String... taskTexts) {
     active.shouldHave(exactTexts(taskTexts));
  }

  private void assertTasksCompleted(String... taskTexts) {
     completed.shouldHave(exactTexts(taskTexts));
  }
/*
    писала выше  - что посмотреть - почему такие проверки не нужны

    тут еще момент
    я не вижу в предоставленном коде - где объявлены переменные active и completed

    ты уверен, что код рабочий?
*/
****************************

    private void filterAll() {
        filters.get(0).click();
    }
/*
    писала тебе про это в http://pastebin.com/ME6M8KvG
    строки 9-17

    касается и filterActive() , и filterCompleted()
*/