From aeb2161b2a3ddaa9108eefc5e31697eb38fb953d Mon Sep 17 00:00:00 2001 From: Karl W Schulz Date: Fri, 23 Feb 2024 09:56:58 -0600 Subject: [PATCH] updating testing files to meet formatting requirements Signed-off-by: Karl W Schulz --- tests/generate_test_analyze_workloads.py | 92 +++++----- tests/test_saved_analysis.py | 205 +++++++++++++++++++---- 2 files changed, 216 insertions(+), 81 deletions(-) diff --git a/tests/generate_test_analyze_workloads.py b/tests/generate_test_analyze_workloads.py index 425ba52f85..20193479dd 100644 --- a/tests/generate_test_analyze_workloads.py +++ b/tests/generate_test_analyze_workloads.py @@ -17,49 +17,49 @@ if __name__ == "__main__": with open("test_analyze_workloads.py", "a") as f: with open("test_saved_analysis.py", "a") as g: - for workload in workloads: - workload_name = workload[workload.rfind("/") + 1 :] - if ( - os.path.exists(workload + "/MI100") - and len(os.listdir(workload + "/MI100")) > 0 - ): - test = ( - "\n\ndef test_analyze_" - + workload_name - + "_MI100():\n with patch('sys.argv',['omniperf', 'analyze', '--path', '" - + workload - + "/MI100']): omniperf.main()" - ) - f.write(test) - test = ( - "\n\ndef test_saved_" - + workload_name - + "_MI100():\n compare('" - + workload - + "/MI100/prev_analysis', '" - + workload - + "/MI100/saved_analysis')" - ) - g.write(test) - if ( - os.path.exists(workload + "/MI200") - and len(os.listdir(workload + "/MI200")) > 0 - ): - test = ( - "\n\ndef test_" - + workload_name - + "_MI200():\n with patch('sys.argv',['omniperf', 'analyze', '--path', '" - + workload - + "/MI200']): omniperf.main()" - ) - f.write(test) - test = ( - "\n\ndef test_saved_" - + workload_name - + "_MI200():\n compare('" - + workload - + "/MI200/prev_analysis', '" - + workload - + "/MI200/saved_analysis')" - ) - g.write(test) + for workload in workloads: + workload_name = workload[workload.rfind("/") + 1 :] + if ( + os.path.exists(workload + "/MI100") + and len(os.listdir(workload + "/MI100")) > 0 + ): + test = ( + "\n\ndef test_analyze_" + + workload_name + + "_MI100():\n with patch('sys.argv',['omniperf', 'analyze', '--path', '" + + workload + + "/MI100']): omniperf.main()" + ) + f.write(test) + test = ( + "\n\ndef test_saved_" + + workload_name + + "_MI100():\n compare('" + + workload + + "/MI100/prev_analysis', '" + + workload + + "/MI100/saved_analysis')" + ) + g.write(test) + if ( + os.path.exists(workload + "/MI200") + and len(os.listdir(workload + "/MI200")) > 0 + ): + test = ( + "\n\ndef test_" + + workload_name + + "_MI200():\n with patch('sys.argv',['omniperf', 'analyze', '--path', '" + + workload + + "/MI200']): omniperf.main()" + ) + f.write(test) + test = ( + "\n\ndef test_saved_" + + workload_name + + "_MI200():\n compare('" + + workload + + "/MI200/prev_analysis', '" + + workload + + "/MI200/saved_analysis')" + ) + g.write(test) diff --git a/tests/test_saved_analysis.py b/tests/test_saved_analysis.py index 1d541d6129..6b34992cb2 100644 --- a/tests/test_saved_analysis.py +++ b/tests/test_saved_analysis.py @@ -25,106 +25,241 @@ def compare(prev, cur): def test_saved_dispatch_invalid_MI100(): - compare('workloads/dispatch_invalid/MI100/prev_analysis', 'workloads/dispatch_invalid/MI100/saved_analysis') + compare( + "workloads/dispatch_invalid/MI100/prev_analysis", + "workloads/dispatch_invalid/MI100/saved_analysis", + ) + def test_saved_kernel_verbose_4_MI100(): - compare('workloads/kernel_verbose_4/MI100/prev_analysis', 'workloads/kernel_verbose_4/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_4/MI100/prev_analysis", + "workloads/kernel_verbose_4/MI100/saved_analysis", + ) + def test_saved_no_roof_MI100(): - compare('workloads/no_roof/MI100/prev_analysis', 'workloads/no_roof/MI100/saved_analysis') + compare( + "workloads/no_roof/MI100/prev_analysis", "workloads/no_roof/MI100/saved_analysis" + ) + def test_saved_kernel_verbose_5_MI100(): - compare('workloads/kernel_verbose_5/MI100/prev_analysis', 'workloads/kernel_verbose_5/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_5/MI100/prev_analysis", + "workloads/kernel_verbose_5/MI100/saved_analysis", + ) + def test_saved_ipblocks_CPC_MI100(): - compare('workloads/ipblocks_CPC/MI100/prev_analysis', 'workloads/ipblocks_CPC/MI100/saved_analysis') + compare( + "workloads/ipblocks_CPC/MI100/prev_analysis", + "workloads/ipblocks_CPC/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_SQC_TCP_CPC_MI100(): - compare('workloads/ipblocks_SQ_SQC_TCP_CPC/MI100/prev_analysis', 'workloads/ipblocks_SQ_SQC_TCP_CPC/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ_SQC_TCP_CPC/MI100/prev_analysis", + "workloads/ipblocks_SQ_SQC_TCP_CPC/MI100/saved_analysis", + ) + def test_saved_kernel_verbose_2_MI100(): - compare('workloads/kernel_verbose_2/MI100/prev_analysis', 'workloads/kernel_verbose_2/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_2/MI100/prev_analysis", + "workloads/kernel_verbose_2/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_MI100(): - compare('workloads/ipblocks_SQ/MI100/prev_analysis', 'workloads/ipblocks_SQ/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ/MI100/prev_analysis", + "workloads/ipblocks_SQ/MI100/saved_analysis", + ) + def test_saved_kernel_substr_MI100(): - compare('workloads/kernel_substr/MI100/prev_analysis', 'workloads/kernel_substr/MI100/saved_analysis') + compare( + "workloads/kernel_substr/MI100/prev_analysis", + "workloads/kernel_substr/MI100/saved_analysis", + ) + def test_saved_ipblocks_CPF_MI100(): - compare('workloads/ipblocks_CPF/MI100/prev_analysis', 'workloads/ipblocks_CPF/MI100/saved_analysis') + compare( + "workloads/ipblocks_CPF/MI100/prev_analysis", + "workloads/ipblocks_CPF/MI100/saved_analysis", + ) + def test_saved_dispatch_0_1_MI100(): - compare('workloads/dispatch_0_1/MI100/prev_analysis', 'workloads/dispatch_0_1/MI100/saved_analysis') + compare( + "workloads/dispatch_0_1/MI100/prev_analysis", + "workloads/dispatch_0_1/MI100/saved_analysis", + ) + def test_saved_kernel_verbose_0_MI100(): - compare('workloads/kernel_verbose_0/MI100/prev_analysis', 'workloads/kernel_verbose_0/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_0/MI100/prev_analysis", + "workloads/kernel_verbose_0/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQC_MI100(): - compare('workloads/ipblocks_SQC/MI100/prev_analysis', 'workloads/ipblocks_SQC/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQC/MI100/prev_analysis", + "workloads/ipblocks_SQC/MI100/saved_analysis", + ) + def test_saved_join_type_grid_MI100(): - compare('workloads/join_type_grid/MI100/prev_analysis', 'workloads/join_type_grid/MI100/saved_analysis') + compare( + "workloads/join_type_grid/MI100/prev_analysis", + "workloads/join_type_grid/MI100/saved_analysis", + ) + def test_saved_ipblocks_TCP_MI100(): - compare('workloads/ipblocks_TCP/MI100/prev_analysis', 'workloads/ipblocks_TCP/MI100/saved_analysis') + compare( + "workloads/ipblocks_TCP/MI100/prev_analysis", + "workloads/ipblocks_TCP/MI100/saved_analysis", + ) + def test_saved_kernel_verbose_3_MI100(): - compare('workloads/kernel_verbose_3/MI100/prev_analysis', 'workloads/kernel_verbose_3/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_3/MI100/prev_analysis", + "workloads/kernel_verbose_3/MI100/saved_analysis", + ) + def test_saved_device_inv_int_MI100(): - compare('workloads/device_inv_int/MI100/prev_analysis', 'workloads/device_inv_int/MI100/saved_analysis') + compare( + "workloads/device_inv_int/MI100/prev_analysis", + "workloads/device_inv_int/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_SPI_MI100(): - compare('workloads/ipblocks_SQ_SPI/MI100/prev_analysis', 'workloads/ipblocks_SQ_SPI/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ_SPI/MI100/prev_analysis", + "workloads/ipblocks_SQ_SPI/MI100/saved_analysis", + ) + def test_saved_kernel_summaries_MI100(): - compare('workloads/kernel_summaries/MI100/prev_analysis', 'workloads/kernel_summaries/MI100/saved_analysis') + compare( + "workloads/kernel_summaries/MI100/prev_analysis", + "workloads/kernel_summaries/MI100/saved_analysis", + ) + def test_saved_dispatch_2_MI100(): - compare('workloads/dispatch_2/MI100/prev_analysis', 'workloads/dispatch_2/MI100/saved_analysis') + compare( + "workloads/dispatch_2/MI100/prev_analysis", + "workloads/dispatch_2/MI100/saved_analysis", + ) + def test_saved_kernel_inv_int_MI100(): - compare('workloads/kernel_inv_int/MI100/prev_analysis', 'workloads/kernel_inv_int/MI100/saved_analysis') + compare( + "workloads/kernel_inv_int/MI100/prev_analysis", + "workloads/kernel_inv_int/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_SPI_TA_TCC_CPF_MI100(): - compare('workloads/ipblocks_SQ_SPI_TA_TCC_CPF/MI100/prev_analysis', 'workloads/ipblocks_SQ_SPI_TA_TCC_CPF/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ_SPI_TA_TCC_CPF/MI100/prev_analysis", + "workloads/ipblocks_SQ_SPI_TA_TCC_CPF/MI100/saved_analysis", + ) + def test_saved_dispatch_7_MI100(): - compare('workloads/dispatch_7/MI100/prev_analysis', 'workloads/dispatch_7/MI100/saved_analysis') + compare( + "workloads/dispatch_7/MI100/prev_analysis", + "workloads/dispatch_7/MI100/saved_analysis", + ) + def test_saved_kernel_inv_str_MI100(): - compare('workloads/kernel_inv_str/MI100/prev_analysis', 'workloads/kernel_inv_str/MI100/saved_analysis') + compare( + "workloads/kernel_inv_str/MI100/prev_analysis", + "workloads/kernel_inv_str/MI100/saved_analysis", + ) + def test_saved_ipblocks_TCC_MI100(): - compare('workloads/ipblocks_TCC/MI100/prev_analysis', 'workloads/ipblocks_TCC/MI100/saved_analysis') + compare( + "workloads/ipblocks_TCC/MI100/prev_analysis", + "workloads/ipblocks_TCC/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_TA_MI100(): - compare('workloads/ipblocks_SQ_TA/MI100/prev_analysis', 'workloads/ipblocks_SQ_TA/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ_TA/MI100/prev_analysis", + "workloads/ipblocks_SQ_TA/MI100/saved_analysis", + ) + def test_saved_ipblocks_SPI_MI100(): - compare('workloads/ipblocks_SPI/MI100/prev_analysis', 'workloads/ipblocks_SPI/MI100/saved_analysis') + compare( + "workloads/ipblocks_SPI/MI100/prev_analysis", + "workloads/ipblocks_SPI/MI100/saved_analysis", + ) + def test_saved_ipblocks_SQ_CPC_MI100(): - compare('workloads/ipblocks_SQ_CPC/MI100/prev_analysis', 'workloads/ipblocks_SQ_CPC/MI100/saved_analysis') + compare( + "workloads/ipblocks_SQ_CPC/MI100/prev_analysis", + "workloads/ipblocks_SQ_CPC/MI100/saved_analysis", + ) + def test_saved_dispatch_6_8_MI100(): - compare('workloads/dispatch_6_8/MI100/prev_analysis', 'workloads/dispatch_6_8/MI100/saved_analysis') + compare( + "workloads/dispatch_6_8/MI100/prev_analysis", + "workloads/dispatch_6_8/MI100/saved_analysis", + ) + def test_saved_dispatch_0_MI100(): - compare('workloads/dispatch_0/MI100/prev_analysis', 'workloads/dispatch_0/MI100/saved_analysis') + compare( + "workloads/dispatch_0/MI100/prev_analysis", + "workloads/dispatch_0/MI100/saved_analysis", + ) + def test_saved_kernel_verbose_1_MI100(): - compare('workloads/kernel_verbose_1/MI100/prev_analysis', 'workloads/kernel_verbose_1/MI100/saved_analysis') + compare( + "workloads/kernel_verbose_1/MI100/prev_analysis", + "workloads/kernel_verbose_1/MI100/saved_analysis", + ) + def test_saved_ipblocks_TA_MI100(): - compare('workloads/ipblocks_TA/MI100/prev_analysis', 'workloads/ipblocks_TA/MI100/saved_analysis') + compare( + "workloads/ipblocks_TA/MI100/prev_analysis", + "workloads/ipblocks_TA/MI100/saved_analysis", + ) + def test_saved_ipblocks_TD_MI100(): - compare('workloads/ipblocks_TD/MI100/prev_analysis', 'workloads/ipblocks_TD/MI100/saved_analysis') + compare( + "workloads/ipblocks_TD/MI100/prev_analysis", + "workloads/ipblocks_TD/MI100/saved_analysis", + ) + def test_saved_path_MI100(): - compare('workloads/path/MI100/prev_analysis', 'workloads/path/MI100/saved_analysis') + compare("workloads/path/MI100/prev_analysis", "workloads/path/MI100/saved_analysis") + def test_saved_join_type_kernel_MI100(): - compare('workloads/join_type_kernel/MI100/prev_analysis', 'workloads/join_type_kernel/MI100/saved_analysis') \ No newline at end of file + compare( + "workloads/join_type_kernel/MI100/prev_analysis", + "workloads/join_type_kernel/MI100/saved_analysis", + )