From a927f246f60688392c5685cb60c06159174813e4 Mon Sep 17 00:00:00 2001 From: abchoudh-amd Date: Mon, 22 Sep 2025 18:07:13 +0530 Subject: [PATCH] Fix test failures (#1059) * Test fix * Added path not exists check --- projects/rocprofiler-compute/pyproject.toml | 3 +++ .../tests/test_profile_general.py | 12 ++++++------ 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/projects/rocprofiler-compute/pyproject.toml b/projects/rocprofiler-compute/pyproject.toml index da826a2855..bca34f3fbc 100644 --- a/projects/rocprofiler-compute/pyproject.toml +++ b/projects/rocprofiler-compute/pyproject.toml @@ -107,4 +107,7 @@ markers = [ "sets_func", "sets_perf", "pc_sampling", + "roofline", + "path", + "sci_notion", ] diff --git a/projects/rocprofiler-compute/tests/test_profile_general.py b/projects/rocprofiler-compute/tests/test_profile_general.py index 00f3ba01bf..6bd182d17b 100644 --- a/projects/rocprofiler-compute/tests/test_profile_general.py +++ b/projects/rocprofiler-compute/tests/test_profile_general.py @@ -1737,8 +1737,8 @@ def test_list_available_metrics(binary_handler_profile_rocprof_compute, capsys): _ = binary_handler_profile_rocprof_compute( config, workload_dir, options, check_success=True, roof=False ) - # workload dir should be empty - assert not os.listdir(workload_dir) + # workload dir should not exist + assert not Path(workload_dir).exists() test_utils.clean_output_dir(config["cleanup"], workload_dir) # Test output @@ -1758,8 +1758,8 @@ def test_list_available_metrics_with_block( ) # Should return code 1 since --block cannot be used with --list-available-metrics assert code == 1 - # workload dir should be empty - assert not os.listdir(workload_dir) + # workload dir should not exist + assert not Path(workload_dir).exists() test_utils.clean_output_dir(config["cleanup"], workload_dir) @@ -1984,6 +1984,6 @@ class TestSetsIntegration: check_success=False, roof=False, ) - # workload dir should be empty - assert not os.listdir(workload_dir) + # workload dir should not exist + assert not Path(workload_dir).exists() test_utils.clean_output_dir(config["cleanup"], workload_dir)