Skip to content

Commit 9f653f9

Browse files
test(prompt): update golden theme count
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent f1589ee commit 9f653f9

2 files changed

Lines changed: 6 additions & 5 deletions

File tree

src/prompt/golden_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ const regenerateFixtureCommand = "go run . config export data --sanitize --theme
5353

5454
// fixturePath is the single sanitized --data fixture every bundled theme
5555
// renders against - see the package comment above for why one shared fixture
56-
// replaces what used to be 124 per-theme files.
56+
// replaces what used to be 125 per-theme files.
5757
var fixturePath = filepath.Join("testdata", "fixtures", "prompt.data.json")
5858

5959
// sanitizedPWD is the PWD value cli/sanitize.go bakes into fixturePath (see
@@ -69,7 +69,7 @@ var update = flag.Bool("update", false, "regenerate the golden manifest and repr
6969
// fixture. code.yml runs `go test ./...` on ubuntu, macOS and windows, so all
7070
// three matter, not just this machine.
7171
func TestMain(m *testing.M) {
72-
// 65 of 124 themes render a time segment through dateInZone(fmt, date,
72+
// 65 of 125 themes render a time segment through dateInZone(fmt, date,
7373
// "Local") (template/date.go). The segment's own state is baked into the
7474
// fixture at record time (restoreData never re-probes it), but formatting
7575
// that recorded instant still happens at render time in the process's
@@ -216,7 +216,7 @@ func renderTheme(t *testing.T, themePath, fixturePath string) []byte {
216216
// template.Init only rebuilds the package-level template.Cache singleton
217217
// when it is nil (template/init.go) - by design, for the serve daemon,
218218
// which stays alive across many renders and calls ResetCache itself
219-
// between them (cli/serve.go). This harness renders 124 different themes
219+
// between them (cli/serve.go). This harness renders 125 different themes
220220
// (each with its own env identity) in one process, and other tests in
221221
// this package also populate Cache directly, so without forcing a rebuild
222222
// here every render after the first would silently reuse whichever env
@@ -314,13 +314,13 @@ func contextWindow(b []byte, offset int) []byte {
314314
}
315315

316316
// TestGoldenThemes renders every bundled theme from its committed fixture and
317-
// checks it two ways: a sha256 line in the manifest (all 124 themes) and,
317+
// checks it two ways: a sha256 line in the manifest (all 125 themes) and,
318318
// additionally for representativeThemes, a byte-for-byte comparison against a
319319
// committed golden file. Run with -update to regenerate both instead of
320320
// comparing.
321321
func TestGoldenThemes(t *testing.T) {
322322
themePaths := themeFiles(t)
323-
require.Lenf(t, themePaths, 124, "expected 124 bundled themes (122 .omp.json + 2 .omp.yaml); "+
323+
require.Lenf(t, themePaths, 125, "expected 125 bundled themes (123 .omp.json + 2 .omp.yaml); "+
324324
"if this changed intentionally, update the fixture/golden set for the new/removed theme(s)")
325325

326326
type manifestEntry struct {

src/prompt/testdata/goldens/manifest.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ if_tea 1485171bfd8dc7ed6c8741682d96b434075286345a7e493ece046180f276ba6d
5151
illusi0n 136fcbf744b7423de9326452b336b9c85e7edc2888821a166939019f41d24c5f
5252
iterm2 39e7436753c8cdae446cd25670bd3f33e11a807720d1f01047f3e724debb679d
5353
jandedobbeleer 28dc389e8d6265370039f2c749559e181a659d35272aebfc8a1120dc1ed4443d
54+
jandedobbeleer-accessible d6f8430dbe8df92601134d90ebd6a5838dfe0bb3d91e13def35d7d0e86a530b1
5455
jblab_2021 72923b996d17e9502f0cc5d182eebc64948eac132fdbf767e6f520ea9a415556
5556
jonnychipz 4a5ef8a7154c7c3c237e387d8d03858ff59b942b937c232fabd23ec419c9efc0
5657
json f8a1ebf6f791cb08a142718cf2ad34e338f562584d59655c903064458244efa3

0 commit comments

Comments
 (0)