Skip to content

Commit

Permalink
Bump Catch2 to d548be26 for new SKIP macro
Browse files Browse the repository at this point in the history
- Change most locations with manual skip message to use the new SKIP
  macro. Some legacy tests with more complex control flow still use
  manual messages.
- Update developer documentation to recommend the use of SKIP macro when
  testing optional features.
  • Loading branch information
psalz committed Jan 12, 2023
1 parent ac20368 commit 66665de
Show file tree
Hide file tree
Showing 78 changed files with 112 additions and 139 deletions.
6 changes: 6 additions & 0 deletions docs/README.adoc
Expand Up @@ -345,6 +345,12 @@ where `<impls...>` is a comma-separated list of `hipSYCL`, `ComputeCpp` and/or `

NOTE: #**TODO:** Custom matchers.#

=== Testing Optional Features

The CTS may include tests that cannot be executed in all circumstances.
Examples of such tests include tests for optional features, tests that depend on certain device capabilities, tests that require multiple devices as well as tests for vendor extensions.
In such scenarios, Catch2's `SKIP` macro should be used to explicitly report a test case as skipped.

=== Best Practices

Here is a list of best practices for writing test cases.
Expand Down
2 changes: 1 addition & 1 deletion tests/cuda_interop/cuda_backend_avail.cpp
Expand Up @@ -27,6 +27,6 @@ TEST_CASE("CUDA backend availability test") {
INFO("Checking sycl::backend::cuda is available");
REQUIRE(q.get_backend() == sycl::backend::cuda);
#else
INFO("The test is skipped because CUDA back-end is not supported");
SKIP("CUDA backend is not supported");
#endif // SYCL_BACKEND_CUDA
}
2 changes: 1 addition & 1 deletion tests/cuda_interop/cuda_interop_constructors.cpp
Expand Up @@ -128,6 +128,6 @@ TEST_CASE("CUDA interop constructor test") {
CHECK(interopEvent == cuEvent);
}
#else
INFO("The test is skipped because CUDA back-end is not supported");
SKIP("CUDA backend is not supported");
#endif // SYCL_BACKEND_CUDA
}
2 changes: 1 addition & 1 deletion tests/cuda_interop/cuda_interop_device_function.cu
Expand Up @@ -86,6 +86,6 @@ TEST_CASE("CUDA interop device function test") {
test_device_function_buffer<int>(queue, "int");
test_device_function_local<int>(queue, "int");
#else
INFO("The test is skipped because CUDA back-end is not supported");
SKIP("CUDA backend is not supported");
#endif // SYCL_BACKEND_CUDA
}
2 changes: 1 addition & 1 deletion tests/cuda_interop/cuda_interop_get.cpp
Expand Up @@ -66,6 +66,6 @@ TEST_CASE("CUDA interop get test") {
check_return_type<CUevent>(interopEvent, "get_native(event)");
}
#else
INFO("The test is skipped because CUDA back-end is not supported");
SKIP("CUDA backend is not supported");
#endif // SYCL_BACKEND_CUDA
};
2 changes: 1 addition & 1 deletion tests/cuda_interop/cuda_interop_kernel.cpp
Expand Up @@ -31,6 +31,6 @@ TEST_CASE("CUDA interop kernel test") {
run_all_tests<c2> tests_c2;
tests_c2(queue, "class with default constructor");
#else
INFO("The test is skipped because CUDA back-end is not supported");
SKIP("CUDA backend is not supported");
#endif // SYCL_BACKEND_CUDA
}
Expand Up @@ -130,9 +130,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
check_device_global_api_arrow_operator_for_type<
user_def_types::arrow_operator_overloaded>{}(log, "arrow_operator");
Expand Down
Expand Up @@ -473,9 +473,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_api_basic_for_type>(types, log);
Expand Down
Expand Up @@ -147,9 +147,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_api_subscript_operator_for_type>(types,
Expand Down
Expand Up @@ -284,9 +284,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_handler_queue_exceptions>(types, log);
Expand Down
Expand Up @@ -133,9 +133,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_define_various_ways>(types, log);
Expand Down
Expand Up @@ -172,9 +172,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_kernel_bundle>(types, log);
Expand Down
Expand Up @@ -216,9 +216,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
for_all_types<check_device_global_one_kernel_for_type>(types, log);
#endif
Expand Down
Expand Up @@ -127,9 +127,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_pass_pointer>(types, log);
Expand Down
Expand Up @@ -139,9 +139,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
for_all_types<check_device_global_serveal_kernels_one_device_for_type>(
types, log);
Expand Down
Expand Up @@ -166,9 +166,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_spec_constants>(types, log);
Expand Down
Expand Up @@ -184,9 +184,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_serveal_kernels_one_device_for_type>(
Expand Down
Expand Up @@ -112,9 +112,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_kernel_bundle>(types, log);
Expand Down
Expand Up @@ -437,9 +437,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_device_global_handler_functions_overloads>(types, log);
Expand Down
Expand Up @@ -130,9 +130,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_queue_overloads_for_device_global_array_for_type>(types,
Expand Down
Expand Up @@ -242,9 +242,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_queue_overloads_for_device_global_for_type>(types, log);
Expand Down
Expand Up @@ -252,9 +252,9 @@ class TEST_NAME : public sycl_cts::util::test_base {
*/
void run(util::logger& log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTY_LIST)
WARN("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTY_LIST is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
auto types = device_global_types::get_types();
for_all_types<check_queue_memcpy_overloads_for_type>(types, log);
Expand Down
Expand Up @@ -29,9 +29,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
4 changes: 2 additions & 2 deletions tests/extension/oneapi_properties/properties_has_property.cpp
Expand Up @@ -28,9 +28,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
Expand Up @@ -28,9 +28,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
Expand Up @@ -28,9 +28,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
Expand Up @@ -30,9 +30,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
4 changes: 2 additions & 2 deletions tests/extension/oneapi_properties/properties_prop_eq_op.cpp
Expand Up @@ -28,9 +28,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
using namespace sycl::ext::oneapi::experimental;
Expand Down
Expand Up @@ -67,9 +67,9 @@ class TEST_NAME : public util::test_base {
*/
void run(util::logger &log) override {
#if !defined(SYCL_EXT_ONEAPI_PROPERTIES)
WARN("SYCL_EXT_ONEAPI_PROPERTIES is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_PROPERTIES is not defined");
#elif !defined(SYCL_EXT_ONEAPI_DEVICE_GLOBAL)
WARN("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_DEVICE_GLOBAL is not defined");
#else
{
properties props{device_image_scope};
Expand Down
Expand Up @@ -65,7 +65,7 @@ class TEST_NAME : public util::test_base {
log.note("Check all() for mask with true predicate");
check_diff_sub_group_sizes<verification_func_for_true_predicate>(log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -66,7 +66,7 @@ class TEST_NAME : public util::test_base {
check_diff_sub_group_sizes<verification_func_for_false_predicate>(log);

#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -51,7 +51,7 @@ class TEST_NAME : public util::test_base {
#ifdef SYCL_EXT_ONEAPI_SUB_GROUP_MASK
check_diff_sub_group_sizes<verification_func_for_first_half_predicate>(log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -89,7 +89,7 @@ class TEST_NAME : public util::test_base {
#ifdef SYCL_EXT_ONEAPI_SUB_GROUP_MASK
for_all_types_and_marrays<check_for_type>(types, log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -71,7 +71,7 @@ class TEST_NAME : public util::test_base {
log.note("Check find_high() for mask with false predicate");
check_diff_sub_group_sizes<verification_func_for_false_predicate>(log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -71,7 +71,7 @@ class TEST_NAME : public util::test_base {
log.note("Check find_low() for mask with false predicate");
check_diff_sub_group_sizes<verification_func_for_false_predicate>(log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down
Expand Up @@ -60,7 +60,7 @@ class TEST_NAME : public util::test_base {
#ifdef SYCL_EXT_ONEAPI_SUB_GROUP_MASK
check_diff_sub_group_sizes<verification_func_for_even_predicate>(log);
#else
log.note("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined, test is skipped");
SKIP("SYCL_EXT_ONEAPI_SUB_GROUP_MASK is not defined");
#endif // SYCL_EXT_ONEAPI_SUB_GROUP_MASK
}
};
Expand Down

0 comments on commit 66665de

Please sign in to comment.