Skip to content

Commit c328fa2

Browse files
committed
test: link Odoo 19 skips to tracking issues
1 parent 564f3f4 commit c328fa2

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

endpoint_route_handler/tests/test_endpoint.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ def test_as_tool_base_data(self):
5050
new_route.route += "/new"
5151
self.assertNotEqual(new_route.endpoint_hash, first_hash)
5252

53-
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers")
53+
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers (#51)")
5454
@mute_logger("odoo.addons.base.models.ir_http")
5555
def test_as_tool_register_single_controller(self):
5656
new_route = make_new_route(self.env)
@@ -75,7 +75,7 @@ def test_as_tool_register_single_controller(self):
7575
self.assertNotIn("/my/test/route", [x.rule for x in rmap._rules])
7676
self.assertIn("/my/test/route/new", [x.rule for x in rmap._rules])
7777

78-
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers")
78+
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers (#51)")
7979
@mute_logger("odoo.addons.base.models.ir_http")
8080
def test_as_tool_register_controllers(self):
8181
new_route = make_new_route(self.env)
@@ -100,7 +100,7 @@ def test_as_tool_register_controllers(self):
100100
self.assertNotIn("/my/test/route", [x.rule for x in rmap._rules])
101101
self.assertIn("/my/test/route/new", [x.rule for x in rmap._rules])
102102

103-
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers")
103+
@unittest.skip("Odoo 19: routing_map() no longer reflects dynamically registered controllers (#51)")
104104
@mute_logger("odoo.addons.base.models.ir_http")
105105
def test_as_tool_register_controllers_dynamic_route(self):
106106
route = "/my/app/<model(app.model):foo>"
@@ -124,7 +124,7 @@ def setUp(self):
124124
super().setUp()
125125
EndpointRegistry.wipe_registry_for(self.env.cr)
126126

127-
@unittest.skip("Deadlocks on Odoo 19: Registry() acquisition conflicts with test cursor lock")
127+
@unittest.skip("Deadlocks on Odoo 19: Registry() acquisition conflicts with test cursor lock (#52)")
128128
@mute_logger("odoo.addons.base.models.ir_http", "odoo.modules.registry")
129129
def test_cross_env_consistency(self):
130130
"""Ensure route updates are propagated to all envs."""

fastapi/tests/test_fastapi.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ def test_lang(self):
108108
self._assert_expected_lang("fr-FR,en;q=0.7,en-GB;q=0.3", b'"fr_BE"')
109109
self._assert_expected_lang("fr-FR;q=0.1,en;q=1.0,en-GB;q=0.8", b'"en_US"')
110110

111-
@unittest.skip("Odoo 19: FastAPI retrying mechanism returns 500 in test mode")
111+
@unittest.skip("Odoo 19: FastAPI retrying mechanism returns 500 in test mode (#53)")
112112
def test_retrying(self):
113113
"""Test that the retrying mechanism is working as expected with the
114114
FastAPI endpoints.
@@ -119,7 +119,7 @@ def test_retrying(self):
119119
self.assertEqual(response.status_code, 200)
120120
self.assertEqual(int(response.content), nbr_retries)
121121

122-
@unittest.skip("Odoo 19: FastAPI retrying mechanism returns 500 in test mode")
122+
@unittest.skip("Odoo 19: FastAPI retrying mechanism returns 500 in test mode (#53)")
123123
def test_retrying_post(self):
124124
"""Test that the retrying mechanism is working as expected with the
125125
FastAPI endpoints in case of POST request with a file.
@@ -206,7 +206,7 @@ def test_request_validation_error(self) -> None:
206206
mocked_commit.assert_not_called()
207207
self.assertEqual(response.status_code, status.HTTP_422_UNPROCESSABLE_ENTITY)
208208

209-
@unittest.skip("Odoo 19: BaseCursor.commit mock not invoked by HTTP test runner")
209+
@unittest.skip("Odoo 19: BaseCursor.commit mock not invoked by HTTP test runner (#54)")
210210
def test_no_commit_on_exception(self) -> None:
211211
# this test check that the way we mock the cursor is working as expected
212212
# and that the transaction is rolled back in case of exception.

0 commit comments

Comments
 (0)