Skip to content

Commit

Permalink
Merge branch 'develop' into arp-vrf
Browse files Browse the repository at this point in the history
  • Loading branch information
mirceaulinic committed Nov 15, 2017
2 parents 243d190 + 4db8bcf commit 83a9dc5
Show file tree
Hide file tree
Showing 16 changed files with 21 additions and 164 deletions.
1 change: 1 addition & 0 deletions napalm/_SUPPORTED_DRIVERS.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
SUPPORTED_DRIVERS = [
"base",
"eos",
"ios",
"iosxr",
Expand Down
12 changes: 6 additions & 6 deletions napalm/base/validate.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ def _compare_getter_list(src, dst, mode):
i = 0
while True:
try:
intermediate_match = _compare_getter(src_element, dst[i])
intermediate_match = compare(src_element, dst[i])
if isinstance(intermediate_match, dict) and intermediate_match["complies"] or \
not isinstance(intermediate_match, dict) and intermediate_match:
found = True
Expand Down Expand Up @@ -79,7 +79,7 @@ def _compare_getter_dict(src, dst, mode):
try:
dst_element = dst.pop(key)
result["present"][key] = {}
intermediate_result = _compare_getter(src_element, dst_element)
intermediate_result = compare(src_element, dst_element)

if isinstance(intermediate_result, dict):
nested = True
Expand Down Expand Up @@ -111,7 +111,7 @@ def _compare_getter_dict(src, dst, mode):
return result


def _compare_getter(src, dst):
def compare(src, dst):
if isinstance(src, py23_compat.string_types):
src = py23_compat.text_type(src)

Expand All @@ -127,7 +127,7 @@ def _compare_getter(src, dst):

elif isinstance(src, py23_compat.string_types):
if src.startswith('<') or src.startswith('>'):
cmp_result = compare_numeric(src, dst)
cmp_result = _compare_numeric(src, dst)
return cmp_result
else:
m = re.search(src, py23_compat.text_type(dst))
Expand All @@ -147,7 +147,7 @@ def _compare_getter(src, dst):
return src == dst


def compare_numeric(src_num, dst_num):
def _compare_numeric(src_num, dst_num):
"""Compare numerical values. You can use '<%d','>%d'."""
dst_num = float(dst_num)

Expand Down Expand Up @@ -191,7 +191,7 @@ def compliance_report(cls, validation_file=None, validation_source=None):
try:
kwargs = expected_results.pop('_kwargs', {})
actual_results = getattr(cls, getter)(**kwargs)
report[key] = _compare_getter(expected_results, actual_results)
report[key] = compare(expected_results, actual_results)
except NotImplementedError:
report[key] = {"skipped": True, "reason": "NotImplemented"}

Expand Down
22 changes: 11 additions & 11 deletions test/base/validate/test_unit.py
Original file line number Diff line number Diff line change
Expand Up @@ -282,21 +282,21 @@ class TestValidate:
@pytest.mark.parametrize('src, dst, result', _compare_getter)
def test__compare_getter_list(self, src, dst, result):
"""Test for _compare_getter_list."""
assert validate._compare_getter(src, dst) == result
assert validate.compare(src, dst) == result

def test_numeric_comparison(self):
assert validate.compare_numeric("<2", 1)
assert not validate.compare_numeric("<2", 3)
assert validate.compare_numeric("<=2", 2)
assert validate.compare_numeric("<3", "2")
assert validate.compare_numeric("!=3", "2")
assert validate._compare_numeric("<2", 1)
assert not validate._compare_numeric("<2", 3)
assert validate._compare_numeric("<=2", 2)
assert validate._compare_numeric("<3", "2")
assert validate._compare_numeric("!=3", "2")
with pytest.raises(ValueError):
assert validate.compare_numeric("a2a", 2)
assert validate._compare_numeric("a2a", 2)
with pytest.raises(ValueError):
assert validate.compare_numeric("<1a1", 2)
assert validate._compare_numeric("<1a1", 2)
with pytest.raises(ValueError):
assert validate.compare_numeric("a<1", 2)
assert validate._compare_numeric("a<1", 2)
with pytest.raises(ValueError):
assert validate.compare_numeric("<1", "asdasd2")
assert validate._compare_numeric("<1", "asdasd2")
with pytest.raises(ValueError):
assert validate.compare_numeric("<1", "2asdasd")
assert validate._compare_numeric("<1", "2asdasd")
1 change: 1 addition & 0 deletions test/eos/utils/textfsm_templates
46 changes: 0 additions & 46 deletions test/eos/utils/textfsm_templates/bgp_detail.tpl

This file was deleted.

6 changes: 0 additions & 6 deletions test/eos/utils/textfsm_templates/ntp_peers.tpl

This file was deleted.

14 changes: 0 additions & 14 deletions test/eos/utils/textfsm_templates/snmp_config.tpl

This file was deleted.

17 changes: 0 additions & 17 deletions test/eos/utils/textfsm_templates/vrf.tpl

This file was deleted.

1 change: 1 addition & 0 deletions test/nxos/utils/textfsm_templates
9 changes: 0 additions & 9 deletions test/nxos/utils/textfsm_templates/lldp_neighbors.tpl

This file was deleted.

10 changes: 0 additions & 10 deletions test/nxos/utils/textfsm_templates/snmp_config.tpl

This file was deleted.

13 changes: 0 additions & 13 deletions test/nxos/utils/textfsm_templates/users.tpl

This file was deleted.

1 change: 1 addition & 0 deletions test/nxos_ssh/utils/textfsm_templates
9 changes: 0 additions & 9 deletions test/nxos_ssh/utils/textfsm_templates/lldp_neighbors.tpl

This file was deleted.

10 changes: 0 additions & 10 deletions test/nxos_ssh/utils/textfsm_templates/snmp_config.tpl

This file was deleted.

13 changes: 0 additions & 13 deletions test/nxos_ssh/utils/textfsm_templates/users.tpl

This file was deleted.

0 comments on commit 83a9dc5

Please sign in to comment.