public class TodoMVCTest {
/*
    отступи тут строку - легче будет восприниматься код
*/
   @Test
   public void testTasksCommonFlow()

   {
/*
    используй стандартное форматирование
    это значит - стандартное расположение отступов, например

    почитай - https://www.jetbrains.com/help/idea/2016.3/reformatting-source-code.html

    в IntelIJ Idea, выдели код и выполни code->reformat code
    например, положение открывающейся фигурной скобки (выше)
    и отступ для  open - изменятся

    всегда - отдавай код в реформатированном состоянии
    это хоороший тон
    это обеспечивает лучшую наглядность
*/
**************************************************
   open("https://todomvc4tasj.herokuapp.com/");


   /*
        пропуск одной строки - это полезно для форматирования кода
        две и более строки пропускать - уже нету смысла
        https://google.github.io/styleguide/javaguide.html#s4.6.1-vertical-whitespace
   */
**********************************************************
      //complete
      toggle("4");
      assertTasks("1", "3", "4");
      assertItemsLeft(2);
/*
    неплохо придумал - проверить после закомпличивания - и тексты тасок, и состояние счетчика ItemsLeft
    с одной стороны - убедился, что таски на all фильтре - отображаются вне зависимости от их статуса
    с другой стороны - счетчик ItemsLeft - показывает сколько тасок активно, и ты его проверил
*/
      clearCompleted();
      assertTasks("1", "3");
      assertItemsLeft(2);
/*
    а вот тут уже - нет смысла проверять ItemsLeft
    перегружать е2е сценарий проверками с невысоким приоритетом - не стоит

    да, проверка ItemsLeft повле закомпличивания таски - была очень к месту
    а вот после clear completed - без нее точно можно обойтись

    проверка ItemsLeft - имеет невысокий приоритет
    т к даже если она будет работать неверно - приложение можно вполне успешно использовать
    т е - особо на важной функциональности приложения это не скажется

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

    но - более не нужно это делать в рамках е2е сценария
    далее - когда мы напишем несколько тест-методов - мы разберем, где лучше ItemsLeft покрывать
*/
***************************************
      //complete & clear all
      toggleAll();
      clearCompleted();
      assertNoTasks();
/*
    Тут все ок
    в принципе - ты мог после закомпличивания одной таски - пропустить проверку
    как тут ты сделал

    но и твой вариант - вполне жизнеспособный и более точный
*/
****************************************
   ElementsCollection tasks = $$("#todo-list > li");
/*
    да, такое написание селектора - допускается
    советую дебе вокруг > - не ставить пробелов
    потому что пробел в css selector - имеет собственное значение
    и чтоб не путаться - лучше оперировать вот таким селектором "#todo-list>li"

    хотя и твой вариант - технически верный

    http://www.w3schools.com/cssref/css_selectors.asp
*/