#2380: tests: Fix preprocessor clang if statements

DARMA-tasking · Jan 29, 2025 · 05d1f25 · 05d1f25
1 parent b8fa81e
commit 05d1f25
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/unit/runtime/test_cli_arguments.extended.cc b/tests/unit/runtime/test_cli_arguments.extended.cc
@@ -57,13 +57,14 @@ TEST_F(TestCliArguments, test_vt_assert) {
   #if __clang_major__ == 9 || __clang_major__ == 10
     GTEST_SKIP() << "Skipping test for Clang 9 or 10.";
   #endif
-#endif
+#else
   EXPECT_EQ(theConfig()->vt_no_assert_fail, false);
 
   ASSERT_THROW(
     vtAssert(false, "Should throw."),
     std::runtime_error
   );
+#endif
 }
 #endif
 

diff --git a/tests/unit/runtime/test_preconfig.nompi.cc b/tests/unit/runtime/test_preconfig.nompi.cc
@@ -54,12 +54,11 @@ struct TestPreconfig : TestHarness { };
 TEST_F(TestPreconfig, test_vt_assert) {
 #if vt_check_enabled(production_build) or defined(__INTEL_COMPILER)
   GTEST_SKIP();
-#else
 #elif defined(__clang__)
   #if __clang_major__ == 9 || __clang_major__ == 10
     GTEST_SKIP() << "Skipping test for Clang 9 or 10.";
   #endif
-#endif
+#else
   EXPECT_EQ(vt::debug::preConfig()->vt_throw_on_abort, true)
     << "vt_throw_on_abort should be enabled by default";