diff options
| author | Jonathan Amsterdam <jba@google.com> | 2025-02-05 12:14:00 -0500 |
|---|---|---|
| committer | Jonathan Amsterdam <jba@google.com> | 2025-02-05 09:46:08 -0800 |
| commit | a4ef958e79ee7aaaed7bba65c322da478d1ec487 (patch) | |
| tree | bce1112477fe6f616d50a8e96b965d9772d9a2ba | |
| parent | 50d4697cc82f2eedc49fc659871d1e81ca4c6164 (diff) | |
| download | go-x-pkgsite-a4ef958e79ee7aaaed7bba65c322da478d1ec487.tar.xz | |
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 <noreply+kokoro@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Robert Findley <rfindley@google.com>
| -rw-r--r-- | tests/screentest/testcases.txt | 3 |
1 files changed, 2 insertions, 1 deletions
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 |