julia_v_iluhina

Untitled

Sep 22nd, 2016
79
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 8.73 KB | None | 0 0
  1. Smoke ETE test
  2. All filter:
  3.  
  4.     add (1, 2 ,3)
  5.     /*
  6.         добавив 3 таски сразу - получаем ситуацию
  7.         когда следующим действием  - не проиверить предыдущее
  8.  
  9.         так зачем тогда добавлять 3 таски сразу?
  10.         добавил бы лишь одну - так операция cancel edit проверила бы добавление таски
  11.  
  12.         и мы бы сэкономили на одном шаге
  13.         для е2е, реализующего smoke coverage - эффективность - это очень важно
  14.  
  15.         кроме того  add (1, 2 ,3) - это не одна, а 3 операции
  16.         потому - таким образом экономить точно не надо
  17.     */
  18.     count all
  19.  
  20.     cancel edit (1)
  21.     /*
  22.         у этого метода будет 2 параметра
  23.         старый текст таски = как найти таску
  24.         новый текст таски = на что поменяем текст таски
  25.  
  26.         смысл этой операции в том - что уже поменяв текст таски на другой и нажав esc
  27.         мы отменим редактирование = вернем старый текст таски
  28.     */
  29.     count all  (1, 2, 3)
  30.  
  31.     complete (1)
  32.     clear completed
  33.     /*
  34.         я бы не торопилась очищать закомпличеные таски
  35.         чуть ниже объясню почему
  36.  
  37.         помимо этого, вообще не надо торопиться удалять таски
  38.         не важно - каким способом
  39.         чем позже по сценарию мы их удалим
  40.         тем меньше тасок придется создавать
  41.     */
  42.     count all (2, 3)
  43.  
  44. Complete filter:
  45.     /*
  46.         complete__d___ filter
  47.  
  48.         переход на другой фильтр - это тоже операция
  49.         и это действие тоже требует проверки
  50.  
  51.         сейчас ситуация - мы переходим на completed фильтр
  52.             на all в списке - были видимы таски 2 и 3
  53.             перешли а completed фильтр
  54.             в списке - ни одной видимой таски
  55.         ситуация неплохая
  56.             было-стало для списка - разные
  57.             + стало = корректное для этого фильтра
  58.             проверили переход на фильтр - точно
  59.         но - ситуация не идеальная
  60.             вспомни свой main use cases list
  61.             для completed фильтра  - ты не указывал complete all - как main use case
  62.             и был прав)
  63.             такую операцию - делают редко на этом фильтре
  64.             и приоритет у нее - не будет высоким
  65.             значит - в рамках smoke покрытия - не нужно покрывать complete all на completed фильтре
  66.             то же можно отнести и к добавлению таски - на этом фильтре не стоит добавлять таску из тех же
  67.             соображений
  68.  
  69.             получается - перейдя на этот фильтр в состоянии, когда в списке - нет видимых тасок
  70.             тут покрыть мы ничего не сможем
  71.  
  72.             а это - плохо
  73.             лучше распределить по всем фильтрам операции
  74.             и поравномернее
  75.             это улучшит покрытие - мы получим фидбек и о таких проблемах:
  76.             если на каком-то из фильтров перестанут корректно работать операции -
  77.             мы это вероятнее поймаем еще в этом тесте -
  78.             если операции будут распределены равномерно по фильтрам
  79.     */
  80.     complete all
  81.     /*
  82.         мы на all между закомпличиванием и clear completed пропускали проверку
  83.         чтобы не делать проверок недостаточно точных
  84.  
  85.         тут - такой ситуации нету
  86.         нужна проверка
  87.     */
  88.     clear completed
  89.     /*
  90.         мы это уже проверяли, не нужно повторять - достаточно операцию в рамках сценария покрыть единожды
  91.     */
  92.     count empty
  93.  
  94. Active filter:
  95.     /*
  96.         было - список пуст
  97.         перешли на другой фильтр
  98.         стало - список пуст
  99.  
  100.         с одной стороны - согласно логики приложения - все ок -
  101.         состояние списка верное
  102.  
  103.         с другой стороны - оно не поменялось
  104.         и такое может быть и в случае - если переход на другой фильтр вообще не работает
  105.  
  106.         потому - лучше получить тестовую ситуацию такого рода
  107.         было - в списке одна картина
  108.         перешли на фильтр
  109.         стало - в списке другая картина
  110.         т е - важно - чтобы было-стало отличались
  111.  
  112.         нужна проверка после перехода на фильтр)
  113.     */
  114.     add 1
  115.     /*
  116.         советую добавлять таски уже с другими текстами
  117.         чтобы даже при беглом взгляде на код - было понятно - где какая таска
  118.     */
  119.     count all
  120.  
  121.     edit (1, 2)
  122.     /*
  123.         кстати - вот тут - можно было и не делать проверку count all перед edit
  124.         таска в списке одна
  125.         так что edit  - проверит add
  126.  
  127.         и далее ты тоже хорошо неявную проверку использовал - с delete
  128.  
  129.         про edit (1, 2)
  130.         лучше edit (1, 1 edited)
  131.         тогда - глядя на код - будет яснее - что с таской уже было ранее
  132.     */
  133.     delete (2)
  134.     count empty
  135. **********************************************
  136. /*
  137.     анализируем покрытие
  138.  
  139.     All filter:
  140.  
  141.     add * 3
  142.     cancel edit
  143.     complete
  144.     clear completed
  145.  
  146.     Completed filter:
  147.     complete all (покрыто в низкоприоритетном варианте)
  148.     clear completed (лишнее повторение)
  149.  
  150.     Active filter:
  151.     add*1
  152.     edit
  153.     delete
  154.  
  155.  
  156.  
  157.     Что еще не покрыто
  158.         reopen
  159.        
  160.         switch to filter All (на 2 других фильтра мы переходили, надо и переход на этот фильтр покрыть)
  161.        
  162.         “items left” counting - да, не высокоприоритетное, Но - этот счетчик реагирует на любые изменения
  163.         потому стоит его единожды покрыть в рамках этого е2е. Больше не нужно -
  164.         т к код и так не самы простой у е2е - не надо его загромождать, потом разберем - где его еще разумно покрывать.
  165.         Пока будет достаточно одного раза
  166.        
  167.     Можно обойтись меньшим количеством тасок - если удалять их в последнюю очередь
  168.     Реально - может и 2-ух тасок на весь сценарий хватить
  169.        
  170.     Стоит добавлять таски по одной - так будет больше возможностей использовать неявные проверки
  171.     Да и если добавлять таски на разных фильтрах - да, мы повторимся - но контексты будут разные
  172.     и это лучше  - чем повторение действия, да еще и на одном контексте
  173.        
  174. */
Advertisement
Add Comment
Please, Sign In to add comment