A bad event record says what is wrong with it

`bad event syntax` said nothing of which: now it says a report record
(I D i d) is not written back, a range ends before it starts, or what a
record looks like when the line is not one.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 18 insertions, 0 deletions

diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index c0695c76..5f26a181 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -2061,6 +2061,24 @@ test "a malformed address is bad address syntax, on a terminal as on a text" {
     }
 }
 
+test "a bad event record says what is wrong with it" {
+    const p = try withFile(testing.allocator, "one\n");
+    defer p.deinit();
+    const event = Node.of(serialOf(p), .event);
+    const h = call(p, .{ .tag = 1, .op = .open, .node = event, .omode = 2 }).reply.handle;
+    for ([_][2][]const u8{
+        .{ "MI0 1 0 1 x\n", "only a look or exec" },
+        .{ "MX3 1 0 0 \n", "ends before it starts" },
+        .{ "MX9 9 0 0 \n", "range past end" },
+        .{ "garbage\n", "a record is" },
+    }) |c| {
+        const refused = call(p, .{ .tag = 2, .op = .write, .node = event, .handle = h, .data = c[0] });
+        try testing.expectEqual(E.INVAL, refused.errno());
+        try testing.expect(std.mem.indexOf(u8, refused.reply.ename, c[1]) != null);
+    }
+    _ = call(p, .{ .tag = 3, .op = .release, .node = event, .handle = h });
+}
+
 test "size is monotonic: growing is never refused, and a size once taken is taken again" {
     const p = try Pardes.init(testing.allocator, .{ .tty_only = true, .cols = 160, .rows = 60 });
     defer p.deinit();