tests: use a fixture for setting up a routed endpoint

We have a few cases of boilerplate in the test code, where we're setting
up an endpoint to communicate with mctpd.

Use a fixure - routed_ep - for this pattern, and replace the
boilerplate.

Signed-off-by: Jeremy Kerr <[email protected]>

Author: jk-ozlabs

tests/test_mctpd.py

@@ -1405,30 +1405,17 @@ async def test_query_peer_properties_retry_timeout(nursery, dbus, sysnet):
     assert res == 0
 
 """ Test RegisterVDMTypeSupport when no responders are registered """
-async def test_register_vdm_type_support_empty(mctpd):
-    ep = mctpd.network.endpoints[0]
-    ep.eid = 12
-    iface = mctpd.system.interfaces[0]
-    await mctpd.system.add_route(mctpd.system.Route(ep.eid, 1, iface=iface))
-    await mctpd.system.add_neighbour(
-        mctpd.system.Neighbour(iface, ep.lladdr, ep.eid)
-    )
+async def test_register_vdm_type_support_empty(mctpd, routed_ep):
+    ep = routed_ep
 
     # Verify error response when no VDM is registered
     cmd = MCTPControlCommand(True, 0, 0x06, bytes([0x00]))
     rsp = await ep.send_control(mctpd.network.mctp_socket, cmd)
     assert rsp.hex(' ') == '00 06 02'
 
 """ Test RegisterVDMTypeSupport when a single PCIe VDM is registered """
-async def test_register_vdm_type_support_pcie_only(dbus, mctpd):
-    ep = mctpd.network.endpoints[0]
-    ep.eid = 12
-    iface = mctpd.system.interfaces[0]
-    await mctpd.system.add_route(mctpd.system.Route(ep.eid, 1, iface=iface))
-    await mctpd.system.add_neighbour(
-        mctpd.system.Neighbour(iface, ep.lladdr, ep.eid)
-    )
-
+async def test_register_vdm_type_support_pcie_only(dbus, mctpd, routed_ep):
+    ep = routed_ep
     mctp = await mctpd_mctp_base_iface_obj(dbus)
 
     # Register PCIe VDM: format=0x00, VID=0xABCD, command_set=0x0001
@@ -1446,15 +1433,8 @@ async def test_register_vdm_type_support_pcie_only(dbus, mctpd):
     assert rsp.hex(' ') == '00 06 02'
 
 """ Test RegisterVDMTypeSupport when a single IANA VDM is registered """
-async def test_register_vdm_type_support_iana_only(dbus, mctpd):
-    ep = mctpd.network.endpoints[0]
-    ep.eid = 12
-    iface = mctpd.system.interfaces[0]
-    await mctpd.system.add_route(mctpd.system.Route(ep.eid, 1, iface=iface))
-    await mctpd.system.add_neighbour(
-        mctpd.system.Neighbour(iface, ep.lladdr, ep.eid)
-    )
-
+async def test_register_vdm_type_support_iana_only(dbus, mctpd, routed_ep):
+    ep = routed_ep
     mctp = await mctpd_mctp_base_iface_obj(dbus)
 
     # Register IANA VDM: format=0x01, VID=0x1234ABCD, command_set=0x5678
@@ -1467,14 +1447,8 @@ async def test_register_vdm_type_support_iana_only(dbus, mctpd):
     assert rsp.hex(' ') == '00 06 00 ff 01 12 34 ab cd 56 78'
 
 """ Test RegisterVDMTypeSupport with dbus disconnect """
-async def test_register_vdm_type_support_dbus_disconnect(mctpd):
-    ep = mctpd.network.endpoints[0]
-    ep.eid = 12
-    iface = mctpd.system.interfaces[0]
-    await mctpd.system.add_route(mctpd.system.Route(ep.eid, 1, iface=iface))
-    await mctpd.system.add_neighbour(
-        mctpd.system.Neighbour(iface, ep.lladdr, ep.eid)
-    )
+async def test_register_vdm_type_support_dbus_disconnect(mctpd, routed_ep):
+    ep = routed_ep
 
     # Verify error response when no VDM is registered
     cmd = MCTPControlCommand(True, 0, 0x06, bytes([0x00]))
@@ -1503,15 +1477,8 @@ async def test_register_vdm_type_support_dbus_disconnect(mctpd):
 
 """ Test RegisterVDMTypeSupport error handling """
 async def test_register_vdm_type_support_errors(dbus, mctpd):
-    ep = mctpd.network.endpoints[0]
-    ep.eid = 12
-    iface = mctpd.system.interfaces[0]
-    await mctpd.system.add_route(mctpd.system.Route(ep.eid, 1, iface=iface))
-    await mctpd.system.add_neighbour(
-        mctpd.system.Neighbour(iface, ep.lladdr, ep.eid)
-    )
-
     mctp = await mctpd_mctp_base_iface_obj(dbus)
+
     # Verify DBus call fails with invalid format 0x02
     v_type = asyncdbus.Variant('q', 0xABCD)
     with pytest.raises(asyncdbus.errors.DBusError) as ex: