Skip CAPI_LEGACY_SPECTRAL_TEST entirely

rapidsai · Feb 14, 2024 · 87bb1a6 · 87bb1a6
1 parent 52ae751
commit 87bb1a6
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 15 deletions.
diff --git a/cpp/tests/CMakeLists.txt b/cpp/tests/CMakeLists.txt
@@ -737,7 +737,14 @@ ConfigureCTest(CAPI_RANDOM_WALKS_TEST c_api/sg_random_walks_test.c)
 ConfigureCTest(CAPI_TRIANGLE_COUNT_TEST c_api/triangle_count_test.c)
 ConfigureCTest(CAPI_LOUVAIN_TEST c_api/louvain_test.c)
 ConfigureCTest(CAPI_LEIDEN_TEST c_api/leiden_test.c)
-ConfigureCTest(CAPI_LEGACY_SPECTRAL_TEST c_api/legacy_spectral_test.c)
+#############################################################################
+# Skipping due to CUDA 12.2 failure that traces back to RAFT                #
+# TODO: Uncomment this once the issue is fixed.                             #
+#                                                                           #
+# xref: https://github.com/rapidsai/cugraph/issues/4173                     #
+# xref: https://github.com/rapidsai/raft/issues/2186                        #
+#############################################################################
+# ConfigureCTest(CAPI_LEGACY_SPECTRAL_TEST c_api/legacy_spectral_test.c)
 ConfigureCTest(CAPI_LEGACY_ECG_TEST c_api/legacy_ecg_test.c)
 ConfigureCTest(CAPI_CORE_NUMBER_TEST c_api/core_number_test.c)
 ConfigureCTest(CAPI_SIMILARITY_TEST c_api/similarity_test.c)

diff --git a/cpp/tests/c_api/legacy_spectral_test.c b/cpp/tests/c_api/legacy_spectral_test.c
@@ -19,8 +19,6 @@
 #include <cugraph_c/algorithms.h>
 #include <cugraph_c/graph.h>
 
-#include <gtest/gtest.h>
-
 #include <math.h>
 
 typedef int32_t vertex_t;
@@ -271,10 +269,6 @@ int test_spectral()
 
 int test_balanced_cut_unequal_weight()
 {
-  // This test is skipped due to a bug in cuSparse 12.2 that affects RAFT.
-  // See https://github.com/rapidsai/cugraph/issues/4173
-  // See https://github.com/rapidsai/raft/issues/2186
-  GTEST_SKIP();
   size_t num_clusters        = 2;
   size_t num_eigenvectors    = 2;
   size_t num_edges           = 14;
@@ -313,10 +307,6 @@ int test_balanced_cut_unequal_weight()
 
 int test_balanced_cut_equal_weight()
 {
-  // This test is skipped due to a bug in cuSparse 12.2 that affects RAFT.
-  // See https://github.com/rapidsai/cugraph/issues/4173
-  // See https://github.com/rapidsai/raft/issues/2186
-  GTEST_SKIP();
   size_t num_clusters        = 2;
   size_t num_eigenvectors    = 2;
   size_t num_edges           = 14;
@@ -355,10 +345,6 @@ int test_balanced_cut_equal_weight()
 
 int test_balanced_cut_no_weight()
 {
-  // This test is skipped due to a bug in cuSparse 12.2 that affects RAFT.
-  // See https://github.com/rapidsai/cugraph/issues/4173
-  // See https://github.com/rapidsai/raft/issues/2186
-  GTEST_SKIP();
   size_t num_clusters        = 2;
   size_t num_eigenvectors    = 2;
   size_t num_edges           = 14;