summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorJafar Al-Gharaibeh <jafar@atcorp.com>2025-01-20 20:31:47 +0100
committerGitHub <noreply@github.com>2025-01-20 20:31:47 +0100
commit4f65107d20901b136aa158b18e484a15df166cf0 (patch)
tree841f555b6cc7b4f96d23c967619b788d5f920e04 /tests
parentMerge pull request #17882 from LabNConsulting/chopps/doc-update-mgmtd (diff)
parenttests: Fix test_bgp_dynamic_capability_enhe topotest (diff)
downloadfrr-4f65107d20901b136aa158b18e484a15df166cf0.tar.xz
frr-4f65107d20901b136aa158b18e484a15df166cf0.zip
Merge pull request #17883 from opensourcerouting/fix/topotest_dynamic_cap_enhe
tests: Fix test_bgp_dynamic_capability_enhe topotest
Diffstat (limited to 'tests')
-rw-r--r--tests/topotests/bgp_dynamic_capability/test_bgp_dynamic_capability_enhe.py28
1 files changed, 1 insertions, 27 deletions
diff --git a/tests/topotests/bgp_dynamic_capability/test_bgp_dynamic_capability_enhe.py b/tests/topotests/bgp_dynamic_capability/test_bgp_dynamic_capability_enhe.py
index aa0508e88..fd467b8c3 100644
--- a/tests/topotests/bgp_dynamic_capability/test_bgp_dynamic_capability_enhe.py
+++ b/tests/topotests/bgp_dynamic_capability/test_bgp_dynamic_capability_enhe.py
@@ -114,7 +114,7 @@ def test_bgp_dynamic_capability_enhe():
# Clear message stats to check if we receive a notification or not after we
# change the role.
- r1.vtysh_cmd("clear bgp 2001:db8::2 message-stats")
+ r2.vtysh_cmd("clear bgp 2001:db8::1 message-stats")
r1.vtysh_cmd(
"""
configure terminal
@@ -149,32 +149,6 @@ def test_bgp_dynamic_capability_enhe():
_, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
assert result is None, "Session was reset after setting ENHE capability"
- def _bgp_check_if_session_not_reset():
- output = json.loads(r2.vtysh_cmd("show bgp neighbor 2001:db8::1 json"))
- expected = {
- "2001:db8::1": {
- "bgpState": "Established",
- "neighborCapabilities": {
- "dynamic": "advertisedAndReceived",
- "extendedNexthop": "advertisedAndReceived",
- "extendedNexthopFamililesByPeer": {
- "ipv4Unicast": "recieved",
- },
- },
- "messageStats": {
- "notificationsRecv": 0,
- "capabilityRecv": 1,
- },
- }
- }
- return topotest.json_cmp(output, expected)
-
- test_func = functools.partial(
- _bgp_check_if_session_not_reset,
- )
- _, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
- assert result is None, "Session was reset after setting ENHE capability"
-
def _bgp_check_nexthop_enhe():
output = json.loads(r1.vtysh_cmd("show ip route 10.10.10.10/32 json"))
expected = {