Укажите отработанное время

Отработанное время:
Продуктивное время:
Bug 6271 - Научится парсить результаты тестов   Make a simular bug
Summary: Научится парсить результаты тестов
Status: DEFERRED
Alias: None
Product: WINE-tests
Classification: Свободные проекты (Open source projects)
Component: Общее (show other bugs)
Version: не указана
Hardware: PC All
: P4 minor
Target Milestone: ---
Assignee: Виталий Перов
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 6264
  Show dependency tree / graph
 
In work:
Reported: 2010-10-22 17:33 MSD by Виталий Перов
Modified: 2015-02-28 16:49 MSK (History)
1 user (show)

See Also:
Заявки RT:
Связано с:
Дата напоминания:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Виталий Перов 2010-10-22 17:33:04 MSD 
Сейчас тесты работают следующим образом:
Запускаются тесты до и после прикладывания патча. Результаты сохраняются в файл. Затем делается diff. Если изменений нет, то мы считаем, что патч ничего не ломает.

Возникают проблемы:
1) Если после применения патча какой-то из тестов больше не показывает ошибку, то патч всё-равно не принимается, хотя он явно полезен
2) Если патч содержит тесты, то он применяется в любом случае

Для правильного решения можно брать только последнюю строчку с результатами тестов и оттуда извлекать необходимые параметры
Comment 1 Виталий Перов 2010-11-03 16:22:33 MSK 
Возникла проблема, когда патч исправляет какую-то проблему, и меньшее количество тестов падает.
Сейчас скрипт не пропускает этот патч.
Пока чтобы не тратить время на парсинг результатов тестов предлагаю временное решение:
Смотреть количество строк в файлах *.before и *.after.
Если количество строк в *.after меньше, значит патч исправляет какую-то проблему, и его надо принять.
Comment 2 Виталий Перов 2010-11-03 16:59:02 MSK 
Исправил. Проверил. Работает.
Comment 3 Виталий Перов 2010-11-10 18:49:45 MSK 
Пока успешно работает, и не мешает.
"Привильное" решение делать пока не планируется