Това, което забелязвам на пръв поглед е, че getPresent(String name) метода ще върне резултат, дори и ако няма подарък с това име в торбата. В условието на задачата не пише какво да е поведението в този случай - в практиката би следвало да се хвърли exception, да се връща Optional<Present> или - в краен случай - null, но връщането на какъв да е Present е грешно отвсякъде. 

Допълнително, опит за връщане на подаръка с индекс 0, ще хвърли exception при празна торба.

Горната забележка важи и heaviestPresent() метода, който ще се опита да върне първия подарък, ако не е успял да намери най-тежък, а това е невъзможно да се случи, освен ако торбата е празна.

В условието ни се гарантира, че "You will always have a present added before receiving methods.", т.е. не се очаква exception, но логическата грешка остава.

Променете и в двата метода

.orElse(this.data.get(0))

на

.orElse(null)

и вижте дали ще мине решението.