Skip to content

fix: явные мапперы dto в entity и строгий парсинг - #38

Merged
vladcheck merged 3 commits into
mainfrom
vladcheck/mappers
Oct 13, 2026
Merged

vladcheck merged 3 commits into
mainfrom
vladcheck/mappers

Conversation

@vladcheck

Copy link
Copy Markdown
Owner

Что сделано

  • Добавлен пакет mapper: UserMapper (firstname/lastname -> firstName/lastName), ResourceMapper, FileMapper; приватные toEntity в AuthServiceImpl, UsersServiceImpl, ResourceServiceImpl и ручная конвертация в FileServiceImpl заменены на мапперы.
  • UserDto приведен к контракту бэкенда 1:1: поля firstName/lastName переименованы в firstname/lastname (как в RegisterRequest/UpdateUserRequest).
  • Из ApiClient убраны // FIXME и отключение FAIL_ON_UNKNOWN_PROPERTIES — парсинг снова строгий, дрейф контракта ловится fail fast.
  • Добавлены unit-тесты: маппинг всех полей, firstname/lastname -> firstName/lastName, null-список файлов -> пустой список, разбор бэкенд-JSON 1:1 и падение на неизвестных полях.

Проверки

  • mvn -B verify зеленый: 61 тест, 0 ошибок; spotless и checkstyle без нарушений.
  • Интеграционные тесты против живого бэкенда пропущены: в worktree нет .env, живого бэкенда нет.

Closes #19

@vladcheck vladcheck self-assigned this Oct 13, 2026
@vladcheck
vladcheck requested review from Deatheh and miyagi18 October 13, 2026 16:40
@vladcheck vladcheck added enhancement New feature or request java Pull requests that update java code labels Oct 13, 2026
@vladcheck
vladcheck marked this pull request as draft October 13, 2026 16:47
@vladcheck
vladcheck marked this pull request as ready for review October 13, 2026 16:59
@vladcheck
vladcheck enabled auto-merge (squash) October 13, 2026 16:59
@vladcheck
vladcheck merged commit 2d38b43 into main Oct 13, 2026
8 checks passed
@vladcheck
vladcheck deleted the vladcheck/mappers branch October 13, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request java Pull requests that update java code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Реализовать мапперы с бэкенда на фронт

1 participant