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

[Francois Gouget]

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 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 7b58c9ad1..b92d09bf2 100644
--- a/testbot/src/TestLauncher/TestLauncher.c
+++ b/testbot/src/TestLauncher/TestLauncher.c
@@ -87,7 +87,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 */
-- 
2.20.1