Т.е. по определению та же ф-я с теми же данными должна выдавать одинаковый результат

Определение нужно уточнить, что функция не должна при этом:

- иметь побочных эффектов.

- использовать другие классы/функции. В том числе и системные.

Когда он закончен и все тесты прошли - закомментирую

В этом и есть большой недостаток о котором и не задумываешся, пока нет CI.

Тесты должны идти после каждого коммита автоматом. Как то было, даже "полицейская" мигалка включалась при сбое, но в комнате были только программисты.

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

Бывает что всё работает без проблем в одной "среде", но при переносе в другую появляются проблемы. У меня на компе так часто бывало. На компе я меняю для чисел "немецкую запятую" на "привычную точку". И некоторые части проги это может приводить в ступор.

А для строк бывает достаточно сменить язык, чтобы "житроумный метод" накернулся.

не наш тест, который задает все время одни и те же тестовые строки, а жизнь закинет туда что-то такое, чего мы не ожидали

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