1 files changed, 17 insertions, 0 deletions
diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index d7d50cc6..18068ab1 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -770,6 +770,23 @@ test "a second lock is refused until the holder unlocks or closes, and binds nob
     for (p.fs.opens) |o| try testing.expect(o.node == 0);
 }
 
+test "Edit through a pane's ctl renames every foo in one undo step, and a failed one changes nothing" {
+    const p = try withFile(testing.allocator, "foo x foo y foo\n");
+    defer p.deinit();
+    const serial = serialOf(p);
+    const ctl_node = Node.of(serial, .ctl);
+    const body = Node.of(serial, .body);
+    try testing.expectEqual(Status.ok, wr(p, ctl_node, "Edit ,x/foo/c/foobar/\n").reply.status);
+    try testing.expectEqualStrings("foobar x foobar y foobar\n", rd(p, body, 0, 64).bytes);
+    const failed = wr(p, ctl_node, "Edit ,s/nothing/x/\n");
+    try testing.expectEqual(E.IO, failed.errno());
+    try testing.expectStringStartsWith(failed.reply.ename, "Edit: no substitution");
+    try testing.expectEqualStrings("foobar x foobar y foobar\n", rd(p, body, 0, 64).bytes);
+    // The whole rename is one undo step.
+    try testing.expectEqual(Status.ok, wr(p, ctl_node, "Undo").reply.status);
+    try testing.expectEqualStrings("foo x foo y foo\n", rd(p, body, 0, 64).bytes);
+}
+
 test "dirty goes back to 0 when the text is the saved text again" {
     const p = try withFile(testing.allocator, "one\n");
     defer p.deinit();