julia_v_iluhina

Untitled

Dec 30th, 2016
93
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Java 6.16 KB | None | 0 0
  1. по коду
  2.  
  3.  public static final ExpectedCondition<Boolean> ajaxCompleted = new ExpectedCondition<Boolean>() {
  4. ...
  5.             } catch (Exception e) {
  6. /*
  7.     что-то это - пропустила в прошлый раз
  8.     эксепшен, который ловим - нужно специфицировать получше
  9.     если ловить вот так - catch (Exception e) - то можно пропустить ошибки, которые важно увидеть
  10.     http://www.derekashmore.com/2014/11/is-it-bad-practice-to-catch-exception.html
  11.     http://www.hans-eric.com/2009/10/31/the-bad-practices-of-exception-handling/
  12.     http://stackoverflow.com/questions/2416316/why-is-the-catchexception-almost-always-a-bad-idea
  13. */
  14. ***************************************
  15. public class TodoMVCTest extends BaseTest {
  16. /*
  17.     пока в вопросы - не заглядывала
  18.     возможно, там будет ответ
  19.  
  20.     но пока по-прежнему не понимаю - зачем такой предок тест-классу )
  21.  
  22.     И с пейджом, и с тест-классом - уже все идеально
  23. */
  24. *******************************************************
  25. /*
  26.     Табличка - шедевр ) это не сарказм)
  27.     отлично просто
  28.  
  29.     покрытие запланировал - почти идеальное
  30.     есть вопросы к сценарию
  31.     потому и покрытие немного изменится
  32.  
  33.     но суть - верная
  34.     покрыл лишь высокоприоритетное и лишь единожды
  35.     и распределил равномерно по фильтрам
  36.     это ок
  37. */
  38. ********************************************************
  39. Smoke scenario:
  40. 1   add task1, task2, task3
  41. /*
  42.     Добавив сразу 3 таски - ты заберешь у себя возможность использовать неявные проверки
  43.     см http://pastebin.com/xY6bgVat, строки 135-157
  44.     да и это = все равно трижды покрытая операция)
  45.  
  46.     лучше сейчас добавь одну таску
  47.     и позже - добавишь
  48.     когда это будет нужно
  49. */
  50. 2   edit enter task1->etask1
  51. /*
  52.     вариант нового текста task edited - будет лучше
  53.     т к без загадок пояснит - что за таска и что с ней было
  54. */
  55. 3   navigate to active
  56. /*
  57.     Смотри - тут - состояние списка и до перехода на фильтр, и после - одинаково
  58.     А значит - мы сможем пропустить ошибку, когда переход на другой фильтр - вообще ничего не вызывает -
  59.     в плане изменения состояня списка
  60.  
  61.     хорошо проверенный переход на фильтр = проверка в ситуации, когда
  62.     состояние списка изменилось = переход на фильтр работает
  63.     состояние списка правильное = переход на фильтр работает правильно
  64.  
  65.     я писала про это немного вот тут
  66.     http://pastebin.com/YHM0iM7N, строки 131-142
  67.  
  68.     т е - покрыв до перехода на фильтр complete или complete all
  69.     ты как раз получишь нужную тебе тестовую ситуацию
  70.  
  71.     соответственно - придешь на Active со списком тасок
  72.     в котором нет ни одной видимой таски
  73.     как раз - хороший момент добавить еще одну таску (это будет вторая и последняя)
  74. */
  75. 4   edit esc task2->task2
  76. /*
  77.     Применяй термин cancel edit
  78.     он точнее отражает - что мы делаем
  79.     новый текст таски - используй другой какой-то
  80.     например
  81.     task2 edit canceled
  82.  
  83.     наша цель - чтоб еще и тестовые данные поясняли - что мы делаем
  84. */
  85. 5   complete task3,complete all (task1+task2)
  86. /*
  87.     поскольку что-то из этих операций - мы покрыли выше
  88.     тут останется одна
  89.     и это хорошо - оба варианта будут проверены хорошо
  90. */
  91. 6   navigate to completed
  92. /*
  93.     вот тут - отличная тестовая ситуация
  94.     было = в списке нет видимых тасок
  95.     перешли на комплитед
  96.     в списке 2 видимые таски (пишу уже с учетом своих комментариев)
  97. */
  98. 7   activate task1
  99. 8   delete task2
  100. /*
  101.     ниже ты покрываешь удаление таски
  102.     тут - удаление не покрывай
  103.     тем более - с учетом моих комментов - тут всего 2 таски
  104.     одну переоткроешь
  105.     вторую - clear completed
  106.     и на all - переоткрітую удалишь
  107. */
  108. 9   clear completed
  109. 10  navigate to all
  110. 11  delete task1
  111. /*
  112.     как видишь - правки к сценарию совсем небольшие
  113.     так что - не нужно снова отдавать мне его на ревью
  114.  
  115.     подправляй сценарий
  116.     подправляй обозначенное в таблице покрытие
  117.     и впреред кодить)
  118.  
  119.     не забывай - каждое действие - должно быть проверено сразу
  120.     если можно - используй неявные проверки через действие - так будет экономнее
  121.     если это невозможно - делай явные проверки
  122.  
  123.     еще - разок покрой в сценарии "ITEMS LEFT"
  124.     причины - http://pastebin.com/xY6bgVat, строки 98-100
  125. */
Advertisement
Add Comment
Please, Sign In to add comment