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.