[erlang-patches] eunit: fix race conditions
Henrik Nord
henrik@REDACTED
Mon May 2 16:29:25 CEST 2011
On 04/18/2011 11:06 AM, Klas Johansson wrote:
> Hi,
>
> When playing around with eunit's surefire XML reports I
> noticed that the files are sometimes not generated and
> I decided to have a look:
>
> Wait until all eunit listeners terminate
>
> Ensure that all eunit listeners (like
> eunit_surefire) get the chance to finish their work
> before terminating. Prior to this fix, the
> eunit_surefire XML was sometimes not generated.
>
> While addressing that problem I also noticed the
> following:
>
> Don't let eunit_surefire report back to eunit when stopping
>
> When eunit is terminating, a stop message is sent
> to all listeners and eunit then waits for *one*
> result message but previously both eunit_tty and
> eunit_surefire sent a response on error. Don't
> send a result message from eunit_surefire; let
> eunit_tty take care of all result reporting, both
> positive and negative to avoid race conditions and
> inconsistencies.
>
> Tested manually by running eunit with surefire report
> generation over and over again (with a timer:sleep/1
> inserted to make sure that XML file generation takes
> time to complete).
>
>
> Code here:
>
> git fetch git://github.com/klajo/otp.git eunit-race-cond
>
>
>
> Cheers,
> Klas
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches
Hi
Thank you for the contribution!
you branch is now cooking in 'pu'
--
/Henrik Nord Erlang/OTP
More information about the erlang-patches
mailing list