Skip to content

Commit

Permalink
Refactor: Rename perf tests (#1046)
Browse files Browse the repository at this point in the history
- Add number of ports to a test name
- Change the vague test name `4tx_4rx_8nics`, which suggested that the
test was run on 8 nics, when actually the test was run on 4 nics and 8
ports
  • Loading branch information
staszczuk authored Feb 7, 2025
1 parent f53dd3a commit fa47919
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"i4320p59",
],
)
def test_perf_1tx_1nic(build, media, nic_port_list, test_time, video_format):
def test_perf_1tx_1nic_1port(build, media, nic_port_list, test_time, video_format):
video_file = yuv_files[video_format]

config = rxtxapp.create_empty_performance_config()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@
"i4320p59",
],
)
def test_perf_1tx_1rx_2nics(build, media, nic_port_list, test_time, video_format):
def test_perf_1tx_1rx_2nics_2ports(
build, media, nic_port_list, test_time, video_format
):
# Increase time for 4k and 8k streams
if "2160" in video_format:
test_time = 60
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"i4320p59",
],
)
def test_perf_2tx_2nics(build, media, nic_port_list, test_time, video_format):
def test_perf_2tx_2nics_2ports(build, media, nic_port_list, test_time, video_format):
# Increase time for 4k and 8k streams
if "2160" in video_format:
test_time = 60
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@
"i4320p59",
],
)
def test_perf_2tx_2rx_4nics(build, media, nic_port_list, test_time, video_format):
def test_perf_2tx_2rx_4nics_4ports(
build, media, nic_port_list, test_time, video_format
):
# Increase time for 4k and 8k streams
if "2160" in video_format:
test_time = 60
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"i4320p59",
],
)
def test_perf_4tx_4nics(build, media, nic_port_list, test_time, video_format):
def test_perf_4tx_4nics_4ports(build, media, nic_port_list, test_time, video_format):
# For 4 NICs init time of the app is increased
test_time = 60

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@
"i4320p59",
],
)
def test_perf_2tx_2rx_4nics(build, media, nic_port_list, test_time, video_format):
def test_perf_4tx_4rx_4nics_8ports(
build, media, nic_port_list, test_time, video_format
):
# For 4 NICs init time of the app is increased
test_time = 60

Expand Down

0 comments on commit fa47919

Please sign in to comment.