itev-50

[IgorMaksymov]

issue #50 "I go to event" added

[alex-anakin]

Если в тесте EventRestControllerTest дописать так: @WithMockUser(username = "testSubscribejjhhkjhkjhr", password = "testSubscjmhbkjhriberPassword", authorities = "subscrzxczxciber")

• все тесты проходят.

в методе getVisitors вместо List visitors = eventService.getVisitors(event); ставим List visitors = new ArrayList<>();

тоже самое в iWillNotGo User user = new User() вместо User user = userService.getUserByName(SecurityContextHolder.getContext().getAuthentication().getName());

ТЕСТЫ НЕ ПАДАЮТ.

(скорее всего и в iWillGo будет то же самое)

[IgorMaksymov]

тесты не падают потому что в контексте юзер залогинен и там же ты ему задаешь параметры. тут же нету проверки существует ли он в БД и т.д. Если закомментить аннотацию, то будет ошибка Unauthorized. И в самих сервисах нету проверки на логин и прочее, она есть только в spring-security.xml такой строки у меня нет, в ней нет необходимости: User user = userService.getUserByName(SecurityContextHolder.getContext().getAuthentication().getName());

В методе getVisitors возвращается ПУСТОЙ лист с 0 элементов "[ ]" т.к. у него в Билдере нет посетителей и нет подвязаной БД.

думаю, что тут verify можно вообще убрать, а оставить только andExpect.

[vaa25]

• HttpStatus.NOT_FOUND в каких случаях возвращают?
• Термины Subscribe/unsubscribe у нас используется для подписки на рассылку
• Зачем возвращать Event в фиче I Will Go ?
• if (event == null) return new ResponseEntity(HttpStatus.NOT_FOUND); ТРИ!!!! раза повторяется!!!
• HttpStatus.CREATED тоже не в тему
• В тестах зачем atLeastOnce()???
• getUserByID по jcc как нужно назвать?
• myEvents противоречит принципу YAGNI
• Везде в контроллерах переделал ссылки полностью на методы, хотя Виктор прямым текстом сказал, что общую часть нужно выносить на уровень класса
• getVisitors в EventMapper и getUserEvents в UserMapper даром не нужны хотя бы потому, что они дублирут маппинг @Results из оригинальных мапперов
• В UserMapper нарушена разметка текста
• List в User не нужен вообще, не говоря уже о том, что оно с List visitors образует замкнутую петлю
• Изменения в бд внес прямо в ченчсетах!!!! причем добавил каскадное удаление, в то время как от удаления мы вообще отказываемся
• В тестах, даже в тех что нужны, нет форматирования кода пустыми строками
• На Liquibase changeset LOG забил болт

[IgorMaksymov]

Ты не мог эти вопросы раньше написать, чтобы я мержил с исправлениями? 1) это вопрос на знание случае когда его возращают или что? 2) Да это можно переделать на assign. 5) предложи альтернативу и я переделаю, Анакина устроило. Можно заменить на accepted 6) не помню, что-то не проходило в тесте. 7) getUserById это красиво ? можно спутать с Ld 8) не противоречит, а так и задумано. Я зашел в свой профиль и хочу посмотреть на что подписан. 9) ДА, я тоже это только заметил. Жду Сашу чтобы выяснить как это все быстро пофиксить. 10) да, это можно сократить. 11) номер строки пжл. 12) А как мне брать и передавать список посетителей или список ивентов Юзера? Петлю не образуют. 13) Да, внес, накосячил. С Анакиным согласовано. Удаление юзера или отписка от события? 14) я не люблю пустые строки и стараюсь их избегать

[vaa25]

1) и то и другое 5) HttpStatus.OK 6) ну так давай будем пихать в тесты все что угодно лишь бы проходило 7) красиво 8) ссылку на issue или user story дай, где эта задумка описана 13) согласовано до мержа в мастер или после? удаление юзера 14) странно, а меня Анакин не спрашивал, нравится или нет, когда велел пустые строчки вставлять

[IgorMaksymov]

1) https://ru.wikipedia.org/wiki/HTTP_404 Чувак перешел по битому линку и получил 404. там в запросе идет ИвентИД. 8) тебе скрин чата с Кучиным скинуть? 13) думаю, тут ответ очевиден. мне пришлось добавить этот ключ, иначе не проходили тесты. 14) мне не велел. Скажет - сделаю.