Fix failing specs

Author: rsmithlal

app/policies/better_together/conversation_policy.rb

@@ -41,10 +41,12 @@ def permitted_participants
       else
         role = BetterTogether::Role.find_by(identifier: 'platform_manager')
         manager_ids = BetterTogether::PersonPlatformMembership.where(role_id: role.id).pluck(:member_id)
-        BetterTogether::Person.where(id: manager_ids)
-                              .or(BetterTogether::Person.privacy_public.where('preferences @> ?',
-                                                                              { receive_messages_from_members: true }.to_json)) # rubocop:disable Layout/LineLength
-                              .distinct
+        # Include platform managers and any person who has explicitly opted in to receive messages
+        opted_in = BetterTogether::Person.where(
+          'preferences @> ?', { receive_messages_from_members: true }.to_json
+        )
+
+        BetterTogether::Person.where(id: manager_ids).or(opted_in).distinct
       end
     end
 

spec/factories/better_together/people.rb

@@ -6,7 +6,7 @@ module BetterTogether
   FactoryBot.define do
     factory :better_together_person, class: Person, aliases: %i[person inviter invitee creator author] do
       id { Faker::Internet.uuid }
-      name { Faker::Name.name }
+      name { Faker::Name.unique.name }
       description { Faker::Lorem.paragraph(sentence_count: 3) }
       identifier { Faker::Internet.unique.username(specifier: 10..20) }