From 0b711f0a20c5c5c57ae05161d817de0dcf0d16e8 Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Fri, 9 Apr 2021 09:01:17 -0400 Subject: [PATCH] tests: test_bgp_multi_vrf_topo1.py fix formatting issues pointed out by pylint Signed-off-by: Donald Sharp --- .../test_bgp_multi_vrf_topo1.py | 19 ++++++------------- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py b/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py index 5ecaee2ece..1b62d986bd 100644 --- a/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py +++ b/tests/topotests/bgp_multi_vrf_topo1/test_bgp_multi_vrf_topo1.py @@ -2677,14 +2677,12 @@ def test_route_map_within_vrf_to_alter_bgp_attribute_nexthop_p0(request): result = verify_rib(tgen, addr_type, dut, input_dict_1, expected=False) assert ( result is not True - ), "Testcase {} : Failed \n Expected Behaviour: Routes are rejected because" - " nexthop-self config is deleted \n Error {}".format(tc_name, result) + ), "Testcase {} : Failed \n Expected Behaviour: Routes are rejected because nexthop-self config is deleted \n Error {}".format(tc_name, result) result = verify_rib(tgen, addr_type, dut, input_dict_2, expected=False) assert ( result is not True - ), "Testcase {} : Failed \n Expected Behaviour: Routes are rejected because" - " nexthop-self config is deleted \n Error {}".format(tc_name, result) + ), "Testcase {} : Failed \n Expected Behaviour: Routes are rejected because nexthop-self config is deleted \n Error {}".format(tc_name, result) write_test_footer(tc_name) @@ -4958,8 +4956,7 @@ def test_prefix_list_to_permit_deny_prefixes_p0(request): result = verify_rib(tgen, addr_type, dut, denied_routes, expected=False) assert result is not True, "Testcase {} : Failed \n" - "Expected behaviour: Routes are denied by prefix-list \n" - "Error {}".format(tc_name, result) + "{}:Expected behaviour: Routes are denied by prefix-list \nError {}".format(tc_name, result) step( "On router R1, configure prefix-lists to permit 2 " @@ -5169,9 +5166,7 @@ def test_prefix_list_to_permit_deny_prefixes_p0(request): ) result = verify_rib(tgen, addr_type, dut, denied_routes, expected=False) - assert result is not True, "Testcase {} : Failed \n" - "Expected behaviour: Routes are denied by prefix-list \n" - "Error {}".format(tc_name, result) + assert result is not True, "Testcase {} : Failed \nExpected behaviour: Routes are denied by prefix-list \nError {}".format(tc_name, result) write_test_footer(tc_name) @@ -5449,8 +5444,7 @@ def test_route_map_set_and_match_tag_p0(request): result = verify_rib(tgen, addr_type, dut, input_dict_2, expected=False) assert ( result is not True - ), "Testcase {} : Failed \n Expected Behavior: Routes are denied \n" - "Error {}".format(tc_name, result) + ), "Testcase {} : Failed \n Expected Behavior: Routes are denied \nError {}".format(tc_name, result) write_test_footer(tc_name) @@ -5853,8 +5847,7 @@ def test_route_map_set_and_match_metric_p0(request): result = verify_rib(tgen, addr_type, dut, input_dict_2, expected=False) assert ( result is not True - ), "Testcase {} : Failed \n Expected Behavior: Routes are denied \n" - "Error {}".format(tc_name, result) + ), "Testcase {} : Failed \n Expected Behavior: Routes are denied \nError {}".format(tc_name, result) write_test_footer(tc_name) -- 2.39.5