[tools] internal/test/marker: skip basiclit.txt on ppc64

[Robert Findley (Gerrit)]

Robert Findley submitted the change

Change information

Commit message:

internal/test/marker: skip basiclit.txt on ppc64

This test was flaking in a bizarre manner on the aix-ppc64 builder. The
x/text/unicode/runenames package apparently returns a different name for
U+2211.

Since aix-ppc64 is not a first class port, skip to de-flake. However,
the failure mode is so bizarre that I added a unit test in the golang
package to try to reproduce the bug using only the x/text module. If
this new unit test also flakes, I will file a bug against aix-ppc64 and
x/text.

Fixes golang/go#65072

Change-Id: I99a6b60c7fd31b474e0b670e6f26e550de176ba8

Reviewed-on: https://go-review.googlesource.com/c/tools/+/586260

LUCI-TryBot-Result: Go LUCI <golang...@luci-project-accounts.iam.gserviceaccount.com>

Reviewed-by: Peter Weinberger <p...@google.com>

Files:

• A gopls/internal/golang/hover_test.go
• M gopls/internal/test/marker/marker_test.go
• M gopls/internal/test/marker/testdata/hover/basiclit.txt

Change size: S

Delta: 3 files changed, 28 insertions(+), 1 deletion(-)

Branch: refs/heads/master

Submit Requirements:

• Code-Review: +2 by Peter Weinberger
• TryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: merged

Gerrit-Project: tools

Gerrit-Branch: master

Gerrit-Change-Id: I99a6b60c7fd31b474e0b670e6f26e550de176ba8

Gerrit-Change-Number: 586260

Gerrit-PatchSet: 2

Gerrit-Owner: Robert Findley <rfin...@google.com>

Gerrit-Reviewer: Go LUCI <golang...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-Reviewer: Peter Weinberger <p...@google.com>

Gerrit-Reviewer: Robert Findley <rfin...@google.com>