Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- @Before
- public void reset() {
- resetBase();
- }
- /*
- мне кажется, было бы точнее использовать вызов с указанием имени класса-ресурса
- касается всего кода
- не resetBase();
- а TasksRest.resetBase();
- или даже TasksRest.reset(); (можно сократить имя метода в таком случае - как мы и для других методов сделали)
- */
- ******************************************
- @Test
- public void testUnauthorizedReadTasks() {
- response = requestTo(uri).get();
- assertEquals(403, getResponseStatus());
- assertEquals("Unauthorized access", getErrorAnswer());
- }
- /*
- у нас тест-класс называется - RESTfulTasksTest
- т е - про таски мы тут уже написали
- и потому - в именах тест-методов уже про таски можно не писать
- еще момент по пути - согласно правилам CamelCase - не RESTfulTasksTest, а RestFulTasksTest
- https://google.github.io/styleguide/javaguide.html#s5.3-camel-case
- раз мы уже тут оперируем переменной response, то и методы getResponseStatus() и getErrorAnswer() - лишнее
- сравни с вариантом ниже
- соответствующие методы из класса-ресурса - убирай
- */
- @Test
- public void testUnauthorizedRead() {
- Response response = requestTo(TasksRest.uri).get();
- assertEquals(403, response.getStatus());
- assertEquals("Unauthorized access", response.readEntity(ErrorContainer.class).getError());
- }
- **********************************************
- @Test
- public void testReadTasks() {
- get();
- assertEquals(200, getResponseStatus());
- assertEquals(2, getTasksSize());
- assertTasks(DEFAULT_TASKS[0], DEFAULT_TASKS[1]);
- }
- /*
- Тут - хитрее)
- как и для остальных операций, для get - тоже проверяй статус в методе сервиса
- метод get() - возвращает список тасок
- и именно эти таски и надо сверять с DEFAULT_TASKS
- но - нужно ли использовать assertTasks?
- ведь там - мы заново вызываем get
- а вот вторая строчка из assertTasks - как раз то, что нужно и здесь
- грамотнее вместо assertTasks - тут вызвать именно ее
- да и размер списка - проверять незачем
- при сравнении двух списков с помощью assertEquals - и это проверим
- вот и метод getTasksSize() - не нужен
- */
- ***********************************************
- @Test
- public void testCreate() {
- Task myTask = new Task("my task", false, "created task", uri + "/3");
- ...
- /*
- Кстати, есть смысл создавать таску не только по полной информации,
- но и только по title - чтобы полнее проверить создание таски
- Стоит реализовать 2 тест-метода -
- testCreateWithTitle() (такой надо реализовать)
- и testCreateWithFullInformation()(собственно, такой у тебя уже есть)
- В testCreateWithTitle() - тоже проверку делай через assertTasks (на этот момент будут известны все свойства добавленной таски -
- так что это не проблема)
- */
- **********************
- public void testDelete() {
- /*
- В целом - все ок
- единственное - не надо размер списка тасок проверять
- ведь assertTasks и это выполнит
- */
- *******************************
- public void testCreateUpdateDelete() {
- Task createCRUDTask = new Task("my CRUD task");
- create(createCRUDTask);
- /*
- делай проверки - после каждого действия
- тут уже нужен assertTasks
- а следующее действие мы не можем посчитать косвенной проверкой
- во-первых - потому что тасок уже 3 и надо проверить состояние каждой
- во-вторых - даже если бы таска была единственной - следующая операция не проверит состояние всех свойств таски
- */
- Task updatedTask = new Task("updated description", true, "updated task", uri + "/3");
- update(updatedTask);
- /*
- и тут проверку встрой
- */
- delete(3);
- assertEquals(2, getTasksSize());
- /*
- этого не нужно
- */
- assertTasks(DEFAULT_TASKS[0], DEFAULT_TASKS[1]);
- }
- /*
- согласна - что в е2е тесте все операции выполняем уже над свежедобавленной таской, а не над теми, что были по умолчанию
- это улучшит покрытие
- хороший ход)
- */
- ********************************
- /*
- Итого - остались мелочи
- подчистить код, убрать лишнее, добавить недостающее
- но - уже ничего особенно нового или сложного
- */
Advertisement
Add Comment
Please, Sign In to add comment