[FIX] base: fix `populate.randint` tool

The randint generator generated always `None`
because the `return` was forget in method of `populate.compute`.
- Add a test for the randint tool, and fix the issue.
- Change the documentation of `_populate_factories` to be more
explicit about the custom generator.

closes odoo/odoo#60327

Signed-off-by: Xavier Dollé (xdo) <xdo@odoo.com>

2 files changed, 3 insertions, 0 deletions

diff --git a/odoo/addons/test_populate/models.py b/odoo/addons/test_populate/models.py
index d59524d185b..96b8c5565be 100644
--- a/odoo/addons/test_populate/models.py
+++ b/odoo/addons/test_populate/models.py
@@ -17,6 +17,7 @@ class TestPopulateModel(models.Model):
     some_ref = fields.Integer('Reference')
     dependant_field_1 = fields.Char('Dependant 1')
     dependant_field_2 = fields.Char('Dependant 2')
+    sequence = fields.Integer("Sequence")
 
     _populate_dependencies = ['test.populate.category']
 
@@ -53,6 +54,7 @@ class TestPopulateModel(models.Model):
             ('_dependant', generate_dependant),
             ('name', populate.compute(get_name)),
             ('category_id', populate.randomize([False] + category_ids)),
+            ('sequence', populate.randint(1, 10))
         ]
 
 class TestPopulateDependencyModel(models.Model):
diff --git a/odoo/addons/test_populate/tests/test_populate.py b/odoo/addons/test_populate/tests/test_populate.py
index 0a5c80d8025..1cd71ec20f8 100644
--- a/odoo/addons/test_populate/tests/test_populate.py
+++ b/odoo/addons/test_populate/tests/test_populate.py
@@ -47,6 +47,7 @@ class TestPopulate(common.TransactionCase):
         self.assertEqual(records.mapped('some_ref')[5:20], [
             1, 0, 2, 4, 4, 3, 4, 1, 2, 2, 2, 4, 4, 1, 2
         ])
+        self.assertEqual(records.mapped('sequence')[:20], [6, 10, 1, 1, 1, 3, 8, 9, 1, 5, 9, 5, 7, 3, 5, 3, 6, 4, 9, 2])  # Test randint
 
     @mute_logger('odoo.cli.populate')
     def test_populate_inherit(self):