Стоит длинное выражение достать из [], страшно как-то выглядит

а еще r += 3. почему 3, откуда это число? почему при количестве рабочих дней > 10 вываливается ошибка? кругом магические числа.

но стоит учитывать, что человек только учится и не придираться пока к подобному.

Не надо это учитывать, пусть сразу тесты пишет и тестируемый код

Да и вообще логика непонятная, че тут происходит? Хотя может код просто недописан

+
Ну и эксепшоны обрабатывает ещё. А ещё надо знать что возвращает метод, и не возвращает ли он чего-то, что будет неприемлемо для передачи сразу аргументом на вход другому методу

Кстати

Я много че умею в программировании, но вот тесты писать не умею

Суть теста в общем случае: подать на вход такие различные условия и проверить что при определенных заданных условиях получаем ожидаемый ответ.
Это к юнит-тестам конечно же относится.
Тут мы проверяем что код ведёт себя так как запланировали.
Вторым главным эффектом написанного теста, является возможность проверки кода, если что-то внутри поменяли. Если тесты проходят и тесты покрывают пограничные условия и/или весь алгоритм, значит с большой вероятностью метод будет работать как и прежде.
Ну и это мое имхо, я тоже не большой спец по тестам. Это скорее то как я вижу этот момент

действительно

какая то магия

точно не потому что по заданию у каждого сотрудника должно быть максимум 10 рабочих дней

и не потому что для того чтоб чел работал максимум 10 дней он должен работать через 2 дня

и поэтому р равно 3

Ещё есть UI тесты, которые проверяют работу интерфейса

ну да

давай пришлю сюда всю свою говнопортянку

чтоб у тебя инсульт случился

прочитай про магические числа. те программисты, которые будут работать с этим кодом после тебя, в глаза не видели и не увидят первоначальное тз. а теперь предположим, что эта тройка/десятка будет где-то еще в коде встречаться, что ты будешь делать, если тз изменится? будет работать чел не через 2 дня, а через день? везде по коду ручками будешь 3 на 2 менять?

Разумовский-style