tests: use Command API for m2m

Author: Milan Topuzov

queue_job/models/queue_job.py

@@ -222,15 +222,13 @@ def _dependency_graph_vis_node(self):
         }
 
     def _compute_graph_jobs_count(self):
-        # Use _read_group (read_group is deprecated in Odoo 19)
         graph_uuids = [uuid for uuid in self.mapped("graph_uuid") if uuid]
         if graph_uuids:
             rows = self.env["queue.job"]._read_group(
                 [("graph_uuid", "in", graph_uuids)],
                 ["graph_uuid"],
                 ["__count"],
             )
-            # rows are tuples of (graph_uuid, count)
             count_per_graph_uuid = {graph_uuid: cnt for graph_uuid, cnt in rows}
         else:
             count_per_graph_uuid = {}

queue_job/tests/test_json_field.py

@@ -7,6 +7,7 @@
 from lxml import etree
 
 from odoo.tests import common
+from odoo import Command
 
 # pylint: disable=odoo-addons-relative-import
 # we are testing, we want to test as we were an external consumer of the API
@@ -25,8 +26,8 @@ def setUpClass(cls):
                 "name": "Demo User (Queue)",
                 "login": "queue_demo_user",
                 "company_id": main_company.id,
-                "company_ids": [(6, 0, [main_company.id])],
-                "group_ids": [(6, 0, [group_user.id])],
+                "company_ids": [Command.set([main_company.id])],
+                "group_ids": [Command.set([group_user.id])],
             }
         )
 

test_queue_job/tests/test_job.py

@@ -8,6 +8,7 @@
 import odoo.tests.common as common
 
 from odoo.addons.queue_job import identity_exact
+from odoo import Command
 from odoo.addons.queue_job.delay import DelayableGraph
 from odoo.addons.queue_job.exception import (
     FailedJobError,
@@ -43,8 +44,8 @@ def setUpClass(cls):
                 "name": "Demo User (Queue)",
                 "login": "queue_demo_user_3",
                 "company_id": main_company.id,
-                "company_ids": [(6, 0, [main_company.id])],
-                "group_ids": [(6, 0, [group_user.id])],
+                "company_ids": [Command.set([main_company.id])],
+                "group_ids": [Command.set([group_user.id])],
             }
         )
 
@@ -414,8 +415,8 @@ def setUpClass(cls):
                 "name": "Demo User (Queue)",
                 "login": "queue_demo_user_4",
                 "company_id": main_company.id,
-                "company_ids": [(6, 0, [main_company.id])],
-                "group_ids": [(6, 0, [group_user.id])],
+                "company_ids": [Command.set([main_company.id])],
+                "group_ids": [Command.set([group_user.id])],
             }
         )
 
@@ -554,8 +555,8 @@ def setUpClass(cls):
                 "name": "Demo User (Queue)",
                 "login": "queue_demo_user_5",
                 "company_id": main_company.id,
-                "company_ids": [(6, 0, [main_company.id])],
-                "group_ids": [(6, 0, [group_user.id])],
+                "company_ids": [Command.set([main_company.id])],
+                "group_ids": [Command.set([group_user.id])],
             }
         )
 
@@ -652,7 +653,7 @@ def test_follower_when_write_fail(self):
         vals = {
             "name": "xx",
             "login": "xx",
-            "group_ids": [(6, 0, [group.id])],
+            "group_ids": [Command.set([group.id])],
             "active": False,
         }
         inactiveusr = self.user.create(vals)
@@ -711,7 +712,7 @@ def setUp(self):
         self.simple_user = User.create(
             {
                 "partner_id": self.partner_user.id,
-                "company_ids": [(4, main_company.id)],
+                "company_ids": [Command.link(main_company.id)],
                 "login": "simple_user",
                 "name": "simple user",
                 "group_ids": [],
@@ -732,10 +733,10 @@ def setUp(self):
             {
                 "partner_id": self.other_partner_a.id,
                 "company_id": self.other_company_a.id,
-                "company_ids": [(4, self.other_company_a.id)],
+                "company_ids": [Command.link(self.other_company_a.id)],
                 "login": "my_login a",
                 "name": "my user A",
-                "group_ids": [(4, grp_queue_job_manager)],
+                "group_ids": [Command.link(grp_queue_job_manager)],
             }
         )
         self.other_partner_b = Partner.create(
@@ -752,10 +753,10 @@ def setUp(self):
             {
                 "partner_id": self.other_partner_b.id,
                 "company_id": self.other_company_b.id,
-                "company_ids": [(4, self.other_company_b.id)],
+                "company_ids": [Command.link(self.other_company_b.id)],
                 "login": "my_login_b",
                 "name": "my user B",
-                "group_ids": [(4, grp_queue_job_manager)],
+                "group_ids": [Command.link(grp_queue_job_manager)],
             }
         )
 

test_queue_job/tests/test_json_field.py

@@ -4,6 +4,7 @@
 import json
 
 from odoo.tests import common
+from odoo import Command
 
 # pylint: disable=odoo-addons-relative-import
 # we are testing, we want to test as if we were an external consumer of the API
@@ -22,8 +23,8 @@ def setUpClass(cls):
                 "name": "Demo User (Queue)",
                 "login": "queue_demo_user_2",
                 "company_id": main_company.id,
-                "company_ids": [(6, 0, [main_company.id])],
-                "group_ids": [(6, 0, [group_user.id])],
+                "company_ids": [Command.set([main_company.id])],
+                "group_ids": [Command.set([group_user.id])],
             }
         )