proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-07-27 11:44:16 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

tests: Clean up some logging in test_bgp_default_originate_2links.py

Browse Source 
Test was confusing.  Add some useful data and clean up some debugs

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
This commit is contained in:
Donald Sharp 2024-09-30 14:57:45 -04:00
parent 6e0fe595f1
commit 6ff85fc748
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
tests/topotests/bgp_default_originate/test_bgp_default_originate_2links.py
View File

@ -266,21 +266,21 @@ def verify_the_uptime(time_stamp_before, time_stamp_after, incremented=None):
if incremented == True:
if uptime_before < uptime_after:
logger.info(
" The Uptime [{}] is incremented than [{}].......PASSED ".format(
" The Uptime before [{}] is less than [{}].......PASSED ".format(
time_stamp_before, time_stamp_after
)
)
return True
else:
logger.error(
" The Uptime [{}] is expected to be incremented than [{}].......FAILED ".format(
" The Uptime before [{}] is greater than the uptime after [{}].......FAILED ".format(
time_stamp_before, time_stamp_after
)
)
return False
else:
logger.info(
" The Uptime [{}] is not incremented than [{}] ".format(
" The Uptime before [{}] the same as after [{}] ".format(
time_stamp_before, time_stamp_after
)
)
@ -1027,7 +1027,7 @@ def test_verify_bgp_default_originate_with_default_static_route_p1(request):
result = verify_the_uptime(uptime_before_ipv6, uptime_after_ipv6, incremented=False)
assert result is True, "Testcase {} : Failed Error: {}".format(tc_name, result)
step("Taking uptime snapshot before removing redisctribute static ")
step("Taking uptime snapshot before removing redistribute static")
uptime_before_ipv4 = get_rib_route_uptime(tgen, "ipv4", "r2", ipv4_uptime_dict)
uptime_before_ipv6 = get_rib_route_uptime(tgen, "ipv6", "r2", ipv6_uptime_dict)
sleep(1)
@ -1074,6 +1074,7 @@ def test_verify_bgp_default_originate_with_default_static_route_p1(request):
)
assert result is True, "Testcase {} : Failed Error: {}".format(tc_name, result)
step("Now look that the route is not pointed at link2")
result = verify_rib_default_route(
tgen,
topo,
@ -1093,7 +1094,7 @@ def test_verify_bgp_default_originate_with_default_static_route_p1(request):
)
assert result is not True, "Testcase {} : Failed Error: {}".format(tc_name, result)
step("Taking uptime snapshot before removing redisctribute static ")
step("Taking uptime snapshot after removing redistribute static")
uptime_after_ipv4 = get_rib_route_uptime(tgen, "ipv4", "r2", ipv4_uptime_dict)
uptime_after_ipv6 = get_rib_route_uptime(tgen, "ipv6", "r2", ipv6_uptime_dict)