Add item requests and gateway #3
Conversation
Рефакторинг мапперов на static методы
…вил тестирование контроллеров
server/src/main/java/ru/practicum/shareit/booking/BookingController.java
Outdated
Show resolved
Hide resolved
server/src/main/java/ru/practicum/shareit/booking/storage/BookingStorageInDb.java
Show resolved
Hide resolved
| import lombok.Builder; | ||
|
|
||
| @Builder | ||
| public record ItemToUpdateDto( |
There was a problem hiding this comment.
Нужно добавить аннотации для валидации
There was a problem hiding this comment.
Не нужно. Нет правил к заполнению
There was a problem hiding this comment.
Ок, если частично обновлять, то не нужно.
| import ru.practicum.shareit.rest.RestService; | ||
|
|
||
| @Service | ||
| public class BookingService extends RestService { |
There was a problem hiding this comment.
Эти классы не логично называть Service, т.к. в них нет никакой бизнес-логики по работе с сущностями. Лучше назвать RestClient и BookingClient и пометить аннотацией @Component
There was a problem hiding this comment.
Сервис, потому что стоят за контроллером. А клиентом не хочу называть, т.к. клиент внутри используется.
И бизнес-логика подразумевалась в задании, в описании маршрутизации рест-запросов
There was a problem hiding this comment.
Классы получают аннотацию @Service потому что они принадлежат сервисному слою приложения, а не потому что стоят за контроллером. Маршрутизация -это не бизнес-логика. В задании как раз говорится, что "shareIt-server будет содержать всю основную логику ", а в shareIt-gateway нужно вынести "всю валидацию входных данных".
PS
Если хочешь подчеркнуть, что основная задача класса маршрутизация, то можно назвать Router
VadimZharkov
left a comment
VadimZharkov
left a comment
There was a problem hiding this comment.
В тесты нужно добавить интеграционные тесты. Это когда хранилище не мокаем, а используем реальную базу. В данной работе достаточно будет протестировать репозитории с помощью @DataJpaTest
No description provided.