summaryrefslogtreecommitdiff
path: root/addons/test_mass_mailing/models/mass_mail_test.py
AgeCommit message (Collapse)Author
2020-04-24[IMP] link_tracker, (test_)mass_mailing: improve test tools and clean ↵Thibault Delavallée
existing tests URPOSE Prepare code cleaning and optimization in mail, mass_mailing and SMS by cleaning models for readability and code complexity and footprint reduction. SPECIFICATIONS Add tool methods and asserts for link tracker tests. Add tool methods and assets for mass mailing tests, notably about mailing traces and their link with emails. Merge existing and duplicated tests and rewrite them a bit. Rename and improve test_mass_mailing models. LINKS Prepares task ID 2238597 (notification and trace models cleaning) Task ID 1906925 (mass mailing tests cleaning) PR #50169 Related: odoo/enterprise#10180 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2020-04-22[REF] test_mail: sun is shining, no more umbrellasThibault Delavallée
PURPOSE Have a cleaner test_mail addons SPECIFICATIONS Umbrella -> Container, easier to understand that we target a ticket / project like model using mail.test.ticket and mail.test.container . LINKS Prepares Task ID 2238597 (clean notification models) Prepares Task ID 2083854 (improve mass mailing technical flows) PR #49891
2020-04-22[REF] test_mail: rename test models to ease understandingThibault Delavallée
PURPOSE Have a cleaner test_mail addons SPECIFICATIONS Keep only mail-related tests, move odoobot in test mail full, send "update notification" tests in mail (specific to mail). Merge some test files to lessen number of files, perform light file renaming. Split test mail models file to prepare some cleaning in those models and tests. LINKS Prepares Task ID 2238597 (clean notification models) Prepares Task ID 2083854 (improve mass mailing technical flows) PR #49891
2020-03-20[REF] mail: merge blacklist and address mixinsThibault Delavallée
PURPOSE Perform some code cleaning in mail module SPECIFICATIONS mail.address.mixin is used only for mail.thread.blacklist mixin. Let us merge those two and simplify class organization. LINKS Task ID 2222033 closes odoo/odoo#48115 Related: odoo/upgrade#967 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
2019-05-13[REF] mail: make blacklist a mixin built upon mail.thread and add message_bounceThibault Delavallée
* mail.blacklist.mixin now clearly make inheritance on mail.thread. Indeed we consider models using the blacklist mechanism as being used in mailing features, meaning they will anyway inherit form mail.thread. In standard Odoo it is the case for the 3 models using it (Lead/Opportunity, Contact and Mailing Contact); * define message_bounce directly in mail.blacklist.mixin. Bounce counter is indeed linked to the blacklist and mailing mechanism; * rename mail.blacklist.mixin to mail.thread.blacklist to ensure coherency with mail.thread and mail.thread.cc (another mixin build on mailL.thread); Inherit declarations in various addons are updated accordingly. Related to task ID 1911679 Linked to PR #29483
2019-01-04[REF] various: update tracking parametersThibault Delavallée
Purpose is to clean the use of tracking parameters on fields. Parameters are merged and is now tracking=<int> or tracking=True. This commit is linked to task ID 1903814 and PR #28430.
2018-11-20[IMP] mail, mass_mailing, crm : stringify _primary_emailDavid Beguin
_primary_email was an array to be able to use it directly in the depends. Typically a 'depends' with a computed value (lambda) needs an array. Otherwise, it will only take the first character of the field name string. By setting _primary_email as a string, the lambda should always get an array. But it simplifies a lot the usage of the _primary_email. also, having an array was confusing as we could think that more than one field can be considered as the primary email. This is not the goal hare. So having the _primary_email as a single string is more 'self describing'. Only one field should be the _primary_email.
2018-11-20[IMP] mail : apply mail.address.mixin on blacklistDavid Beguin
Apply the mail.address.mixin on blacklist mixin. So that every model inheriting the blacklist.mixin will also inherit the address.mixin. That simplify also the comparaison to - and the searches on - the blacklist. Task ID 1896677
2018-10-02[ADD] test_mass_mailing: add module to hold feature and performance tests ↵Thibault Delavallée
for mass mailing Purpose of this commit is to add the skeleton of a module holding models and tests for mass mailing. It is build on top of test_mail in order to use data, models and parameters already defined in it. Moreover having a separate module allow to define test models that will not bloat production databases. It will soon hold unit tests for blacklist mechanism. Future commits will gradually move tests from mass mailing module and improve them. This commit is related to task ID 33224 (original blacklist implementation done for v12) and its PR #25966 as well as task ID 1889703 (tests and fixes) and its PR #27330. Done with collaboration of @dbeguin.