fix: parameters: fix return type to bytes in internal function

Author: Ran Isenberg

aws_lambda_powertools/utilities/parameters/appconfig.py

@@ -84,7 +84,7 @@ def __init__(
 
         super().__init__()
 
-    def _get(self, name: str, **sdk_options) -> bytes:
+    def _get(self, name: str, **sdk_options) -> str:
         """
         Retrieve a parameter value from AWS App config.
 
@@ -104,7 +104,7 @@ def _get(self, name: str, **sdk_options) -> bytes:
         sdk_options["ClientId"] = CLIENT_ID
 
         response = self.client.get_configuration(**sdk_options)
-        return response["Content"].read()  # read() of botocore.response.StreamingBody
+        return response["Content"].read().decode("utf-8")  # read() of botocore.response.StreamingBody
 
     def _get_multiple(self, path: str, **sdk_options) -> Dict[str, str]:
         """
@@ -121,7 +121,7 @@ def get_app_config(
     force_fetch: bool = False,
     max_age: int = DEFAULT_MAX_AGE_SECS,
     **sdk_options
-) -> Union[str, list, dict, bytes]:
+) -> Union[str, list, dict]:
     """
     Retrieve a configuration value from AWS App Config.
 

tests/functional/test_utilities_parameters.py

@@ -1503,9 +1503,8 @@ def test_appconf_provider_get_configuration_no_transform(mock_name, config):
     stubber.activate()
 
     try:
-        value = provider.get(mock_name)
-        str_value = value.decode("utf-8")
-        assert str_value == json.dumps(mock_body_json)
+        value: str = provider.get(mock_name)
+        assert value == json.dumps(mock_body_json)
         stubber.assert_no_pending_responses()
     finally:
         stubber.deactivate()