Skip to content

Commit 2df7f51

Browse files
committed
tests: minor fixes for retry test
The comment describes the inverse of what we're checking, so fix that. While we're here, fix the no-EOL-EOF, and no need to sort an empty list. Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
1 parent a6bfe82 commit 2df7f51

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

tests/test_mctpd.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1419,13 +1419,12 @@ async def test_query_peer_properties_retry_timeout(nursery, dbus, sysnet):
14191419
# call setup_endpoint again, which will trigger query of peer properties
14201420
(eid, net, path, new) = await mctp.call_setup_endpoint(ep.lladdr)
14211421

1422-
# timeout five times does not prevent us from getting the correct message types
1422+
# timeout five times does prevent us from getting the correct message types
14231423
objep = await mctpd_mctp_endpoint_common_obj(dbus, path)
14241424
objtypes = list(await objep.get_supported_message_types())
1425-
objtypes.sort()
14261425
expected_types = [] # exceeded retry limit, so no types known
14271426
assert objtypes == expected_types
14281427

14291428
# exit mctpd
14301429
res = await mctpd.stop_mctpd()
1431-
assert res == 0
1430+
assert res == 0

0 commit comments

Comments
 (0)