fix(tests): correctly deinit font faces

1 files changed, 12 insertions, 9 deletions

diff --git a/src/font/Collection.zig b/src/font/Collection.zig
index 66e13c109..59f89d402 100644
--- a/src/font/Collection.zig
+++ b/src/font/Collection.zig
@@ -714,15 +714,18 @@ test "add full" {
         ) });
     }
 
-    try testing.expectError(error.CollectionFull, c.add(
-        alloc,
-        .regular,
-        .{ .loaded = try Face.init(
-            lib,
-            testFont,
-            .{ .size = .{ .points = 12 } },
-        ) },
-    ));
+    var face = try Face.init(
+        lib,
+        testFont,
+        .{ .size = .{ .points = 12 } },
+    );
+    // We have to deinit it manually since the
+    // collection doesn't do it if adding fails.
+    defer face.deinit();
+    try testing.expectError(
+        error.CollectionFull,
+        c.add(alloc, .regular, .{ .loaded = face }),
+    );
 }
 
 test "add deferred without loading options" {