tree 90e087b7d1729379b8bede0cf3c337fb8ccae4b3
parent 7b78c2042368d5c56ee9dbd92ab5fa988c763944
author Dmitri Shuralyov <dmitshur@golang.org> 1544492999 -0500
committer Dmitri Shuralyov <dmitshur@golang.org> 1544804845 +0000

maintner/godata: skip TestGerritHashtags when TEST_GERRIT_AUTH unset

TestGerritHashtags is documented to be off by default unless
environment variable TEST_GERRIT_AUTH is set to "user:token",
or we're running in the prod project.

However, the previous code never skipped TestGerritHashtags,
even when TEST_GERRIT_AUTH wasn't set, because strings.SplitN
returns a slice with length 1 when the input string is empty.
This change fixes that.

Updates golang/go#28318

Change-Id: Ib2f113dbc3ccfd933f9944fe8efa105ff3ba5057
Reviewed-on: https://go-review.googlesource.com/c/153442
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>