Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public void testEditTaskAtAllFilter() {
- public void editAtAll() {
- public void editAtActive() {
- public void editAtCompleted() {
- /*
- посмотри - editAtAll - покрыто дважды (testEditTaskAtAllFilter - дублирует, предлагаю убрать этот метод)
- второе - посмотри - ты тестируешь редактирование в одной и той же тествой ситуации
- редактируем единственную таску
- а мы с тобой проговаривали - одну и ту же фичу - тестируй в разных тестовых ситуациях
- редактируем единственную таску
- редактируем вторую таску
- редактируем единственную видимую таску(а есть еще и не видимые)
- проверь - чтоб лишних тестов не было
- разнообразь тетсовые ситуации
- касается не только редактирования, а всех без исключения фиче-тестов
- используй в работе - тест-план, чтоб было легче сориентироваться
- */
- **********************
- @Test
- public void editAtCompleted() {
- given(ACTIVE, "a");
- toggleAll();
- /*
- complete all - мы тут не тестируем
- сразу создай - в гивен-методе - закомпличеную таску
- раз это фиче-тест для редактирования - значит редактирование и проверяй
- а все вспомогательные действия - реалзуй с помощью вызова гивен-метода
- также - смотри - метод - editAtCompleted
- так почему тестируем на All фильтре?
- проверь все фиче-тесты на это предмет
- */
- ******************************
- public void deleteAtAll() {
- /*
- удаляя единствунную таску - покрыть assertItemsLeft - не можешь
- а вот если не единственную - то сможешь)
- */
- **********************************
- @Test
- public void clearCompletedAtAll() {
- given(COMPLETED, "a", "b");
- clearCompleted();
- assertNoTasks();
- }
- /*
- используй одинаковое форматирование -
- пропуск строки между предварительными действиями и тестируемым, например
- тут тоже - можно было бы проверить assertItemsLeft - если бы в предварительных действиях - была добавлена и активная таска
- */
- ************************************
- @Test
- public void completeAtCompleted() {
- givenAtCompleted(ACTIVE, "a");
- assertNoTasks();
- assertItemsLeft(1);
- }
- /*
- А можно разве complete on completed filter выполнить?
- пользуйся тест-планом)
- */
- ********************************
- @Test
- public void FromAllToCompleted() {
- /*
- методы - называем с маленькой буквы
- далее - начни с глагода = что делаем
- switchFromAllToCompleted
- */
- given(ACTIVE, "a");
- /*
- во всех тестах - переходов по фильтрам -
- используй тестовую ситуацию, в которой заданы и активные, и закомпличеные таски
- цель - увидеть - как после перехода на другой фильтр отображаются таски в разных статусах
- в некоторых таких тестах - задай несколько тасок в каждом из статусов
- цель - увидеть как приложение справляется с фильтерингом для нескольких тасок
- */
- ********************************
- @Step
- private void confirmEdit() {
- $("#header>h1").click();
- }
- @Step
- private void confirmEditByTab() {
- $("#header>h1").pressTab();
- }
- /*
- такие методы - это перебор)
- просто, не начиная редактирования - вызови любой из этих методов
- как думаешь - они все равно будут подтверждать редактирование?
- старт редактирования и его подтверждение - должны быть в одном методе
- как ты сделал в методе cancelEdit (там содержится и старт редактирования, и его отмена)
- pressTab() - вызывай для поля, в котором вводил новый текст таски (см пример - с cancelEdit)
- кликать - разумнее на $("#new-todo")
- т к - мы уже используем этот селектор
- только не забудь - раз будем использовать этот селектор дважды -
- используй переменную для такого селектора
- */
Advertisement
Add Comment
Please, Sign In to add comment