Fix and add tests for show requestor email on AddMember task

Luciano Prestes · Thiago Ribeiro
1 parent ced4464b
Showing 1 changed file with 11 additions and 2 deletions Show diff stats
test/unit/add_member_test.rb
@@ -121,12 +121,21 @@ class AddMemberTest &lt; ActiveSupport::TestCase
     assert_match(/#{task.requestor.name} wants to be a member of '#{community.name}'/, email.subject)
   end
-  should 'have target notification description with requestor email' do
+  should 'notification description with requestor email if requestor email is public' do
     new_person = create_user('testuser').person
     new_person.update_attributes!({:fields_privacy => {:email => 'public'}})
     task = AddMember.new(:person => new_person, :organization => community)
-    assert_match(/#{task.requestor.name} \(#{task.requestor.email}\) wants to be a member of '#{community.name}'./, task.target_notification_description)
+    assert_match(/\(#{task.requestor.email}\)/, task.target_notification_description)
+  end
+
+  should 'notification description without requestor email if requestor email is not public' do
+    new_person = create_user('testuser').person
+    new_person.update_attributes!({:fields_privacy => {:email => '0'}})
+
+    task = AddMember.new(:person => new_person, :organization => community)
+
+    assert_not_match(/\(#{task.requestor.email}\)/, task.target_notification_description)
   end
 end