tests/screentest: remove a test cases
Commented out the multi-GOOS 540x1080 test.
I tried and failed to increase its reliability. Running locally against
staging, it still fails about 20% of the time, but always succeeds after
a retry. Running on Cloud Build against staging, the failure rate seems
higher, and retries don't help.
More detail in a comment.
Change-Id: I31e334d03c6bdff6733ed675a97677a723309ce0
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/634555
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
Reviewed-by: Robert Findley <rfindley@google.com>
diff --git a/tests/screentest/testcases.txt b/tests/screentest/testcases.txt
index e4fbae1..0da1634 100644
--- a/tests/screentest/testcases.txt
+++ b/tests/screentest/testcases.txt
@@ -131,7 +131,12 @@
eval {{$hideLinksHeading}}
sleep 1s
capture viewport
-capture viewport 540x1080
+# This fails around 40% of the time with an 11-pixel difference. That is due to
+# a slightly longer scrollbar. Changing "viewport" to "fullscreen" reveals that
+# the real difference is the page length: there is extra blank space at the
+# bottom of varying size.
+# I have not been able to fix that or understand why it happens.
+# capture viewport 540x1080
test retracted package
path /k8s.io/client-go@v1.5.2
