add("1");
       add("2");

       editTask("2");

[10:41]
add("2"); - не проверяет предыдущую операцию

[10:41]
а editTask("2"); - не годится для неявной проверки

[10:42]
т к уже у тебя 2 таски

[10:42]
а editTask("2"); - проверит состояние только одной из них

[10:42]
не торопись добавлять таски

[10:42]
надо будет - потом еще добавишь

[10:42]
editTask("2");

[10:42]
не понятно - на какой текст меняешь текст

[10:42]
лучше не скрывать это

[10:43]
editTask("2", "2 edited");

[10:43]
было бы попонятнее)

[10:43]
toggle("2 edited");
       clearCompleted();

[10:43]
не торопись удалять таски)

[10:43]
успеешь)

[10:44]
чем позже удалишь - теп лучше

[10:44]
меньше добавлять придется)

[10:44]
а еще - если после закопмличивания перейдешь на active  фильтр - то точно сможешь проверить переход с фильтра на фильтр

[10:45]
не забывай после каждой операции делать проверку

[10:45]
вот после clearCompleted();

[10:45]
про точную проверку перехода а эктив

[10:45]
сейчас как

[10:45]
было  = в списке = таска 1

[10:45]
перешли

[10:45]
стало = в списке  = таска 1

[10:46]
т е - не понятно - то ли все ок с фильтром - т к все корректно отображается, то ли фильтр вообще не работает - т к ничего в списке не помеялось

[10:46]
прикольно было бы так

[10:46]
было - закомпличеная таска\

[10:46]
перешли на эктив

[10:46]
стало - пустой список(т к таска закомспличена)

[10:47]
было-стало разное

[10:47]
стало = правильное

[10:47]
и допроверили - что таска закомплитиласть

[10:47]
и проверили - что фильтеринг работает

barocko [10:47 PM]
а как этот код реализовать editTask("2", "2 edited");?

julia.v.iluhina [10:47 PM]
про реализацию метода перехода на нужный фильтр - почитай в факе

[10:47]
щас линку дам

[10:48]
https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#bookmark=id.8bflixemdgfw

[10:48]
про как реализовать  editTask("2", "2 edited")

[10:49]
надо - чтобы у метода эдит - было 2 параметра

[10:49]
сейчас - один параметр

[10:49]
нужно - 2

[10:49]
первый - как искать тастку = старый текст

[10:49]
второй - какое значение вводить  = новый текст

[10:50]
у тебя в коде метода эдит - setValue("2 edited")

[10:50]
а ты будешь использовать не "2 edited", а значение этого параметра

[10:51]
escape("1"); - неудачное имя метода и неудачный набор параметров

[10:51]
вообще не понятно - что это за действие

[10:51]
cancelEdit("1", "1 edit cancelled") - было бы логичнее

[10:52]
логика - та де, как и для эдита - выше писала

[10:52]
кроме того - можно было бы еще что-то тут покрыть)

[10:52]
на эктив фильтре

[10:53]
начнем с того, что вторую таску стоило аж тут добавить)

[10:53]
оу

[10:53]
сорри

[10:53]
я не заметила - ты еще и toggleAll(); тут покрыл)

[10:54]
это правильное решение

[10:54]
только вот где проверки  - после cancel edit

[10:54]
после toggleAll

[10:54]
каждое действие - проверяй сразу

[10:55]
если можно - проверкой неявной - используя следующее действие

[10:55]
если не получается - явной

barocko [10:55 PM]
просто эссертом?

julia.v.iluhina [10:56 PM]
ну да - тексты в списке тасок

[10:56]
или что в списке тасок - нет видимых элементов

[10:56]
вспомни наш разговор про селекторы

[10:56]
и про особенности отображения тасок в списке - на эктив и комплитед фильтре

[10:57]
проверок - очень многих не хватает

[10:57]
вот выоплнил toggleAll

[10:57]
проверь - что нет теперь в списке тасок

[10:58]
перешел на комплитед - снова проверь - что в списке - такие-то таски

[10:58]
что касается reopenall();

[10:58]
во-первых - как считаешь - что чаще будут делать - переоткрывать одну таску или все таски?

barocko [10:59 PM]
одну таску наверное

julia.v.iluhina [10:59 PM]
)

[10:59]
таки да

[10:59]
а раз так - то лучше было бы покрыть именно реопен

[10:59]
а не реопен олл

[11:00]
count("1 item left");

[11:00]
что метод делает - проверяет

[11:01]
раз проверяет - значит название начни с assert

[11:01]
что проверяет = items left

[11:01]
assertItemsLeft

[11:01]
https://docs.google.com/document/d/13dNyFGbI7mV22UUhH8E0LJ7SzabAmX7Bw7VCHScYfiU/edit#heading=h.tezjc2whgbl7

[11:01]
еще)

[11:02]
проверить количество активных тасок = проверка логики

[11:02]
проверить всю фразу "1 item left" - это уже больше про проверку UI

[11:02]
лучше ограничиться проверкой логики)

[11:03]
для первой версии єтого задания - очень-очень неплохо

[11:03]
не забывай - всегда форматируй код

[11:03]
выделил

[11:03]
code-> reformat code

barocko [11:04 PM]
буду исправлять

julia.v.iluhina [11:04 PM]
чтоб всегда он был максимально наглядным)

barocko [11:04 PM]
а вот у меня вопрос про счетчик

[11:04]
а если например не осталось ни одной таски,то как написать код

julia.v.iluhina [11:04 PM]
никак)

[11:04]
счетчик становится не видимым ведь)

barocko [11:05 PM]
и то правда)

julia.v.iluhina [11:05 PM]
проверяй его тогда - когда есть что проверить)

[11:05]
про то - что в футере куча всего становится невидимым - при отстутствии тасок - излишне

[11:05]
т к это опять не про логику

[11:06]
а про UI

barocko [11:06 PM]
попробую разобратся,а то накосячил чуток)

julia.v.iluhina [11:07 PM]
в принципе - такого уровня код просто на ревью скидывай

[11:07]
очень может быть ревью читать будет легче

[11:07]
прежде всего тебе)

[11:07]
т к тут - поток сознания)

barocko [11:08 PM]
а на ревью куда в хип?

julia.v.iluhina [11:08 PM]
а там - по кусочкам бы код рассматривали

[11:08]
да так, как ты и сделал

[11:08]
линку выложил + статус выставил - ready to review

[11:08]
а я там же - оставлю линку на ревью)

[11:08]
ті просто просил тут консультации)

[11:08]
потому - вот тут и поговорили)