Skip to content

Commit a98c880

Browse files
authored
feature: update zetMetricGroupGet return when no metrics found (#345)
Resolves: NEO-17111 Signed-off-by: Matias Cabral <[email protected]>
1 parent 0bb04d0 commit a98c880

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

conformance_tests/tools/metrics/src/test_metric_enable.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,10 @@ LZT_TEST_F(
7272

7373
uint32_t metricGroupCount = 0;
7474
ze_result_t result = zetMetricGroupGet(device, &metricGroupCount, nullptr);
75-
EXPECT_ZE_RESULT_SUCCESS(result);
75+
EXPECT_EQ(
76+
result,
77+
ZE_RESULT_ERROR_DEPENDENCY_UNAVAILABLE); // if no metrics are found,
78+
// something is missing
7679
EXPECT_EQ(metricGroupCount, 0);
7780

7881
lzt::enable_metrics_runtime(device);

layer_tests/tracing/src/test_api_ltracing.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -976,8 +976,9 @@ LZT_TEST_F(
976976

977977
ASSERT_EQ(initial_result, zeCommandListReset(command_list));
978978
uint32_t metricGroupCount = 0;
979-
EXPECT_ZE_RESULT_SUCCESS(
980-
zetMetricGroupGet(device, &metricGroupCount, nullptr));
979+
ze_result_t result = zetMetricGroupGet(device, &metricGroupCount, nullptr);
980+
ASSERT_EQ(result, ZE_RESULT_ERROR_DEPENDENCY_UNAVAILABLE); // Assuming metrics
981+
// are unavailable
981982
}
982983

983984
LZT_TEST_F(

0 commit comments

Comments
 (0)