bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [PATCH] Using parallel test harness


From: Giuseppe Scrivano
Subject: Re: [Bug-wget] [PATCH] Using parallel test harness
Date: Wed, 01 Oct 2014 22:04:53 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Tim Ruehsen <address@hidden> writes:

> While there might be more than one problem, at least one is the naming of the 
> tests. The name given to HTTPServer->new() should always be unique because is 
> is taken as a temporary working directory. Some tests have wrong name 
> (copy&paste I guess) and thus randomly fail when running in parallel !
>
> Here is a patch to fix this.
>
> Tim
>
> From 4a5fd2244545c2c05f40cb7a2dfba7866919d047 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Tim=20R=C3=BChsen?= <address@hidden>
> Date: Wed, 1 Oct 2014 16:42:04 +0200
> Subject: [PATCH] fixed test suite race conditions due to double usage of names
>
> ---
>  tests/ChangeLog                  | 9 +++++++++
>  tests/Test--post-file.px         | 2 +-
>  tests/Test-N-no-info.px          | 2 +-
>  tests/Test-N-smaller.px          | 2 +-
>  tests/Test-c-shorter.px          | 2 +-
>  tests/Test-proxied-https-auth.px | 2 +-
>  tests/Test-proxy-auth-basic.px   | 2 +-
>  7 files changed, 15 insertions(+), 6 deletions(-)

the patch looks correct, I am going to push it soon.

Regards,
Giuseppe



reply via email to

[Prev in Thread] Current Thread [Next in Thread]