Issue(s) when running winetest on Windows

[Paul Vriens]

Paul Vriens wrote:
> Alexandre Julliard wrote:
>> Paul Vriens <paul.vriens.wine at gmail.com> writes:
>>
>>> Anyone?
>>>
>>> This is getting pretty serious now as several test reports that are
>>> submitted will never be processed. I could of course try and fix this
>>> in the parser but I do want to know what the issue is. The strange
>>> thing is that it only happens in the msvcrt test (maybe because it's
>>> file related).
>>
>> The only way is for the parent to wait for the child processes to
>> terminate, by waiting on the process handles.
>>
> So what do you call the parent in this case? The winetest executable or 
> the test itself? The test itself is probably the only one who knows 
> about the processes it spawned/started.
> 
> Do you know the reason why msvcrt/file.c uses _spawnvp, where the rest 
> of the test use CreateProcess (I've read some where that spawnvp is a 
> wrapper around CreateProcess anyway)?
> 
> Cheers,
> 
> Paul.
> 
Something like this then:

diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c
index 46ccad6..98c8ada 100644
--- a/dlls/msvcrt/tests/file.c
+++ b/dlls/msvcrt/tests/file.c
@@ -34,6 +34,8 @@
  #include <process.h>
  #include <errno.h>

+static HANDLE prochandles[2];
+
  static void test_fdopen( void )
  {
      static const char buffer[] = {0,1,2,3,4,5,6,7,8,9};
@@ -922,7 +924,7 @@ static void test_pipes(const char* selfname)
      arg_v[3] = str_fdr; sprintf(str_fdr, "%d", pipes[0]);
      arg_v[4] = str_fdw; sprintf(str_fdw, "%d", pipes[1]);
      arg_v[5] = NULL;
-    _spawnvp(_P_NOWAIT, selfname, arg_v);
+    prochandles[0] = (HANDLE)_spawnvp(_P_NOWAIT, selfname, arg_v);
      ok(close(pipes[1]) == 0, "unable to close %d: %d\n", pipes[1], errno);

      r=read(pipes[0], buf, sizeof(buf)-1);
@@ -947,7 +949,7 @@ static void test_pipes(const char* selfname)
      arg_v[3] = str_fdr; sprintf(str_fdr, "%d", pipes[0]);
      arg_v[4] = str_fdw; sprintf(str_fdw, "%d", pipes[1]);
      arg_v[5] = NULL;
-    _spawnvp(_P_NOWAIT, selfname, arg_v);
+    prochandles[1] = (HANDLE)_spawnvp(_P_NOWAIT, selfname, arg_v);
      ok(close(pipes[1]) == 0, "unable to close %d: %d\n", pipes[1], errno);
      file=fdopen(pipes[0], "r");

@@ -1004,4 +1006,6 @@ START_TEST(file)
      test_get_osfhandle();
      test_setmaxstdio();
      test_pipes(arg_v[0]);
+
+    WaitForMultipleObjects(2, prochandles, TRUE, 5000);
  }

It's enough for the test on my box to have the lines in the correct order.

Cheers,

Paul.