Чему нас научил взаимный перенос тестов между Pii-Mask и его форком

Вот важная новость с фронта ИИ: dewil 17 минут назад Объяснить с Чему нас научил взаимный перенос тестов между Pii-Mask и его форком Средний 4 мин 399 Качество кода * Тестирование IT-систем * Open source * Кейс Автор форка SuperB1aze/pii-mask-enhanced нашел и исправил несколько пропусков обработки Word и Excel в Pii-Mask. Вместе с исправлениями он добавил тесты. Я запустил их на нашей версии и получил 13 провалов.

Так чужая работа дала нам список сценариев для разбора и помогла увидеть ошибки, которых не обнаруживали прежние проверки. Хотелось перенести эти исправления и сохранить улучшения, появившиеся у нас после разделения веток. Поэтому сравнение сделал в обе стороны: Office-тесты автора форка запустил на нашей версии, а выбранные наши тесты - на форке.

Технические детали

Ниже покажу один найденный у нас пропуск, результаты сравнения и порядок проверки. Это разбор зафиксированных версий, на момент аудита перенос исправлений еще не был выполнен. Фамилия после вложенной надписиОдин из тестов автора форка собирает минимальный искусственный OOXML-пакет с такой структурой.

Здесь показан тестовый случай вложенности, без полной разметки настоящей надписи Word. ФИО искусственное: Договор. Приложение Директор Соколова Анна Владимировна Это фрагмент XML, не самостоятельный DOCX.

В тесте он помещается в архив с остальными необходимыми частями документа. Во внешнем абзаце есть надпись со своим вложенным абзацем. На проверенной нашей версии тест test_text_after_textbox_is_masked падает: фамилия остается в выходном XML.

Отраслевые последствия

Этот тест проверяет конкретное место в структуре файла. Обычный абзац с тем же ФИО мог маскироваться правильно. Причина искать дополнительные проверки здесь - вложенность документа, которую простой текстовый пример не воспроизводит.

В Office-исправлении форка есть и другие случаи: удаленный текст рецензирования, коды полей, адреса ссылок и авторы комментариев. Для Excel добавлены печатные колонтитулы и новые комментарии. Это полезные дополнения к нашему покрытию.

Как проверяли совместимость улучшенийЯ зафиксировал коммиты и работал во временных копиях. В перенесенных тестах менял только импорты: пакет форка называется pii_mask_enhanced, наш - pii_mask. Утверждения и входные данные оставил прежними.

Событие, по словам экспертов, усилит конкуренцию в сфере ИИ.