[IMP] survey_record_generation : Add tests on record creation

survey_record_generation/models/survey_question_answer.py

@@ -9,15 +9,15 @@ _logger = logging.getLogger(__name__)
 class SurveyQuestionAnswer(models.Model):
     _inherit = 'survey.question.answer'
 
-    record_id = fields.Reference(string="Referenced record",  selection='_selection_target_model')    
-    model_id = fields.Many2one('ir.model', related="question_id.model_id")    
+    record_id = fields.Reference(string="Referenced record",  selection='_selection_target_model')
+    model_id = fields.Many2one('ir.model', related="question_id.model_id")
     answer_values_type = fields.Selection(related="question_id.answer_values_type")
     value_char = fields.Char('Value')
 
     @api.model
     def _selection_target_model(self):
         return [(model.model, model.name) for model in self.env['ir.model'].sudo().search([])]
-    
+
     @api.onchange('record_id')
     def onchange_record_id(self):
         if self.record_id:
@@ -31,7 +31,7 @@ class SurveyQuestionAnswer(models.Model):
             or "record_id" not in fields
         ):
             return result
-        
+
         model = self.env['ir.model'].browse(result.get("model_id")).model
         res = self.env[model].search([], limit=1)
         if res:
@@ -39,4 +39,4 @@ class SurveyQuestionAnswer(models.Model):
                 model,
                 res.id,
             )
-        return result
+        return result

survey_record_generation/models/survey_record_creation_field_values.py

@@ -8,7 +8,7 @@ from odoo.tools.misc import format_date
 _logger = logging.getLogger(__name__)
 
 
-type_mapping = {
+type_mapping = {  #field types on the left, question types on the right. TODO : what about booleans ?
     "char": ["char_box", "numerical_box", "date", "datetime", "simple_choice", "multiple_choice"],
     "text": ["char_box", "date", "simple_choice"],
     "html": ["text_box", "numerical_box", "datetime", "simple_choice"],
@@ -17,7 +17,7 @@ type_mapping = {
     "date": ["date"],
     "datetime": ["datetime"],
     "many2one": ["simple_choice"],
-    "many2many": ["multiple_choice"], 
+    "many2many": ["multiple_choice"],
     "selection": ["char_box", "simple_choice"]
 }
 
@@ -32,7 +32,7 @@ class SurveyRecordCreationFieldValues(models.Model):
     model_id = fields.Many2one('ir.model', related="survey_record_creation_id.model_id")
 
     field_id = fields.Many2one(
-        'ir.model.fields', 
+        'ir.model.fields',
         domain="[('model_id','=',model_id),('readonly','=',False),('ttype','in',['char','selection','text','html','integer','float','date','datetime','many2one','many2many', 'boolean'])]",
         ondelete="cascade")
     field_relation = fields.Char(related='field_id.relation')
@@ -40,10 +40,10 @@ class SurveyRecordCreationFieldValues(models.Model):
     field_help = fields.Html('Help', compute="_compute_field_help")
 
     value_origin = fields.Selection(
-        [('fixed','Fixed'),('question','Question'),('other_record','From other created record')], 
-        string="Value origin", 
-        required=True, 
-        default='fixed', 
+        [('fixed','Fixed'),('question','Question'),('other_record','From other created record')],
+        string="Value origin",
+        required=True,
+        default='fixed',
         help="""* Fixed: you can set the value in value field
         * Question: Response of the question will set the value. If you do not see your question, maybe the type of question do not match the type of field
         * From other created record: You can set other record creation to link several created records. Can only be used with many2one fields.""")
@@ -57,7 +57,7 @@ class SurveyRecordCreationFieldValues(models.Model):
     fixed_value_integer = fields.Integer("Value")
     fixed_value_float = fields.Float("Value")
     fixed_value_date = fields.Date("Value")
-    fixed_value_datetime = fields.Datetime("Value")    
+    fixed_value_datetime = fields.Datetime("Value")
     fixed_value_boolean = fields.Boolean("Value")
 
     displayed_value = fields.Char("Value", compute="_compute_displayed_value")
@@ -84,7 +84,7 @@ class SurveyRecordCreationFieldValues(models.Model):
                 record_creation_field_values.allowed_question_ids = None
                 return
             question_domain = [('survey_id','=',record_creation_field_values.survey_id.id)]
-            
+
             if record_creation_field_values.field_id.ttype in ['many2one','many2many']:
                 question_domain.extend(['|','&',('answer_values_type','=','record'),('model_id','=',record_creation_field_values.field_id.relation),('answer_values_type','=','value')])
             if record_creation_field_values.field_id.ttype in type_mapping:
@@ -96,7 +96,7 @@ class SurveyRecordCreationFieldValues(models.Model):
     @api.model
     def _selection_target_model(self):
         return [(model.model, model.name) for model in self.env['ir.model'].sudo().search([])]
-    
+
     def clean_values(self):
         # clean values
         self.fixed_value_many2many = None
@@ -119,7 +119,7 @@ class SurveyRecordCreationFieldValues(models.Model):
         # Set reference field model and select first record
         if self.field_id and self.field_id.ttype == 'many2one' and self.field_id.relation:
             rec = self.env[self.field_id.relation].search([], limit=1)
-            if rec:            
+            if rec:
                 self.fixed_value_many2one = f"{self.field_id.relation},{rec.id}"
             else:
                 model_name = self.env['ir.model'].search([('model','=',self.field_id.relation)]).name
@@ -136,12 +136,12 @@ class SurveyRecordCreationFieldValues(models.Model):
         if self.value_origin == 'fixed':
             if self.field_type == 'many2one':
                 if self.fixed_value_many2one:
-                    return self.fixed_value_many2one.id            
+                    return self.fixed_value_many2one.id
             elif self.field_type == 'many2many':
                 return [m2m.value_reference.id for m2m in self.fixed_value_many2many if m2m.value_reference]
             else:
                 return self["fixed_value_"+self.field_type]
-        
+
 
     @api.onchange("fixed_value_char","fixed_value_selection","fixed_value_text","fixed_value_html","fixed_value_integer","fixed_value_float","fixed_value_date","fixed_value_datetime",'fixed_value_many2one', "fixed_value_many2many","other_created_record_id","question_id")
     def _compute_displayed_value(self):
@@ -172,7 +172,7 @@ class SurveyRecordCreationFieldValues(models.Model):
                 record.displayed_value = ""
 
 class SurveyRecordCreationFieldValuesX2m(models.Model):
-    """O2m an M2m default values 
+    """O2m an M2m default values
     """
     _name = 'survey.record.creation.field.values.x2m'
 
@@ -182,16 +182,16 @@ class SurveyRecordCreationFieldValuesX2m(models.Model):
     @api.model
     def _selection_target_model(self):
         return [(model.model, model.name) for model in self.env['ir.model'].sudo().search([])]
-    
+
 
     @api.onchange('survey_record_creation_field_values_id')
     def _onchange_model_name(self):
         # Set reference field model and select first record
-        field = self.survey_record_creation_field_values_id.field_id 
+        field = self.survey_record_creation_field_values_id.field_id
         if field and "2many" in field.ttype and field.relation:
             rec = self.env[field.relation].search([], limit=1)
-            if rec:            
+            if rec:
                 self.value_reference = f"{field.relation},{rec.id}"
             else:
                 model_name = self.env['ir.model'].search([('model','=',field.relation)]).name
-                raise ValueError(_('You should append at least one record in %s',(model_name,)))
+                raise ValueError(_('You should append at least one record in %s',(model_name,)))

survey_record_generation/models/survey_user_input.py

@@ -41,7 +41,7 @@ class SurveyUserInput(models.Model):
                 model = record_creation.model_id.model
                 vals = {}
                 ModelClass = self.env[model]
-                
+
                 for field_value in record_creation.field_values_ids:
                     if field_value.value_origin == 'fixed':
                         vals[field_value.field_id.name] = field_value.get_fixed_value_for_record_creation()
@@ -55,7 +55,7 @@ class SurveyUserInput(models.Model):
                         if field_value.question_id.question_type in ['simple_choice', 'multiple_choice','matrix']:
                             if field_value.question_id.answer_values_type == 'record':
                                 record_ids = []
-                                for user_input_line in user_input_lines:                            
+                                for user_input_line in user_input_lines:
                                     if user_input_line.suggested_answer_id and user_input_line.suggested_answer_id.record_id:
                                         record_ids.append(user_input_line.suggested_answer_id.record_id.id)
                                 if field_value.question_id.question_type == 'simple_choice':
@@ -67,7 +67,7 @@ class SurveyUserInput(models.Model):
                                     vals[field_value.field_id.name] = record_ids
                             if field_value.question_id.answer_values_type == 'value':
                                 if field_value.field_id.ttype == "boolean":
-                                    boolean_value = user_input_lines[0].suggested_answer_id.value_char in [True, 1, "1", "True", "true", "Oui", "oui"]
+                                    boolean_value = user_input_lines[0].suggested_answer_id.value_char in [True, 1, "1", "True", "true", "Oui", "oui", "Yes", "yes"]
                                     vals[field_value.field_id.name] = boolean_value
                                 else:
                                     vals[field_value.field_id.name] = user_input_lines[0].suggested_answer_id.value_char
@@ -110,8 +110,8 @@ class SurveyUserInput(models.Model):
                     # Link generated records to user input
                     self.env['survey.generated.record'].create({
                         'survey_record_creation_name':record_creation.name,
-                        'survey_record_creation_id':record_creation.id, 
-                        'user_input_id':user_input.id, 
+                        'survey_record_creation_id':record_creation.id,
+                        'user_input_id':user_input.id,
                         "created_record_id":"%s,%s" % (model,record.id)
                     })