Общие замечания:

Не нужно использовать на уровне DAO/Repository имена классов, в которых присутствует слово "Controller", потому что это не контроллер. Нужно называть, например, AlbumDAO или AlbomRepository.
Присутствует дублирование кода(открытие и коммит транзакции).
Нет отката транзакций в случае исключительных ситуаций.
Использовать более конкретные интерфейсы вместо Collection, например List, Set...
Почему в тестовом коде используется entityManager при заполнении БД???
Почитать про каскадные типы операций. Вы уверены, что нужно везде использовать cascade = CascadeType.ALL? Например, при обновлении альбома по id(в методе main) создается новый трэк и все его значение принимают значение null - это неправильно.