pep8

resource_management/hardware/juniper.py

@@ -293,7 +293,7 @@ def _load_pic_connected_ports(
         yield _p
 
 
-def _load_aggregates(router: 'jnpr.junos.Device'):
+def load_aggregates(router: 'jnpr.junos.Device'):
     """
     computes 802.3ad lag bundles
 
@@ -336,7 +336,7 @@ def _load_aggregates(router: 'jnpr.junos.Device'):
     return bundles
 
 
-def _load_logical_interfaces(router: 'jnpr.junos.Device'):
+def load_logical_interfaces(router: 'jnpr.junos.Device'):
     """
     loads logical interfaces
 
@@ -376,7 +376,7 @@ def _load_logical_interfaces(router: 'jnpr.junos.Device'):
     return interfaces
 
 
-def _load_installed_ethernet_ports(router: 'jnpr.junos.Device'):
+def load_installed_ethernet_ports(router: 'jnpr.junos.Device'):
     """
     can't just use jnpr.junos.op.ethport.EthPortTable
 

test/test_parse_router.py

@@ -67,10 +67,9 @@ def test_new_load_interfaces(mocked_router, netconf_rpc_replies):
         #     print(len(config))
 
         # just sanity check there are non-empty responses
-        physical = list(juniper._load_installed_ethernet_ports(dev))
+        physical = list(juniper.load_installed_ethernet_ports(dev))
         assert physical
-        logical = juniper._load_logical_interfaces(dev)
+        logical = juniper.load_logical_interfaces(dev)
         assert logical
-        aggregates = juniper._load_aggregates(dev)
+        aggregates = juniper.load_aggregates(dev)
         assert aggregates
-