[IMP]:improved test case

bzr revid: apa@tinyerp.com-20120906105914-iwoi15ghgtibt2lr

addons/mail/data/mail_data.xml

@@ -15,13 +15,16 @@
         
         <record id="mail_subtype_other" model="mail.message.subtype">
             <field name="name">other</field>
-            <field name="default" eval="False"/>
+            <field name="model_ids" eval="[(4,ref('mail.model_mail_group'))]"/>
+            <field name="default" eval='False'/>
         </record>
         <record id="mail_subtype_email" model="mail.message.subtype">
             <field name="name">email</field>
+            <field name="model_ids" eval="[(4,ref('mail.model_mail_group'))]"/>
         </record>
         <record id="mail_subtype_comment" model="mail.message.subtype">
             <field name="name">comment</field>
+            <field name="model_ids" eval="[(4,ref('mail.model_mail_group'))]"/>
         </record>
     </data>
 </openerp>

addons/mail/mail_thread.py

@@ -562,7 +562,7 @@ class mail_thread(osv.AbstractModel):
         self.message_post(cr, uid, [id], message, context=context)
 
     def message_post(self, cr, uid, thread_id, body='', subject=False,
-            type='notification', parent_id=False, attachments=None, subtype='other', context=None, **kwargs):
+            type='notification', parent_id=False, attachments=None, subtype='comment', context=None, **kwargs):
         """ Post a new message in an existing thread, returning the new
             mail.message ID. Extra keyword arguments will be used as default
             column values for the new mail.message record.

addons/mail/tests/test_mail.py

@@ -616,9 +616,7 @@ class test_mail(common.TransactionCase):
         # 0 - Admin
         p_a_id = user_admin.partner_id.id
         # Subscribe #1,
-        self.mail_group_model_id = self.ir_model.search(cr, uid, [('model','=', 'mail.group')])[0]
-        subtype_ids = self.mail_message_subtype.search(cr, uid, [])
-        self.mail_message_subtype.write(cr,uid,subtype_ids,{'model_ids':[(4,self.mail_group_model_id )]})
+   
         group_pigs.message_subscribe_users([uid])
 
         # Mail data