From 8d604493076e999156fa9d36ed6594fd5b5dd49a Mon Sep 17 00:00:00 2001 From: Hristo Staykov Date: Fri, 22 May 2026 12:44:58 +0300 Subject: [PATCH] Fix multi-provenance test to exercise the provenance kill path MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The test was leaving c->user set to tempuser when ACL LOAD fired. Since tempuser is absent from the ACL file, the walk killed the client at the "current user doesn't exist" check — before reaching the provenance entry validation loop. Re-auth back to alice so the client survives the initial check and is killed for the correct reason: a provenance entry referencing a deleted user. --- tests/unit/acl.tcl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/unit/acl.tcl b/tests/unit/acl.tcl index 64f5ce711..52a1198aa 100644 --- a/tests/unit/acl.tcl +++ b/tests/unit/acl.tcl @@ -1192,6 +1192,8 @@ start_server [list overrides [list "dir" $server_path "acl-pubsub-default" "allc $rd1 read $rd1 SUBSCRIBE test2 $rd1 read + $rd1 AUTH alice alice + $rd1 read $rd1 CLIENT SETNAME multi-prov $rd1 read