Add item requests #7
Conversation
|
Не знаю как обойти ошибку UnfinishedStubbingException в BookingServiceTest, погуглила, но никакого решения не нашла |
TyutterinYakov
left a comment
TyutterinYakov
left a comment
There was a problem hiding this comment.
Добрый день! Отметил то, что следует исправить на данном этапе. Так же следует разобраться с ошибкой, из-за которой не запускаются тесты на гитхаб.
Хорошего дня! Если возникнут вопросы - пиши в пачку, буду рад помочь🌅
src/main/java/ru/practicum/shareit/request/dto/ItemRequestDto.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/practicum/shareit/item/service/ItemServiceImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/practicum/shareit/request/service/RequestServiceImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/practicum/shareit/request/service/RequestServiceImpl.java
Outdated
Show resolved
Hide resolved
src/main/java/ru/practicum/shareit/request/service/RequestServiceImpl.java
Outdated
Show resolved
Hide resolved
TyutterinYakov
left a comment
TyutterinYakov
left a comment
There was a problem hiding this comment.
Добрый день! Поздравляю с успешной сдачей спринта! Отметил еще парочку мелких недочетов, но думаю, что ты справишься с ними вне ревью! Успехов в обучении и прекрасного дня!🌅🥳
| RequestDTOWithItems itemRequestDtoResult; | ||
| List<ItemDTO> itemsToReturn; | ||
| if (!items.isEmpty()) { | ||
| itemsToReturn = items.get(request) |
There was a problem hiding this comment.
В данном случае следует использовать getOrDefault, так как не у всех ItemRequest гарантированно будут предметы, из-за чего может возникнуть NPE
No description provided.