[pkgsite] tests: update docs for updating screenshot tests
3 views
Skip to first unread message
Ethan Lee (Gerrit)
Dec 17, 2025, 10:39:00 PM (2 days ago) Dec 17
to goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com
Ethan Lee has uploaded the change for review![]( "Open in Gerrit")
Commit message
tests: update docs for updating screenshot tests
Change-Id: I85997dce21dae9c40ca68eb4007e7a724e519094
Change diff
diff --git a/tests/README.md b/tests/README.md
index 7f9fdfb..9b1d2bd 100644
--- a/tests/README.md
+++ b/tests/README.md
@@ -94,6 +94,13 @@
are suitable for tests with data that is consistent between environments or
static pages.
+### Updating the tests
+
+Update screen shot diffs using the following:
+
+1. Run `./tests/screentest/run.sh ci -update` to collect the updated screenshots.
+2. Copy the new screenshots from `./tests/screentest/output` into `./tests/screentest/testdata/testcases`
+
### Understanding Test Failures
If the tests fail with diffs, a file with the diffs will be created in the
Change information
Files:
- M tests/README.md
Change size: XS
Delta: 1 file changed, 7 insertions(+), 0 deletions(-)
Related details
Attention set is empty
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass
kokoro-CI-Passes
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Jonathan Amsterdam (Gerrit)
Dec 18, 2025, 8:45:13 AM (yesterday) Dec 18
to Ethan Lee, goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com
Attention needed from Ethan Lee
Jonathan Amsterdam voted and added 1 comment![]( "Open in Gerrit")
Votes added by Jonathan Amsterdam
| Code-Review | +2 |
1 comment
File tests/README.md
Line 101, Patchset 1 (Latest):
1. Run `./tests/screentest/run.sh ci -update` to collect the updated screenshots.Jonathan Amsterdam . unresolved
Mention `-run`.
Related details
Attention is currently required from:
- Ethan Lee
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass
kokoro-CI-Passes
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ethan Lee (Gerrit)
Dec 18, 2025, 3:11:19 PM (20 hours ago) Dec 18
to goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com
Attention needed from Ethan Lee
Ethan Lee uploaded new patchset![]( "Open in Gerrit")
Ethan Lee uploaded patch set #3 to this change.
Related details
Attention is currently required from:
- Ethan Lee
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass
kokoro-CI-Passes
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ethan Lee (Gerrit)
Dec 18, 2025, 3:12:00 PM (19 hours ago) Dec 18
to goph...@pubsubhelper.golang.org, Jonathan Amsterdam, golang-co...@googlegroups.com
Ethan Lee voted and added 2 comments![]( "Open in Gerrit")
Votes added by Ethan Lee
| Auto-Submit | +1 |
| Commit-Queue | +1 |
2 comments
Patchset-level comments
File tests/README.md
Line 101, Patchset 1:
1. Run `./tests/screentest/run.sh ci -update` to collect the updated screenshots.Jonathan Amsterdam . resolved
Ethan LeeMention `-run`.
Done
Related details
Attention set is empty
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass
kokoro-CI-Passes
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ethan Lee (Gerrit)
Dec 18, 2025, 3:20:39 PM (19 hours ago) Dec 18
to goph...@pubsubhelper.golang.org, Go LUCI, Jonathan Amsterdam, golang-co...@googlegroups.com
Ethan Lee voted and added 1 comment![]( "Open in Gerrit")
Votes added by Ethan Lee
| Auto-Submit | +1 |
| Commit-Queue | +1 |
Ethan Lee . resolved
kokoro rerun
Related details
Attention set is empty
Submit Requirements:
Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass
kokoro-CI-Passes
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages