OpenERP Community Backports (Addons)

Merge lp:~acsone-openerp/ocb-addons/fix_test_20_message_post into lp:ocb-addons

  1. fix_test_20_message_post
  2. Merge into 7.0
Proposed by Stéphane Bidoul (Acsone)
Status: Merged
Merged at revision: 9624
Proposed branch: lp:~acsone-openerp/ocb-addons/fix_test_20_message_post
Merge into: lp:ocb-addons
Diff against target: 12 lines (+1/-1)
1 file modified
mail/tests/test_mail_features.py (+1/-1)
To merge this branch: bzr merge lp:~acsone-openerp/ocb-addons/fix_test_20_message_post
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Guewen Baconnier @ Camptocamp code review Approve
Stefan Rijnhart (Opener) code review Approve
Review via email: mp+192821@code.launchpad.net

Commit message

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

Description of the change

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

To post a comment you must log in.
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks!

review: Approve (code review)
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve (code review)
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

Preview Diff

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