Я присоединился к команде, которая работает над продуктом. Этот продукт был вокруг в течение ~5 лет или около того, и использует ASP.NET WebForms. Его оригинальная архитектура со временем исчезла, и вещи стали относительно неорганизованными на протяжении всего решения. Это ни в коем случае не ужасно, но определенно может использовать некоторую работу; вы все знаете, что я имею в виду.

Я выполнял некоторые рефакторинги с момента прихода в команду проекта около 6 месяцев назад. Некоторые из этих рефакторингов просты, метод извлечения, метод вытягивания и т. д. Некоторые из рефакторингов являются более структурными. Последние изменения заставляют меня нервничать, поскольку нет полного набора модульных тестов для сопровождения каждого компонента.

Вся команда готова внести структурные изменения через рефакторинг, но наш руководитель проекта выразил некоторые опасения, что у нас нет адекватных тестов для рефакторинга с уверенностью, что мы не вводим ошибки регрессии в систему. Он хотел бы, чтобы мы сначала написали больше тестов (против существующей архитектуры), а затем выполнили рефакторинг. Мой аргумент заключается в том, что структура классов системы слишком тесно связана, чтобы писать адекватные тесты, и что использование более тестового подхода при выполнении наших рефакторингов может быть лучше. Я имею в виду не написание тестов для существующих компонентов, а написание тестов для конкретных функциональных требований, а затем рефакторинг существующего кода для удовлетворения этих требований. Это позволит нам писать тесты, которые, вероятно, будут иметь больше долговечности в системе, а не писать кучу тестов 'throw away'.

Есть ли у кого-нибудь опыт относительно того, каков наилучший курс действий? У меня есть свои собственные мысли, но я хотел бы услышать некоторый вклад от сообщества.