Legacy-код и правило бойскаута

Legacy-код — это просто код без тестов, и это определение говорит, что делать. Рабочее определение Майкла Фезерса снимает с термина эмоции: legacy — это не «старый», не «чужой» и не «на языке, который мне не нравится». Это код, который ты не можешь менять с уверенностью, потому что ничто не подскажет, когда ты его сломал. Свежий код, написанный тобой сегодня утром без тестов, уже legacy по этой мерке.

Смысл определения — операционный. Если проблема в том, что «нет страховочной сетки», то решение не «будь осторожнее» и не «читай внимательнее» — это сначала подведи под него сетку. Всё остальное в этом уроке следует из этого. Ты не рефакторишь legacy-код, а потом его тестируешь; ты его тестируешь, и тогда можешь рефакторить.

Прежде чем тестировать, тебе нужен шов — место, где можно изменить поведение, не редактируя код в самой этой точке. Причина, по которой legacy-код тяжело тестировать, обычно в том, что он сам подключает своих коллабораторов: создаёт через new клиента БД, вызывает статический синглтон, читает Date.now() напрямую. Шов — это точка, где можно подставить фейк. Часто самый дешёвый шов — внедрение зависимости на границе, которую тебе и так нужно тронуть.

// Шва нет: зависимость вварена, поэтому ветку нельзя протестировать изолированно.
class OrderService {
  price(order: Order): number {
    const now = new Date();                 // приварено к системным часам
    const fx = new FxClient().rateFor(order.currency); // приварено к сети
    return applyRules(order, now, fx);
  }
}

// Шов, введённый параметризацией границы — поведение сегодня идентично.
class OrderService {
  constructor(private clock: () => Date, private fx: FxRates) {}
  price(order: Order): number {
    return applyRules(order, this.clock(), this.fx.rateFor(order.currency));
  }
}

Ты не изменил ни одного правила. Ты сделал код подставляемым, а это предусловие для закрепления его поведения.

Закрепи текущее поведение характеризующими тестами — не «правильное» поведение, а текущее. Характеризующий тест документирует, что код реально делает сегодня, баги включительно. Ты не утверждаешь, какой цена должна быть; ты запускаешь код, смотришь, что он возвращает, и замораживаешь это как ожидаемое значение. Задача теста — не корректность, а проволока-растяжка, которая срабатывает в момент, когда твоя правка меняет любой наблюдаемый вывод.

// Ты не решал, что 47.5 «правильно» — код это выдал. Ты это закрепил.
test("characterize: GBP order at 2021 rates", () => {
  const svc = new OrderService(() => new Date("2021-06-01"), fakeRates({ GBP: 1.18 }));
  expect(svc.price(gbpOrder)).toBe(47.5); // что бы он ни возвращал, зафиксировано
});

Если характеризующий тест вскрывает то, что выглядит багом, ты не чинишь его сейчас. Ты закрепляешь багованный вывод, отгружаешь своё настоящее изменение и чинишь баг отдельным, видимым изменением с собственным тестом. Смешивать «внести моё изменение» и «починить найденный баг» — вот как однострочная правка превращается в нечитаемый на ревью диф.

Теперь внеси изменение, а потом прибирайся по-бойскаутски только внутри радиуса поражения, который ты и так тронул. Когда сетка на месте, твоя настоящая правка безопасна: если ты сломаешь закреплённое поведение, тест станет красным. Правило бойскаута — всегда оставляй код чуть чище, чем нашёл — это инкрементальная стратегия, которая, повторённая на сотнях коммитов, стаскивает кодовую базу с дорогого конца кривой изменения без того, чтобы кто-то когда-либо назначал «спринт уборки».

Senior-уточнение — это слово оппортунистический. Ты улучшаешь код, который и так уже читаешь и уже покрываешь тестами, — переименуй вводящую в заблуждение переменную в редактируемой функции, вынеси продублированную ветку, которую ты только что был вынужден понять. Ты не открываешь соседние файлы, чтобы «прибрать заодно». Уборка должна ехать внутри существующего радиуса поражения изменения. В момент, когда она выходит за этот радиус, это новая работа, которой нужно своё обоснование, свои тесты и своё ревью, — а вкручивание её в этот диф лишь делает оба изменения труднее для ревью и рискованнее для отгрузки.