първото което забелязах без да съм дебъгвал е, че във Product за setCost трябва да имаш проверка за парите също -> 

if (cost < 0) {
    throw new IllegalArgumentException("Money cannot be negative");
}

и проверката за имената проверявай винаги първо дали не е null защото String е нулабъл -> 

if (name == null || name.trim().isEmpty()) {
    throw new IllegalArgumentException("Name cannot be empty");
}

Във class Main 1. махни try {...} catch {...} блоковете от всякъде. След това, след декларирането на двата Map<> огради всичко останало във един try {...} catch {...} за да може, ако получиш невалидни данни за People или за Product - да ти изпечата съобщението от грешката и да ти прекъсне изпълнението на програмата на цяло. Вътре трябва да има само един try {...} catch {...} блок, който да огради единственно копуването на продуктите - като методъд buy() трябва да ти хвърля exception, защото не е добре да печаташ директно от класът както си го направил не принтирай от класа, има си принцип за това казава се Single Responsibility.