Ревью 🔍

Несколько ключевых моментов, которые помогут в процессе проверки:

⏱️ Уровень сложности ревью: 3 🔵🔵🔵⚪⚪

🧪 Запрос содержит тесты

🔒 Проблем с безопасностью не выявлено

⚡ Рекомендовано к рассмотрению

Частые ререндеры UI

Вызов setResults внутри цикла вызывает перерисовку UI на каждом шаге теста. Для тестового приложения это допустимо, но может замедлить отладку при большом количестве шагов. Рекомендуется накапливать результаты и обновлять стейт один раз в конце или использовать batch updates.

for (const spec of mainBatch) {
  const r = await callSafely(spec);
  mainResults.push(r);
  log(`[TEST-APP] ${r.ok ? 'OK ' : 'ERR'} ${r.name.padEnd(34)} ${String(r.ms).padStart(4)}ms ${r.ok ? `→ ${r.data ?? ''}` : `→ ${r.err}`}`);
  if (cancelled) return;
  setResults([...mainResults]);
}

Дублирование логики инициализации TurboModule

Логика получения writeTestLog через TurboModuleRegistry продублирована в App.tsx и AutoTestRunner.ts. Рекомендуется вынести её в отдельную утилиту (например, utils/logger.ts) для соблюдения принципа DRY и упрощения поддержки.

try {
  const { TurboModuleRegistry } = require('react-native');
  const tm = TurboModuleRegistry.get('NativeSidModule');
  writeTestLog = tm?.writeTestLog ?? null;
} catch {
  // ignore
}

Сбер ID

Популярные репозитории