В ходе тестирования правил линтинга разработчики выявили три серьёзные ошибки в девяти основных правилах, которые были пропущены юнит-тестами. Это подчеркивает важность комплексного подхода к тестированию кода для обеспечения его качества.
Как проводилось тестирование
В тестирования был добавлен новый скрипт, который запускал правила для анализа кода на примерах, помеченных как безопасные или содержащие уязвимости. Первые результаты показали, что из девяти протестированных правил шесть сработали корректно, а три — нет. Это говорит о низкой точности некоторых правил линтинга.
Из неработающих правил можно выделить:
- react-features/hooks-exhaustive-deps: 67% точность, 100% полнота.
- mongodb-security/no-unsafe-query: 100% точность, 50% полнота.
- vercel-ai-security/no-unsafe-output-handling: не обнаружило проблем.
Подробности о выявленных проблемах
Первая ошибка связана с неправильным срабатыванием правила hooks-exhaustive-deps, которое не учло параметр колбеков, что могло привести к потенциальным утечкам данных. Вторая проблема касалась уязвимости NoSQL инъекций через $where, что могло позволить злоумышленникам получить доступ к защищенной информации. Третье правило не обнаружило ошибок в коде с генерацией текста, хотя она была выполнена некорректно.
Почему это важно для разработчиков
Для разработчиков это сигнал: юнит-тесты не всегда могут выявить все проблемы в коде. На примере этих трёх ошибок стало очевидно, что нужно применять более гибкие методы тестирования, чтобы обнаружить уязвимости, которые могут быть причиной реальных угроз. Это особенно актуально для команд, работающих над критически важными приложениями.
В дальнейшем важно будет следить за обновлениями линтинговых правил и улучшениями алгоритмов тестирования, чтобы минимизировать риски в разработке.