From a4ef958e79ee7aaaed7bba65c322da478d1ec487 Mon Sep 17 00:00:00 2001 From: Jonathan Amsterdam Date: Wed, 5 Feb 2025 12:14:00 -0500 Subject: tests/screentest: omit mobile expanded README test Comment out the test for expanded README on small screens. It fails about half the time in CI, making it annoying to deploy and submit CLs. Maybe more effort could deflake it, but I'm not optimistic. I spent a while throwing everything I could at it and it's still misbehaving. It would require someone more familiar than I with the intricacies of HTML page loading and headless chrome to make progress. It is only for mobile, not our main concern, and it is only one test. Change-Id: Icd9c2bd3f3c99ee0efaf8a617f9b2b31a0c8dde6 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/646855 kokoro-CI: kokoro LUCI-TryBot-Result: Go LUCI Reviewed-by: Robert Findley --- tests/screentest/testcases.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/screentest/testcases.txt b/tests/screentest/testcases.txt index 0da16341..62805efc 100644 --- a/tests/screentest/testcases.txt +++ b/tests/screentest/testcases.txt @@ -47,7 +47,8 @@ eval {{$scrollTop}} eval {{$hideElements}} sleep 3s capture viewport -capture viewport 540x1080 +# This fails about half the time in CI. +# capture viewport 540x1080 test x/net directories expanded path /golang.org/x/net@v0.31.0 -- cgit v1.3