Skip to content

Commit 6ede943

Browse files
oxygen980falodiya
andauthored
Modified testcase to use enum_rand_one_per_hwsku_frontend_hostname (#2954)
Description of PR Modified route/test_route_perf.py to use 'enum_rand_one_per_hwsku_frontend_hostname' instead of 'rand_one_dut_hostname' Approach What is the motivation for this PR? The testcase was already modified to use 'enum_rand_one_per_hwsku_frontend_hostname' but was missing in some functions, This PR takes care of that. How did you do it Replaced 'rand_one_dut_hostname' with 'enum_rand_one_per_hwsku_frontend_hostname' Co-authored-by: falodiya <renu.falodiy@nokia.com>
1 parent 1081f70 commit 6ede943

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

tests/route/test_route_perf.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
ROUTE_TABLE_NAME = 'ASIC_STATE:SAI_OBJECT_TYPE_ROUTE_ENTRY'
2121

2222
@pytest.fixture(autouse=True)
23-
def ignore_expected_loganalyzer_exceptions(rand_one_dut_hostname, loganalyzer):
23+
def ignore_expected_loganalyzer_exceptions(enum_rand_one_per_hwsku_frontend_hostname, loganalyzer):
2424
"""
2525
Ignore expected failures logs during test execution.
2626
@@ -38,7 +38,7 @@ def ignore_expected_loganalyzer_exceptions(rand_one_dut_hostname, loganalyzer):
3838
]
3939
if loganalyzer:
4040
# Skip if loganalyzer is disabled
41-
loganalyzer[rand_one_dut_hostname].ignore_regex.extend(ignoreRegex)
41+
loganalyzer[enum_rand_one_per_hwsku_frontend_hostname].ignore_regex.extend(ignoreRegex)
4242

4343
@pytest.fixture(params=[4, 6])
4444
def ip_versions(request):
@@ -189,8 +189,8 @@ def _check_num_routes(expected_num_routes):
189189
# Retuen time used for set/del routes
190190
return (end_time - start_time).total_seconds()
191191

192-
def test_perf_add_remove_routes(duthosts, rand_one_dut_hostname, request, ip_versions):
193-
duthost = duthosts[rand_one_dut_hostname]
192+
def test_perf_add_remove_routes(duthosts, enum_rand_one_per_hwsku_frontend_hostname, request, ip_versions):
193+
duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]
194194
# Number of routes for test
195195
set_num_routes = request.config.getoption("--num_routes")
196196

0 commit comments

Comments
 (0)