julia_v_iluhina

Untitled

Dec 29th, 2016
88
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 3.61 KB | None | 0 0
  1. http://joxi.ru/Q2KpJYOs44GxoA
  2.  
  3. /*
  4.     1 - edit by press Enter  - это просто edit
  5.         который уже выше перечислен - вместе с другими базовыми операциями
  6.         из группы Additional edit operations - убирай эту строку
  7.  
  8.         обратила внимание, приоритеты edit by press Enter и edit - разные
  9.         может, конечно, ты под edit имел в виду что-то другое
  10.         хотя, в прошлом задании - edit - именно так и реализован = edit by press Enter
  11.         что верно
  12.  
  13.     edit and cancel - это по пути уже
  14.         в предыдущем задании - имя метода соответствующего - более точно отражает суть дела
  15.         startEditThenCancel - лучше
  16.         чем edit and cancel
  17.  
  18.     2 - определись, где покроешь delete - в e2e или feature тесте
  19.         и там, и там покрывать - будет перебором - ведь мы по-прежнему реализуем Smoke
  20.         надумаешь покрыть в feature тесте - заодно покрой и проверку items left counter
  21.         это можно будет сделать, если будешь удалять не единственную таску
  22.         то же самое - насчет edit - покрой это или в e2e или feature тесте (единожды)
  23.  
  24.     3 - как собираешься покрыть проверку items left counter в feature тесте,
  25.         если само действие - edit and cancel - покрываешь в е2е?
  26.  
  27.     4 - у этих Additional edit operations - приоритет низкий
  28.         и потому - покрывать их в рамках Smoke покрытия - лишнее
  29.         советую не покрывать их пока
  30.         если все же хочешь покрыть - тогда в коде - надо будет обозначить эти тесты как extra coverage
  31.  
  32.     5 - приоритет действия edit and cancel на completed фильтре
  33.         не может быть выше приоритета edit на том же фильтре
  34.  
  35.     6 - приоритет reopen all на Active фильтре = будет низкий
  36.         как и приоритет complete All на Completed фильтре
  37.         в таких контекстах - эти действия будут меньше востребованы
  38.  
  39.     7 - эти обозначения уже излишни
  40.         т к мы тут уже оперируем более точными категориями - приоритетами и покрытием
  41.  
  42.     одинаковое выравнивание текста в столбцах B, C, D, E - тоже облегчит восприятие таблицы
  43.     советую это сделать
  44.  
  45.     приоритет items left counter
  46.         могу допустить, что после разных действий - он будет разным
  47.         но вот высоким - он точно не будет
  48.        
  49.     в общем-то, уже достаточно информации для того чтоб идти дальше
  50.     по этому ревью - доработай это задание
  51.     и двигайся дальше
  52.     не думаю, что тут будут какие-то сложности
  53. */
Advertisement
Add Comment
Please, Sign In to add comment