Merge branch 'zwei' into deprecate-util-functions

Author: ajaykarpur

src/sagemaker/deserializers.py

@@ -266,7 +266,8 @@ def deserialize(self, stream, content_type):
             list: A list of JSON serializable objects.
         """
         try:
-            lines = stream.read().rstrip().split("\n")
+            body = stream.read().decode("utf-8")
+            lines = body.rstrip().split("\n")
             return [json.loads(line) for line in lines]
         finally:
             stream.close()

tests/unit/sagemaker/test_deserializers.py

@@ -219,16 +219,16 @@ def json_lines_deserializer():
 @pytest.mark.parametrize(
     "source, expected",
     [
-        ('["Name", "Score"]\n["Gilbert", 24]', [["Name", "Score"], ["Gilbert", 24]]),
-        ('["Name", "Score"]\n["Gilbert", 24]\n', [["Name", "Score"], ["Gilbert", 24]]),
+        (b'["Name", "Score"]\n["Gilbert", 24]', [["Name", "Score"], ["Gilbert", 24]]),
+        (b'["Name", "Score"]\n["Gilbert", 24]\n', [["Name", "Score"], ["Gilbert", 24]]),
         (
-            '{"Name": "Gilbert", "Score": 24}\n{"Name": "Alexa", "Score": 29}',
+            b'{"Name": "Gilbert", "Score": 24}\n{"Name": "Alexa", "Score": 29}',
             [{"Name": "Gilbert", "Score": 24}, {"Name": "Alexa", "Score": 29}],
         ),
     ],
 )
 def test_json_lines_deserializer(json_lines_deserializer, source, expected):
-    stream = io.StringIO(source)
+    stream = io.BytesIO(source)
     content_type = "application/jsonlines"
     actual = json_lines_deserializer.deserialize(stream, content_type)
     assert actual == expected