From f8d9a3426e2d3b19ab33b539149e311e04cdbc93 Mon Sep 17 00:00:00 2001 From: Dashuai Zhang Date: Wed, 13 Nov 2024 00:09:29 +1100 Subject: [PATCH 1/2] change enum with completeness_level --- .../pfc/test_multidut_pfc_pause_lossless_with_snappi.py | 1 - .../multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py index efb0bca8254..911b0f2c3ae 100644 --- a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py +++ b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py @@ -147,7 +147,6 @@ def test_pfc_pause_single_lossless_prio_reboot(snappi_api, # n all_prio_list, # noqa: F811 get_snappi_ports, # noqa: F811 tbinfo, # noqa: F811 - enum_dut_lossless_prio, # noqa: F811 setup_ports_and_dut, # noqa: F811 disable_pfcwd, # noqa: F811 reboot_duts): # noqa: F811 diff --git a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py index 8a03b72ac0e..475d2d49acd 100644 --- a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py +++ b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py @@ -136,7 +136,7 @@ def test_pfc_pause_single_lossy_prio_reboot(snappi_api, # noqa: F811 fanout_graph_facts_multidut, # noqa: F811 duthosts, localhost, - enum_dut_lossy_prio, + enum_dut_lossy_prio_with_completeness_level, prio_dscp_map, # noqa: F811 lossy_prio_list, # noqa: F811 all_prio_list, # noqa: F811 @@ -166,7 +166,7 @@ def test_pfc_pause_single_lossy_prio_reboot(snappi_api, # noqa: F811 """ testbed_config, port_config_list, snappi_ports = setup_ports_and_dut - _, lossy_prio = enum_dut_lossy_prio.split('|') + _, lossy_prio = enum_dut_lossy_prio_with_completeness_level.split('|') lossy_prio = int(lossy_prio) pause_prio_list = [lossy_prio] test_prio_list = [lossy_prio] From 7302a1fb6243b2d9877d0c9de6fb1d610c8e7b94 Mon Sep 17 00:00:00 2001 From: Dashuai Zhang Date: Thu, 14 Nov 2024 00:34:40 +1100 Subject: [PATCH 2/2] fix comment --- .../pfc/test_multidut_pfc_pause_lossless_with_snappi.py | 4 ++-- .../multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py index 911b0f2c3ae..bc131deb4fc 100644 --- a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py +++ b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossless_with_snappi.py @@ -159,10 +159,10 @@ def test_pfc_pause_single_lossless_prio_reboot(snappi_api, # n fanout_graph_facts_multidut (pytest fixture): fanout graph duthosts (pytest fixture): list of DUTs localhost (pytest fixture): localhost handle - enum_dut_lossless_prio (str): lossless priority to test, e.g., 's6100-1|3' + enum_dut_lossless_prio_with_completeness_level (str): lossless priority to test, e.g., 's6100-1|3' all_prio_list (pytest fixture): list of all the priorities prio_dscp_map (pytest fixture): priority vs. DSCP map (key = priority). - enum_dut_lossless_prio_with_completeness_level (str): lossless priority to test, e.g., 's6100-1|3' + lossless_prio_list (pytest fixture): list of all the lossless priorities tbinfo (pytest fixture): fixture provides information about testbed get_snappi_ports (pytest fixture): gets snappi ports and connected DUT port info and returns as a list Returns: diff --git a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py index 475d2d49acd..e44c5a86de1 100644 --- a/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py +++ b/tests/snappi_tests/multidut/pfc/test_multidut_pfc_pause_lossy_with_snappi.py @@ -154,7 +154,7 @@ def test_pfc_pause_single_lossy_prio_reboot(snappi_api, # noqa: F811 fanout_graph_facts_multidut (pytest fixture): fanout graph duthosts (pytest fixture): list of DUTs localhost (pytest fixture): localhost handle - enum_dut_lossy_prio (str): name of lossy priority to test, e.g., 's6100-1|2' + enum_dut_lossy_prio_with_completeness_level (str): name of lossy priority to test, e.g., 's6100-1|2' prio_dscp_map (pytest fixture): priority vs. DSCP map (key = priority). lossy_prio_list (pytest fixture): list of all the lossy priorities all_prio_list (pytest fixture): list of all the priorities