Controllers films users

[dendzim]

Реализована функциональность согласно ТЗ спринта 10.
Добавлены тесты валидаций и логирование.

[VadimZharkov]

Методы лучше называть с использованием глагола, т.к. по сути это некоторые действия. Т.е. здесь лучше
validateFilm(Film film)

[VadimZharkov]

Хотел бы предложить тебе методику написания метода с помощью так называемого раннего выхода (early return). Смысл в том, чтобы выходить из метода сразу при невыполнении условий, необходимых для успешного выполнения метода. Код будет более легким для чтения.
https://habr.com/ru/articles/348074/

Твой метод можно переписать

@PutMapping
public Film update(@Valid @RequestBody Film newFilm) {
    if (newFilm.getId() == null) {
        log.warn("Не указан id");
        throw new ValidationException("Id должен быть указан");
    }
    if (!films.containsKey(newFilm.getId())) {       
        log.warn("Фильм с указанным id не найден");
        throw new ValidationException("Фильм с id = " + newFilm.getId() + " не найден");
    }
    validateFilm(newFilm);
    Film oldFilm = films.get(newFilm.getId());
    oldFilm.setDescription(newFilm.getDescription());
    oldFilm.setDuration(newFilm.getDuration());
    oldFilm.setName(newFilm.getName());
    oldFilm.setReleaseDate(newFilm.getReleaseDate());
    log.info("Данные о фильме: {} обновлены", oldFilm);
    return oldFilm;
}

[VadimZharkov]

Хорошая работа, все сделано верно. Оставил пару стилистических рекомендаций.

[dendzim]

Благодарю за ревью. Такой подход и вправду делает кот читабельнее.