Чувак я говорю о том что не надо ревьюить тесты тестировщика, так же лично я сам дописываю и юнит тесты на функции(компоненты) и свои отдельно.
В том случае когда ты сам непревзойденный ниндзя, и знаешь код продукта, и сам пишешь юнит-тесты — оно действительно может выглядеть ненужным.
Но я слушал как покойный Джерри Вайнберг рассказывал историю о ревью кода.
Краткий пересказ:
90% соглашались, 10% отказывались.
Им было предложено брать на себя стоимость ошибки. Несогласных на ревью осталось двое.
Один на протяжении года допустил ошибку, попал, что называется. Не помню уже что с ним стало потом.
Второй действительно писал идеальный код (один из всех программистов). Ему сказали что он выше похвал, но пусть ревью тоже будет, для того чтобы другие могли у него поучиться как писать код.
Это был весьма интересный пример как ревью может быть нужно даже когда оно, казалось бы, ненужно.
________
Обычно же происходит не так круто, например: юнит-тесты пишут разработчики, интеграционные пишут вместе с тестировщиками. Где-то разработчик что-то накопает, где-то тестировщик. Пусть тестировщик накопает вдвое больше разработчика, но разработчик тоже накопает.
Ревью нужно для того чтобы видеть-судить как построена иерархия компонентов-классов, + если уж есть один на всех код стиль, то можно материться как математик, но его придется соблюдать, ять.