From 46be87f2ca40920de1c0af66cef8437f9c6619b3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Adri=C3=A0=20Arrufat?= Date: Sun, 12 Jul 2026 22:13:49 +0200 Subject: [PATCH] test(agent): cover resolveStream precedence and stream round-trip Verify the stream default is on, that a remembered value wins, and that the field parses from (and is absent-safe in) .lp-agent.zon. --- src/agent/settings.zig | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/agent/settings.zig b/src/agent/settings.zig index 0e59b54ed..1e17ffb57 100644 --- a/src/agent/settings.zig +++ b/src/agent/settings.zig @@ -338,4 +338,19 @@ test "parseRemembered: valid file round-trips" { defer std.zon.parse.free(testing.allocator, remembered); try testing.expect(remembered.provider == null); try testing.expectString("some-model", remembered.model); + // Absent `stream` is null so pre-streaming files still fall back to the default. + try testing.expect(remembered.stream == null); +} + +test "parseRemembered: stream field round-trips" { + const remembered = parseRemembered(testing.allocator, ".{ .model = \"m\", .stream = false }").?; + defer std.zon.parse.free(testing.allocator, remembered); + try testing.expect(remembered.stream == false); +} + +test "resolveStream: default on, remembered wins" { + try testing.expect(resolveStream(null)); + try testing.expect(resolveStream(.{ .model = "m", .stream = null })); + try testing.expect(resolveStream(.{ .model = "m", .stream = true })); + try testing.expect(!resolveStream(.{ .model = "m", .stream = false })); }