Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Test
- public void testTaskDelete() {
- addTasks("a","b");
- filterActive();
- deleteTask("a");
- assertTasksAre("b");
- assertItemsLeft(1);
- }
- /*
- http://joxi.ru/5md7jYwtvz941r
- не все советы учел
- и то, и то - важно - учти оба момента для всех фиче-тестов
- указать в имени метода - на каком фильтре покрыли - полезно
- в реальности - тесты, реализованные для smoke покрытия - это лишь неболшая часть от всего реализованных тестов
- и хорошо бы - чтобы из названий тест-класса и тест-метода - возникало полное понимание - что мы тестировали
- в рамках Smoke - мы покрыли Delete - для активного фильтра
- а для полного покрытия, допустим, мы реализовали аналогичный фиче-тест - и для All, и для Completed фильтров
- и вот тут важно - чтобы из названий тест-класса и тест-метода - мы для каждого из таких методов понимали - что он делает
- про комментарий //given
- тоже важная и полезная вещь
- сейчас предварительные действия реализованы через действия на UI =
- мы просто вызываем методы-действия над тасками, чтобы получить нужную нам тестовую ситуацию
- и из кода без комментариев - не ясно - это мы тестируем эти действия
- или все же только готовим тестовую ситуацию
- цель - обозначить - вот это = предварительные действия
- а вот это - мы тестируем
- когда предварительные действия будут вызываться как given...(...) - так и в таком комментарии не будет потребности
- deleteTask vs TaskDelete ?
- правильнее придерживаться одного термина
- действие = deleteTask
- тест этого действия = testDeleteTask
- тест этого действия на Active фильтре = testDeleteTaskAtActive
- test+название действия+на каком фильтре
- */
- ***********************************************
- public void testCancelTaskUpdate(){
- ...
- startTaskUpdate("a", "a edit canceled").pressEscape();
- ...
- }
- public void testTaskUpdate(){
- ...
- startTaskUpdate("a", "a edit").pressEnter();
- ...
- /*
- тут вроде бы стройная картина
- термин = TaskUpdate
- остальное - производные от него
- startTaskUpdate
- testTaskUpdate
- testCancelTaskUpdate
- а теперь вспоминаем про testDeleteTaskAt....
- для Delete - писали - глагол-действие+Task (удалить таску = что делаем + над чем делаем)
- для Update - уже Update-уже существительное (редактирование таски = над чем делаем + какой процесс)
- и снова не красиво....
- советую придерживаться одной линии
- имя вспомогательного метода = начни с описания действия, раз важно для тебя - уточни субъект - над чем выполняется действие
- имя фиче-теста = test + имя метода-действия + описание контекста
- startUpdateTask
- testCancelUpdateTaskAtActive(){
- testUpdateTaskAt...
- deleteTask
- testDeleteTask
- без Task в именах методов(вспомогательных для действий и фиче-тестов) - было бы попроще)
- но и в твоем варианте можно найти мотивы для именно такого порядка слов в именах методов
- если в вопросах нейминга остаешься при своем мнении насчет места слова Task в именах методов - я поставлю accepted
- accepted - значит принято, но есть вопросы, на которые можно закрыть глаза по моему мнению
- */
- *******************************
- @Test
- public void testTaskUpdate(){
- addTasks("a");
- filterActive();
- /*
- Вроде бы запланировано покрытие этого на all
- посмотри в тест-плане
- */
Advertisement
Add Comment
Please, Sign In to add comment