Skip to content

Commit e7037cf

Browse files
author
Jussi Kukkonen
authored
Merge pull request #1860 from MVrachev/serialization-bytes-array
Metadata test full serialization cycle
2 parents 29e4e63 + be2c8f0 commit e7037cf

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

tests/test_metadata_serialization.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,15 @@ def test_invalid_metadata_serialization(self, test_data: bytes) -> None:
8080
@utils.run_sub_tests_with_dataset(valid_metadata)
8181
def test_valid_metadata_serialization(self, test_case_data: bytes) -> None:
8282
md = Metadata.from_bytes(test_case_data)
83-
input_dict = json.loads(test_case_data)
84-
self.assertDictEqual(input_dict, md.to_dict())
83+
84+
# Convert to a JSON and sort the keys the way we do in JSONSerializer.
85+
separators = (",", ":")
86+
test_json = json.loads(test_case_data)
87+
test_bytes = json.dumps(
88+
test_json, separators=separators, sort_keys=True
89+
).encode("utf-8")
90+
91+
self.assertEqual(test_bytes, md.to_bytes())
8592

8693
invalid_signatures: utils.DataSet = {
8794
"missing keyid attribute in a signature": '{ "sig": "abc" }',

0 commit comments

Comments
 (0)