Francois Gouget : tests: Report all errors when failing to wait for a child process.

Alexandre Julliard julliard at winehq.org
Tue Mar 17 17:21:09 CDT 2020


Module: wine
Branch: master
Commit: a909baeb4e5ee8050a32832c28ff3ee79c54fbfa
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=a909baeb4e5ee8050a32832c28ff3ee79c54fbfa

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Mar 16 23:52:21 2020 +0100

tests: Report all errors when failing to wait for a child process.

Report the line number where the test failed to wait for the child so
one can identify which child process did not behave as expected.
Also wait_child_process() is meant for the general case so report
all non-crash error cases as test failures so they are accounted for.
Omit the "winetest_" prefix to match the other Wine test functions and
so the underlying winetest_wait_child_process() function can be wrapped
with the usual line-capturing macros.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=48651
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/advapi32/tests/security.c |  4 ++--
 dlls/dbgeng/tests/dbgeng.c     |  4 ++--
 dlls/gdi32/tests/font.c        |  2 +-
 dlls/kernel32/tests/actctx.c   |  4 ++--
 dlls/kernel32/tests/console.c  |  2 +-
 dlls/kernel32/tests/heap.c     |  2 +-
 dlls/kernel32/tests/pipe.c     |  6 +++---
 dlls/kernel32/tests/toolhelp.c |  2 +-
 dlls/kernel32/tests/virtual.c  |  4 ++--
 dlls/mfplat/tests/mfplat.c     |  4 ++--
 dlls/mscoree/tests/comtest.c   |  2 +-
 dlls/msvcrt/tests/data.c       |  2 +-
 dlls/msvcrt/tests/file.c       |  4 ++--
 dlls/ntdll/tests/exception.c   |  4 ++--
 dlls/ole32/tests/marshal.c     |  2 +-
 dlls/oleacc/tests/main.c       |  2 +-
 dlls/qmgr/tests/qmgr.c         |  2 +-
 dlls/rpcrt4/tests/server.c     |  6 +++---
 dlls/shell32/tests/shellpath.c |  2 +-
 dlls/shlwapi/tests/ordinal.c   |  2 +-
 dlls/sxs/tests/sxs.c           |  2 +-
 dlls/ucrtbase/tests/misc.c     |  2 +-
 dlls/urlmon/tests/misc.c       |  2 +-
 dlls/urlmon/tests/sec_mgr.c    |  2 +-
 dlls/user32/tests/class.c      |  2 +-
 dlls/user32/tests/clipboard.c  |  2 +-
 dlls/user32/tests/cursoricon.c |  2 +-
 dlls/user32/tests/msg.c        |  2 +-
 dlls/user32/tests/win.c        |  6 +++---
 include/wine/test.h            | 32 ++++++++++++++++++++++----------
 30 files changed, 64 insertions(+), 52 deletions(-)

Diff:   https://source.winehq.org/git/wine.git/?a=commitdiff;h=a909baeb4e5ee8050a32832c28ff3ee79c54fbfa



More information about the wine-cvs mailing list