Merge pull request #736 from kbenzie/benie/cts-fix-platform-get-native-handle-test

kbenzie · web-flow · commit c12d60264812 · 2023-07-20T16:12:25.000+01:00
[ur] Fix urPlatformGetNativeHandle test
diff --git a/test/conformance/platform/urPlatformGetNativeHandle.cpp b/test/conformance/platform/urPlatformGetNativeHandle.cpp
@@ -10,7 +10,9 @@ using urPlatformGetNativeHandleTest = uur::platform::urPlatformsTest;
 TEST_F(urPlatformGetNativeHandleTest, Success) {
     for (auto platform : platforms) {
         ur_native_handle_t native_handle = nullptr;
-        ASSERT_SUCCESS(urPlatformGetNativeHandle(platform, &native_handle));
+        if (auto error = urPlatformGetNativeHandle(platform, &native_handle)) {
+            ASSERT_EQ(UR_RESULT_ERROR_UNSUPPORTED_FEATURE, error);
+        }
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,9 @@ using urPlatformGetNativeHandleTest = uur::platform::urPlatformsTest;`
`10`	`10`	`TEST_F(urPlatformGetNativeHandleTest, Success) {`
`11`	`11`	`for (auto platform : platforms) {`
`12`	`12`	`ur_native_handle_t native_handle = nullptr;`
`13`		`- ASSERT_SUCCESS(urPlatformGetNativeHandle(platform, &native_handle));`
	`13`	`+ if (auto error = urPlatformGetNativeHandle(platform, &native_handle)) {`
	`14`	`+ ASSERT_EQ(UR_RESULT_ERROR_UNSUPPORTED_FEATURE, error);`
	`15`	`+ }`
`14`	`16`	`}`
`15`	`17`	`}`
`16`	`18`