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

Отработанное время:
Продуктивное время:
Bug 6742 - Вывод теста может меняться без изменения кода wine   Make a simular bug
Summary: Вывод теста может меняться без изменения кода wine
Status: CLOSED FIXED
Alias: None
Product: WINE-tests
Classification: Свободные проекты (Open source projects)
Component: Общее (show other bugs)
Version: не указана
Hardware: PC All
: P4 major
Target Milestone: ---
Assignee: Виталий Перов
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 6264
  Show dependency treegraph
 
In work:
Reported: 2010-12-30 01:30 MSK by Александр Морозов
Modified: 2011-07-12 18:46 MSK (History)
2 users (show)

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


Attachments
process.tst.before из письма, сообщающего о том, что патч не принят (1.15 KB, text/plain)
2010-12-30 01:33 MSK, Александр Морозов
Details
process.tst.after из письма, сообщающего о том, что патч не принят (1.15 KB, text/plain)
2010-12-30 01:33 MSK, Александр Морозов
Details

Note You need to log in before you can comment on or make changes to this bug.
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.