Skip to content

Commit 916e783

Browse files
committed
Actually use pytest.raises in test.
1 parent 99692a9 commit 916e783

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

tests/unit/test_serialize.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,5 @@ def test_deserialize_new_json_cassette():
3131
side_effect=UnicodeDecodeError('utf-8', b'unicode error in serialization',
3232
0, 10, 'blew up'))
3333
def test_serialize_constructs_UnicodeDecodeError(mock_dumps):
34-
jsonserializer.serialize({})
34+
with pytest.raises(UnicodeDecodeError):
35+
jsonserializer.serialize({})

vcr/serializers/jsonserializer.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def serialize(cassette_dict):
1717
b"Error serializing cassette to JSON",
1818
original.start,
1919
original.end,
20-
original.message +
20+
original.args[-1] +
2121
("Does this HTTP interaction contain binary data? "
2222
"If so, use a different serializer (like the yaml serializer) "
2323
"for this request?")

0 commit comments

Comments
 (0)