From 0faa0cb532c4c1d1c4eb5f4886f767002cdc5e5c Mon Sep 17 00:00:00 2001 From: Ruben Fiszel Date: Tue, 4 Aug 2026 07:47:57 +0000 Subject: [PATCH] test: state what the guard pins, not what it replaced --- backend/tests/preserve_on_behalf_of.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/backend/tests/preserve_on_behalf_of.rs b/backend/tests/preserve_on_behalf_of.rs index f195dcb421..52c6e85801 100644 --- a/backend/tests/preserve_on_behalf_of.rs +++ b/backend/tests/preserve_on_behalf_of.rs @@ -2466,9 +2466,9 @@ async fn test_schedule_group_permissioned_as(db: Pool) -> anyhow::Resu "the schedule response derives the address from the principal, not from the column" ); - // Toggling is not a change of identity. `set_enabled` used to stamp the column with whoever - // flipped it, which is how a row came to name someone other than its principal; the - // desynchronised value above doubles as the sentinel that nothing rewrote it. + // Toggling is not a change of identity: it must leave the address column alone, or the row + // ends up naming whoever flipped the switch rather than its principal. The desynchronised + // value above doubles as the sentinel that nothing rewrote it. let resp = authed( client().post(format!( "{base}/schedules/setenabled/u/test-user/schedule_group_perm"