Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- по коду
- public static final ExpectedCondition<Boolean> ajaxCompleted = new ExpectedCondition<Boolean>() {
- ...
- } catch (Exception e) {
- /*
- что-то это - пропустила в прошлый раз
- эксепшен, который ловим - нужно специфицировать получше
- если ловить вот так - catch (Exception e) - то можно пропустить ошибки, которые важно увидеть
- http://www.derekashmore.com/2014/11/is-it-bad-practice-to-catch-exception.html
- http://www.hans-eric.com/2009/10/31/the-bad-practices-of-exception-handling/
- http://stackoverflow.com/questions/2416316/why-is-the-catchexception-almost-always-a-bad-idea
- */
- ***************************************
- public class TodoMVCTest extends BaseTest {
- /*
- пока в вопросы - не заглядывала
- возможно, там будет ответ
- но пока по-прежнему не понимаю - зачем такой предок тест-классу )
- И с пейджом, и с тест-классом - уже все идеально
- */
- *******************************************************
- /*
- Табличка - шедевр ) это не сарказм)
- отлично просто
- покрытие запланировал - почти идеальное
- есть вопросы к сценарию
- потому и покрытие немного изменится
- но суть - верная
- покрыл лишь высокоприоритетное и лишь единожды
- и распределил равномерно по фильтрам
- это ок
- */
- ********************************************************
- Smoke scenario:
- 1 add task1, task2, task3
- /*
- Добавив сразу 3 таски - ты заберешь у себя возможность использовать неявные проверки
- см http://pastebin.com/xY6bgVat, строки 135-157
- да и это = все равно трижды покрытая операция)
- лучше сейчас добавь одну таску
- и позже - добавишь
- когда это будет нужно
- */
- 2 edit enter task1->etask1
- /*
- вариант нового текста task edited - будет лучше
- т к без загадок пояснит - что за таска и что с ней было
- */
- 3 navigate to active
- /*
- Смотри - тут - состояние списка и до перехода на фильтр, и после - одинаково
- А значит - мы сможем пропустить ошибку, когда переход на другой фильтр - вообще ничего не вызывает -
- в плане изменения состояня списка
- хорошо проверенный переход на фильтр = проверка в ситуации, когда
- состояние списка изменилось = переход на фильтр работает
- состояние списка правильное = переход на фильтр работает правильно
- я писала про это немного вот тут
- http://pastebin.com/YHM0iM7N, строки 131-142
- т е - покрыв до перехода на фильтр complete или complete all
- ты как раз получишь нужную тебе тестовую ситуацию
- соответственно - придешь на Active со списком тасок
- в котором нет ни одной видимой таски
- как раз - хороший момент добавить еще одну таску (это будет вторая и последняя)
- */
- 4 edit esc task2->task2
- /*
- Применяй термин cancel edit
- он точнее отражает - что мы делаем
- новый текст таски - используй другой какой-то
- например
- task2 edit canceled
- наша цель - чтоб еще и тестовые данные поясняли - что мы делаем
- */
- 5 complete task3,complete all (task1+task2)
- /*
- поскольку что-то из этих операций - мы покрыли выше
- тут останется одна
- и это хорошо - оба варианта будут проверены хорошо
- */
- 6 navigate to completed
- /*
- вот тут - отличная тестовая ситуация
- было = в списке нет видимых тасок
- перешли на комплитед
- в списке 2 видимые таски (пишу уже с учетом своих комментариев)
- */
- 7 activate task1
- 8 delete task2
- /*
- ниже ты покрываешь удаление таски
- тут - удаление не покрывай
- тем более - с учетом моих комментов - тут всего 2 таски
- одну переоткроешь
- вторую - clear completed
- и на all - переоткрітую удалишь
- */
- 9 clear completed
- 10 navigate to all
- 11 delete task1
- /*
- как видишь - правки к сценарию совсем небольшие
- так что - не нужно снова отдавать мне его на ревью
- подправляй сценарий
- подправляй обозначенное в таблице покрытие
- и впреред кодить)
- не забывай - каждое действие - должно быть проверено сразу
- если можно - используй неявные проверки через действие - так будет экономнее
- если это невозможно - делай явные проверки
- еще - разок покрой в сценарии "ITEMS LEFT"
- причины - http://pastebin.com/xY6bgVat, строки 98-100
- */
Advertisement
Add Comment
Please, Sign In to add comment