diff --git a/shopfloor_base/models/shopfloor_app.py b/shopfloor_base/models/shopfloor_app.py index 65bc40713f7..3731d9c1cbc 100644 --- a/shopfloor_base/models/shopfloor_app.py +++ b/shopfloor_base/models/shopfloor_app.py @@ -163,7 +163,8 @@ def _prepare_endpoint_rules(self, options=None): def _registered_endpoint_rule_keys(self): # `endpoint.route.sync.mixin` api - return [x[0] for x in self._registered_routes()] + # TODO: add tests + return [x.key for x in self._registered_routes()] def _register_hook(self): super()._register_hook() diff --git a/shopfloor_base/tests/test_shopfloor_app.py b/shopfloor_base/tests/test_shopfloor_app.py index 690e76b76dd..166003bc3cb 100644 --- a/shopfloor_base/tests/test_shopfloor_app.py +++ b/shopfloor_base/tests/test_shopfloor_app.py @@ -65,7 +65,12 @@ def _test_registered_routes(self, rec): ), "method_name": "_process_endpoint", } - self.assertEqual(rule.handler_options, expected_handler_opts) + for k, v in expected_handler_opts.items(): + self.assertEqual( + rule.handler_options[k], + v, + f"{k} differs: {rule.handler_options[k]} != {v}", + ) _check[rule.route] = set(rule.routing["methods"]) expected = { # TODO: review methods @@ -145,7 +150,10 @@ def test_make_app_info(self): ) def test_make_app_manifest(self): - param = "http://localhost:8069" + self.env["ir.config_parameter"].sudo().set_param( + "web.base.url", "http://foo.com" + ) + param = "http://foo.com" manifest = self.shopfloor_app._make_app_manifest() expected = { "name": self.shopfloor_app.name,