code review 154950043: net: disable TestDualStackUDPListener in short mode

net: disable TestDualStackUDPListener in short mode

This test is flaky, just like TestDualStackTCPListener.
That one was disabled. Disable this one too.

Update issue 5001

[rsc, 2014-10-06 16:06:27]

Hello rlh (cc: golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://code.google.com/p/go/

[bradfitz, 2014-10-06 16:56:44]

LGTM


On Mon, Oct 6, 2014 at 9:06 AM, <rsc@golang.org> wrote:

> Reviewers: rlh,
>
> Message:
> Hello rlh (cc: golang-codereviews@googlegroups.com),
>
> I'd like you to review this change to
> https://code.google.com/p/go/
>
>
> Description:
> net: disable TestDualStackUDPListener in short mode
>
> This test is flaky, just like TestDualStackTCPListener.
> That one was disabled. Disable this one too.
>
> Update issue 5001
>
> Please review this at https://codereview.appspot.com/154950043/
>
> Affected files (+3, -0 lines):
>   M src/net/unicast_posix_test.go
>
>
> Index: src/net/unicast_posix_test.go
> ===================================================================
> --- a/src/net/unicast_posix_test.go
> +++ b/src/net/unicast_posix_test.go
> @@ -204,6 +204,9 @@
>  // to a test listener with various address families, differnet
>  // listening address and same port.
>  func TestDualStackUDPListener(t *testing.T) {
> +       if testing.Short() {
> +               t.Skip("skipping in -short mode, see issue 5001")
> +       }
>         switch runtime.GOOS {
>         case "plan9":
>                 t.Skipf("skipping test on %q", runtime.GOOS)
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "golang-codereviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to golang-codereviews+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
>

[rsc, 2014-10-06 17:09:21]

*** Submitted as https://code.google.com/p/go/source/detail?r=c83fd77ddd77 ***

net: disable TestDualStackUDPListener in short mode

This test is flaky, just like TestDualStackTCPListener.
That one was disabled. Disable this one too.

Update issue 5001

LGTM=bradfitz
R=rlh, bradfitz
CC=golang-codereviews
https://codereview.appspot.com/154950043

[rlh, 2014-10-06 18:27:58]

thanks,
LGTM