From: Donald Sharp Date: Fri, 8 Jan 2021 21:52:41 +0000 (-0500) Subject: tests: Wait for bgp convergence *after* we re-add the interfaces X-Git-Tag: base_7.6~75^2 X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=refs%2Fpull%2F7842%2Fhead;p=mirror%2Ffrr.git tests: Wait for bgp convergence *after* we re-add the interfaces In test_bgp_mutli_vrf_topo2.py it's clear that we remove then re-add the vrf interfaces. Then the test was immediately checking to ensure that the routes were available. BGP needs time to reconverge. Let's ensure that first. Signed-off-by: Donald Sharp --- diff --git a/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py b/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py index d34446e2ee..19a9140c13 100644 --- a/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py +++ b/tests/topotests/bgp_multi_vrf_topo2/test_bgp_multi_vrf_topo2.py @@ -2687,6 +2687,9 @@ def test_delete_and_re_add_vrf_p1(request): } } + result = verify_bgp_convergence(tgen, topo) + assert result is True, "Testcase {}: Failed\n Error {}".format(tc_name, result) + result = verify_bgp_rib(tgen, addr_type, dut, input_dict_2) assert result is True, "Testcase {} :Failed \n Error {}".format(tc_name, result)