From 66dbd29cd194a4c84f796f32827429895c2a3bba Mon Sep 17 00:00:00 2001 From: Joe Wilm Date: Sat, 19 Nov 2016 16:16:20 -0800 Subject: Add support for recording/running ref tests Ref tests use a recording of the terminal protocol and a serialization of the grid state to check that the parsing and action handling systems produce the correct result. Ref tests may be recorded by running alacritty with `--ref-test` and closing the terminal by using the window "X" button. At that point, the recording is fully written to disk, and a serialization of important state is recorded. Those files should be moved to an appropriate folder in the `tests/ref/` tree, and the `ref_test!` macro invocation should be updated accordingly. A couple of changes were necessary to make this work: * Ref tests shouldn't create a pty; the pty was refactored out of the `Term` type. * Repeatable lines/cols were needed; on startup, the terminal is resized * by default to 80x24 though that may be changed by passing `--dimensions w h`. * Calculating window size based on desired rows/columns and font metrics required making load_font callable multiple times. * Refactor types into library crate so they may be imported in an integration test. * A whole bunch of types needed symmetric serialization and deserialization. Mostly this was just adding derives, but the custom deserialization of Rgb had to change to a deserialize_with function. This initially adds one ref test as a sanity check, and more will be added in subsequent commits. This initial ref tests just starts the terminal and runs `ll`. --- font/src/ft/mod.rs | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) (limited to 'font/src/ft/mod.rs') diff --git a/font/src/ft/mod.rs b/font/src/ft/mod.rs index 55829e12..fa4325a0 100644 --- a/font/src/ft/mod.rs +++ b/font/src/ft/mod.rs @@ -29,6 +29,7 @@ pub struct Rasterizer { faces: HashMap>, library: Library, system_fonts: HashMap, + keys: HashMap, dpi_x: u32, dpi_y: u32, dpr: f32, @@ -51,6 +52,7 @@ impl Rasterizer { Rasterizer { system_fonts: get_font_families(), faces: HashMap::new(), + keys: HashMap::new(), library: library, dpi_x: dpi_x as u32, dpi_y: dpi_y as u32, @@ -77,11 +79,16 @@ impl Rasterizer { } pub fn load_font(&mut self, desc: &FontDesc, _size: Size) -> Option { - self.get_face(desc) - .map(|face| { - let key = FontKey::next(); - self.faces.insert(key, face); - key + self.keys + .get(&desc.to_owned()) + .map(|k| *k) + .or_else(|| { + self.get_face(desc) + .map(|face| { + let key = FontKey::next(); + self.faces.insert(key, face); + key + }) }) } -- cgit v1.2.3-54-g00ecf