cleanup

Author: pcrespov

packages/service-library/src/servicelib/aiohttp/rest_middlewares.py

@@ -31,7 +31,11 @@
 from .typing_extension import Handler, Middleware
 
 DEFAULT_API_VERSION = "v0"
-FMSG_INTERNAL_ERROR_USER_FRIENDLY = "We apologize for the inconvenience. Our team has recorded the issue and is working to resolve it as quickly as possible. Thank you for your patience [{}]"
+_FMSG_INTERNAL_ERROR_USER_FRIENDLY = (
+    "We apologize for the inconvenience."
+    " Our team has recorded the issue [{error_code}] and is working to resolve it as quickly as possible."
+    " Thank you for your patience"
+)
 
 
 _logger = logging.getLogger(__name__)
@@ -58,7 +62,7 @@ def _process_and_raise_unexpected_error(request: web.BaseRequest, err: Exception
         if isinstance(err, OsparcErrorMixin):
             error_context.update(err.error_context())
 
-        frontend_msg = FMSG_INTERNAL_ERROR_USER_FRIENDLY.format(error_code)
+        frontend_msg = _FMSG_INTERNAL_ERROR_USER_FRIENDLY.format(error_code)
         log_msg = create_troubleshotting_log_message(
             message_to_user=frontend_msg,
             error=err,

packages/service-library/tests/aiohttp/test_rest_middlewares.py

@@ -16,7 +16,7 @@
 from models_library.utils.json_serialization import json_dumps
 from servicelib.aiohttp import status
 from servicelib.aiohttp.rest_middlewares import (
-    FMSG_INTERNAL_ERROR_USER_FRIENDLY,
+    _FMSG_INTERNAL_ERROR_USER_FRIENDLY,
     envelope_middleware_factory,
     error_middleware_factory,
 )
@@ -238,7 +238,10 @@ async def test_raised_unhandled_exception(
         # user friendly message with OEC reference
         assert "OEC" in error["message"]
         parsed_oec = parse_error_code(error["message"]).pop()
-        assert FMSG_INTERNAL_ERROR_USER_FRIENDLY.format(parsed_oec) == error["message"]
+        assert (
+            _FMSG_INTERNAL_ERROR_USER_FRIENDLY.format(error_code=parsed_oec)
+            == error["message"]
+        )
 
         # avoids details
         assert not error.get("errors")