tree ff1044f89477b799885eea9a47f84c8547ba0091
parent 67e2d13fc2f3cec1d5051788fad4e25aa7f7854e
author Dmitri Shuralyov <dmitshur@golang.org> 1570554507 -0400
committer Dmitri Shuralyov <dmitshur@golang.org> 1570555485 +0000

dashboard: catch potential pitfall in TestBuilderConfig

Previously, it was possible to accidentally try to specify the
branch name for the Go repository by writing something like:

	{b("nacl-386", "go@go1.13"), none},

That looks like it would test the right thing,
but in reality it would be equivalent to:

	{b("nacl-386@master", "go"), none},

Because the branch of the builder always overwrites the branch
of the "go" repository.

This is not intuitive and easy to miss when reviewing or reading
the code.

Add a check that detects and panics on this b() API pitfall,
so that it is not accidentally introduced in the future, and
so that code readers don't need to spend time verifying that
none of the existing test cases have fallen victim to it.

Updates golang/go#34738

Change-Id: Id2986e88c0e3585eaa7b45f33de97e1902847305
Reviewed-on: https://go-review.googlesource.com/c/build/+/199817
Reviewed-by: Bryan C. Mills <bcmills@google.com>