openerp-community-reviewer team mailing list archive

[Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>]

Stéphane Bidoul (Acsone) has proposed merging lp:~acsone-openerp/ocb-addons/fix_test_20_message_post into lp:ocb-addons.

Commit message:
[FIX] fix test_20_message_post to take care of different notification_email_send default value in ocb

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1163832 in OpenERP Community Backports (Addons): "Partner settings should default to opt-out for email messages"
  https://bugs.launchpad.net/ocb-addons/+bug/1163832

For more details, see:
https://code.launchpad.net/~acsone-openerp/ocb-addons/fix_test_20_message_post/+merge/192821

[FIX] fix test_20_message_post to take care of different notification_email_send default value in ocb
-- 
https://code.launchpad.net/~acsone-openerp/ocb-addons/fix_test_20_message_post/+merge/192821
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~acsone-openerp/ocb-addons/fix_test_20_message_post into lp:ocb-addons.

=== modified file 'mail/tests/test_mail_features.py'
--- mail/tests/test_mail_features.py	2013-08-23 12:37:16 +0000
+++ mail/tests/test_mail_features.py	2013-10-27 15:10:37 +0000
@@ -237,7 +237,7 @@
         self.res_users.write(cr, uid, [uid], {'email': 'a@a', 'notification_email_send': 'comment'})
         self.res_users.write(cr, uid, [self.user_raoul_id], {'email': 'r@r'})
         # 1 - Bert Tartopoils, with email, should receive emails for comments and emails
-        p_b_id = self.res_partner.create(cr, uid, {'name': 'Bert Tartopoils', 'email': 'b@b'})
+        p_b_id = self.res_partner.create(cr, uid, {'name': 'Bert Tartopoils', 'email': 'b@b', 'notification_email_send': 'comment'})
         # 2 - Carine Poilvache, with email, should receive emails for emails
         p_c_id = self.res_partner.create(cr, uid, {'name': 'Carine Poilvache', 'email': 'c@c', 'notification_email_send': 'email'})
         # 3 - Dédé Grosbedon, without email, to test email verification; should receive emails for every message