Скорее всего потому что он замоканый
Твой метод делает - вызов репозитория - вызов маппера - return Нет ни одной строчки самостоятельной логики. Замокав маппер и репозиторий, ты по сути заменяешь тело метода на return expected. А потом просто проверяешь, что expected.equals(expected), в чём мало смысла. Не надо мокать всю логику разом, должно оставаться что-то, что ты собственно проверяешь
Я честно говоря тоже не совсем понимаю. По сути же в этом методе собственная логика отсутсвует. На маппер и на репозиторий должны быть свои тесты. А на этот метод нормальный тест, разве что можно добавить verify на вызов маппера и на вызов репозитория. Если мы не будем мокать маппер или репозиторий, то тест же перестанет быть модульным, разве нет?
Собственно, я бы предложил этот метод просто не тестировать, ибо там нечего
мне кто-то говорил что можно классы для sonar помечать чтобы они не засчитывались
Гугли, в пом в пропертях нужно прописывать теги sonar.exclusions, sonar.coverage.exclusions
Обсуждают сегодня