Проходит ли Code Review плохой код покрытый тестами

Плохой код, даже с тестами, не должен проходить Code Review. Тесты проверяют функциональность, но не качество кода.

Основные проблемы:

• Поддержка: плохой код сложно поддерживать и расширять
• Читаемость: коллеги тратят время на понимание
• Тесты хрупкие: тесты для плохого кода часто ломаются при рефакторинге

Пример плохого кода с тестами:

// Плохо: магические числа, дублирование, нет разделения ответственности
function calc(a, b) {
  return a * 0.2 + b * 0.8;
}

// Тест проходит, но код плохой
test('calc works', () => {
  expect(calc(10, 20)).toBe(18);
});

Лучший подход: сначала исправить код, затем писать/обновлять тесты. Code Review должен фокусироваться на:

1. Качестве кода
2. Архитектуре
3. Читаемости
4. Соответствии стандартам

Тесты - важны, но не оправдывают плохой код.