From 72b2abef650c5a04221cf1cd228ca8057ae16da6 Mon Sep 17 00:00:00 2001 From: Morgan Roderick Date: Thu, 25 Jun 2026 20:36:32 +0200 Subject: [PATCH] fix: flaky PII test when Faker generates short member names The test checks error logs exclude the member's name, but Faker can generate names as short as 3 chars (e.g. Man) that happen to be substrings of other words in the log message (InvitationManager). Use word-boundary regex instead of substring include. Refs: https://github.com/codebar/planner/actions/runs/28191685483 --- spec/services/invitation_manager_spec.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spec/services/invitation_manager_spec.rb b/spec/services/invitation_manager_spec.rb index d8d28c4e6..a192301ac 100644 --- a/spec/services/invitation_manager_spec.rb +++ b/spec/services/invitation_manager_spec.rb @@ -290,7 +290,7 @@ expect(message).to include("workshop_id=#{workshop.id}") expect(message).to include('role=Student') expect(message).not_to include(member.email) - expect(message).not_to include(member.name) + expect(message).not_to match(/#{Regexp.escape(member.name)}/) end manager.send(:create_invitation, workshop, member, 'Student')