This test was added in CL 96495 for issue #37791.

@rasky, is it sufficient for the test to run on GOOS=windows only, and skip for other GOOSes?

/cc @alexbrainman

Ok, I think I have a pretty good sense of what's going on.

The fix implemented in in CL 96495 isn't having an effect on the *.bat files, and the test is correctly catching a problem.

The reason for this is related to how release pushes the source to a buildlet:

// Push source to buildlet.
b.logf("Pushing source to buildlet.")
...
tar := "https://go.googlesource.com/go/+archive/" + *rev + ".tar.gz"
if err := client.PutTarFromURL(ctx, tar, goDir); err != nil {
	b.logf("failed to put tarball %q into dir %q: %v", tar, goDir, err)
	return err
}

This came up as a question in #37791:

Will the Gerrit *.tar.gz serving endpoint respect this gitattributes file? I'm not sure.

What kind of test do we want? How can we verify Gerrit *.tar.gz?

The answer is by downloading a .tar.gz from a URL as shown above. For example:

https://go.googlesource.com/go/+archive/bdf76ca045c989bcff9a0111819559d632a4a39a.tar.gz

The all.bat file in it has Unix LF endings and the winbatch.go test reports a problem.

Another place to place a test for this could be in release, but it doesn't seem to be necessary since the current test works.

Sent CL 236437 which fixes this issue.

Change https://golang.org/cl/236437 mentions this issue: all: treat all files as binary, but check in .bat with CRLF