Skip to content

Commit

Permalink
Change back to only cisco-8000 check, and xfail instead of skip.
Browse files Browse the repository at this point in the history
  • Loading branch information
rraghav-cisco committed Dec 13, 2024
1 parent ff67b46 commit 44fe9b3
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions tests/pfcwd/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -252,8 +252,7 @@ def skip_pfcwd_higher_speeds(
enum_rand_one_per_hwsku_frontend_hostname,
setup_pfc_test):
duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]
if duthost.facts["asic_type"] != "cisco-8000" and \
not duthost.get_facts().get("modular_chassis", None):
if duthost.facts["asic_type"] != "cisco-8000":
yield
return

Expand All @@ -263,7 +262,7 @@ def skip_pfcwd_higher_speeds(
"show interfaces status {}".format(test_port))[0]['speed']
speed_int = speed.split("G")[0]
if int(speed_int) > 100:
pytest.skip(
pytest.xfail(
"The tests using pfc_gen or pfc_gen_t2 can be run " +
"only for speeds less than " +
f"100Gbps(for:{enum_rand_one_per_hwsku_frontend_hostname})")
Expand Down

0 comments on commit 44fe9b3

Please sign in to comment.