move asserts out of round trip test utils

Author: silmeth

tests/snippets/exceptions.py

@@ -2,22 +2,22 @@ def exceptions_eq(e1, e2):
     return type(e1) is type(e2) and e1.args == e2.args
 
 def round_trip_repr(e):
-    assert exceptions_eq(e, eval(repr(e)))
+    return exceptions_eq(e, eval(repr(e)))
 
 # KeyError
 empty_exc = KeyError()
 assert str(empty_exc) == ''
-round_trip_repr(empty_exc)
+assert round_trip_repr(empty_exc)
 assert len(empty_exc.args) == 0
 assert type(empty_exc.args) == tuple
 
 exc = KeyError('message')
 assert str(exc) == "'message'"
-round_trip_repr(exc)
+assert round_trip_repr(exc)
 
 exc = KeyError('message', 'another message')
 assert str(exc) == "('message', 'another message')"
-round_trip_repr(exc)
+assert round_trip_repr(exc)
 assert exc.args[0] == 'message'
 assert exc.args[1] == 'another message'
 
@@ -31,7 +31,7 @@ def __eq__(self, other):
 
 exc = KeyError(A())
 assert str(exc) == 'A()'
-round_trip_repr(exc)
+assert round_trip_repr(exc)
 
 # ImportError / ModuleNotFoundError
 exc = ImportError()

tests/snippets/json_snippet.py

@@ -5,7 +5,7 @@ def round_trip_test(obj):
     # serde_json and Python's json module produce slightly differently spaced
     # output; direct string comparison can't pass on both so we use this as a
     # proxy
-    assert obj == json.loads(json.dumps(obj))
+    return obj == json.loads(json.dumps(obj))
 
 assert '"string"' == json.dumps("string")
 assert "1" == json.dumps(1)
@@ -17,7 +17,7 @@ def round_trip_test(obj):
 assert '[]' == json.dumps([])
 assert '[1]' == json.dumps([1])
 assert '[[1]]' == json.dumps([[1]])
-round_trip_test([1, "string", 1.0, True])
+assert round_trip_test([1, "string", 1.0, True])
 
 assert '[]' == json.dumps(())
 assert '[1]' == json.dumps((1,))
@@ -26,7 +26,7 @@ def round_trip_test(obj):
 assert [1, "string", 1.0, True] == json.loads(json.dumps((1, "string", 1.0, True)))
 
 assert '{}' == json.dumps({})
-round_trip_test({'a': 'b'})
+assert round_trip_test({'a': 'b'})
 
 # should reject non-str keys in jsons
 assert_raises(json.JSONDecodeError, lambda: json.loads('{3: "abc"}'))