Francois Gouget : testbot/TestLauncher: Don't use an empty test unit name on skip lines.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: c791e0234404ad15821b1f4cb751904f41cf37bb
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=c791e0234404ad15821b1f4c...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Tue Mar 16 12:14:18 2021 +0100

testbot/TestLauncher: Don't use an empty test unit name on skip lines.

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 testbot/src/TestLauncher/TestLauncher.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/testbot/src/TestLauncher/TestLauncher.c b/testbot/src/TestLauncher/TestLauncher.c
index 5ff8060..dd34a76 100644
--- a/testbot/src/TestLauncher/TestLauncher.c
+++ b/testbot/src/TestLauncher/TestLauncher.c
@@ -109,7 +109,11 @@ BOOL CALLBACK DetectCriticalErrorDialog(HWND TopWnd, LPARAM lParam)
    if (Reason)
    {
       Skips++;
-      printf("%s.c:0: Tests skipped: %s (details below)\n", Subtest, Reason);
+      /* An empty test unit name is allowed on the start, summary and done
+       * lines, but not on individual failure / skip lines.
+       */
+      printf("%s.c:0: Tests skipped: %s (details below)\n",
+             (*Subtest ? Subtest : "testlauncher"), Reason);
       printf("| %s\n", Buffer);
       EnumChildWindows(TopWnd, DumpCriticalErrorDescription, (LPARAM)TopWnd);
       /* Leave the dialog open so it appears on screenshots */