Thread overview
[Issue 18103] test17868 failing often but not every time
Jun 19, 2020
Walter Bright
December 19, 2017
https://issues.dlang.org/show_bug.cgi?id=18103

Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |code@dawg.eu,
                   |                            |schveiguy@yahoo.com

--- Comment #1 from Steven Schveighoffer <schveiguy@yahoo.com> ---
Looks like its a race condition.

The failure line says this:

diff: generated/runnable/test17868.d.out.2: No such file or directory

Note that the postscript does something with the output, copies it to a ".out.2" and then diffs it against the expected output.

But I see no "release" or "debug" in that path. The appropriate line is this:

diff --strip-trailing-cr runnable/extra-files/test17868.d.out ${RESULTS_DIR}/runnable/test17868.d.out.2

So clearly, RESULTS_DIR just contains "generated". That's not enough to avoid races. I actually see a lot of cases in the postscript files that are subject to possible races.

CCing Martin, as he seems to be the one that set up these tests.

--
June 19, 2020
https://issues.dlang.org/show_bug.cgi?id=18103

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |TestSuite
                 CC|                            |bugzilla@digitalmars.com

--
June 19, 2020
https://issues.dlang.org/show_bug.cgi?id=18103

moonlightsentinel@disroot.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |moonlightsentinel@disroot.o
                   |                            |rg
         Resolution|---                         |FIXED

--- Comment #2 from moonlightsentinel@disroot.org ---
That test doesn't use a POST_SCRIPT anymore.

--