Bug 6742

Summary: Вывод теста может меняться без изменения кода wine
Product: [Свободные проекты (Open source projects)] WINE-tests Reporter: Александр Морозов <amorozov>
Component: ОбщееAssignee: Виталий Перов <vitperov>
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P4 CC: baraka, lav
Version: не указана   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on:    
Bug Blocks: 6264    
Attachments: process.tst.before из письма, сообщающего о том, что патч не принят
process.tst.after из письма, сообщающего о том, что патч не принят

Description Александр Морозов 2010-12-30 01:30:33 MSK
Сравнение вывода до прикладывания патча и после с помощью diff может показывать различие даже если ни один тест на самом деле не был затронут. Пример: тесты для process.c в kernel32.
Comment 1 Александр Морозов 2010-12-30 01:33:02 MSK
Created attachment 2084 [details]
process.tst.before из письма, сообщающего о том, что патч не принят
Comment 2 Александр Морозов 2010-12-30 01:33:54 MSK
Created attachment 2085 [details]
process.tst.after из письма, сообщающего о том, что патч не принят
Comment 3 Виталий Перов 2010-12-30 02:58:10 MSK
Хорошо, я понял. Разница в именах временных файлов.

Была идея сравнивать файлы по количеству строк, но тогда может возникать проблема если патч добавляет одну новую проблему и одну устраняет.

Ещё есть другое решение: предварительно усекать длину каждой строки.
В принципе, полезная информация, по которой можно сравнивать, это только "process.c:857: Test failed:", всё остальное можно вырезать.
Comment 4 Виталий Перов 2011-01-10 17:41:52 MSK
> Ещё есть другое решение: предварительно усекать длину каждой строки.
> В принципе, полезная информация, по которой можно сравнивать, это только
> "process.c:857: Test failed:", всё остальное можно вырезать.

Добавил патч, делающий усечение каждой строки до 60 символов перед запуском diff.