openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-10-25 18:28:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

Cache the test pet spritesheet encoding (#39028)

Browse Source 
## Why

Pet tests reuse the same blank WebP spritesheet fixture, so encoding a new image
for every temporary path does unnecessary repeated work.

## What changed

Encode the test spritesheet once with `LazyLock`, then write the cached WebP bytes
to each requested fixture path.

GitOrigin-RevId: 26d75bb1bac448717ca63aeb82eb4e500c86fad8
This commit is contained in:
Charlie Marsh
2026-08-17 16:58:24 +00:00
committed by copyberry
parent 171ae66d74
commit 10fbb61838
1 changed files with 16 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
codex-rs/tui/src/pets/catalog.rs
+16 -2
View File
@@ -1,5 +1,12 @@
//! Built-in pet catalog ported from the Codex App avatar catalog.
#[cfg(test)]
use std::fs;
#[cfg(test)]
use std::io::Cursor;
#[cfg(test)]
use std::sync::LazyLock;
pub(super) const DEFAULT_FRAME_WIDTH: u32 = 192;
pub(super) const DEFAULT_FRAME_HEIGHT: u32 = 208;
pub(super) const DEFAULT_FRAME_COLUMNS: u32 = 8;
@@ -72,6 +79,13 @@ pub(super) fn builtin_pet(id: &str) -> Option<BuiltinPet> {
#[cfg(test)]
pub(super) fn write_test_spritesheet(path: &std::path::Path) {
let image = image::RgbaImage::new(SPRITESHEET_WIDTH, SPRITESHEET_HEIGHT);
image.save(path).unwrap();
static TEST_SPRITESHEET: LazyLock<Vec<u8>> = LazyLock::new(|| {
let image = image::RgbaImage::new(SPRITESHEET_WIDTH, SPRITESHEET_HEIGHT);
let mut bytes = Vec::new();
image::DynamicImage::ImageRgba8(image)
.write_to(&mut Cursor::new(&mut bytes), image::ImageFormat::WebP)
.unwrap();
bytes
});
fs::write(path, TEST_SPRITESHEET.as_slice()).unwrap();
}