From 228db6d8a650adb4d47dbc1ac66d4eedf4e659c2 Mon Sep 17 00:00:00 2001
From: Erik Reid <erik.reid@geant.org>
Date: Sun, 19 Mar 2023 14:04:40 +0100
Subject: [PATCH] pep8
---
resource_management/hardware/juniper.py | 6 +++---
test/test_parse_router.py | 7 +++----
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/resource_management/hardware/juniper.py b/resource_management/hardware/juniper.py
index 6f00f5f..460f4e4 100644
--- a/resource_management/hardware/juniper.py
+++ b/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
diff --git a/test/test_parse_router.py b/test/test_parse_router.py
index 87359d6..ca4ae63 100644
--- a/test/test_parse_router.py
+++ b/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
-
--
GitLab