Skip to content

Commit 1fe6376

Browse files
authored
Remove unit tests that were really utility programs (#85)
This PR removes a number of tests that were really utility programs: `include/test/gen_*.cc`, `include/test/time_*.cc` and `unit_time_open.cc`. It also cleans up a few unit tests that weren't properly configured (e.g., because a file that was expected to be in a certain place no longer is in the current repo). Verified that all unit tests build and pass with clang on MacOS and Linux, and with g++ on Linux.
1 parent 50f2dde commit 1fe6376

File tree

10 files changed

+20
-1021
lines changed

10 files changed

+20
-1021
lines changed

src/include/test/CMakeLists.txt

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -52,20 +52,13 @@ message(STATUS "Data files for testing will be extracted to ${SIFT_TEST_DIR}")
5252
# Deprecated / removed
5353
# kmeans_add_test(unit_sift_array)
5454
# kmeans_add_test(unit_sift_db)
55-
56-
57-
kmeans_add_test(gen_partitioned)
58-
59-
kmeans_add_test(gen_queries)
60-
61-
kmeans_add_test(time_insert)
62-
63-
kmeans_add_test(time_l2)
64-
65-
kmeans_add_test(time_open)
66-
67-
kmeans_add_test(time_queries)
68-
target_link_libraries(time_queries PRIVATE flat_blas)
55+
# kmeans_add_test(gen_partitioned)
56+
# kmeans_add_test(gen_queries)
57+
# kmeans_add_test(time_insert)
58+
# kmeans_add_test(time_l2)
59+
# kmeans_add_test(time_open)
60+
# kmeans_add_test(time_queries)
61+
# target_link_libraries(time_queries PRIVATE flat_blas)
6962

7063
kmeans_add_test(unit_algorithm)
7164

@@ -74,7 +67,7 @@ kmeans_add_test(unit_concepts)
7467
kmeans_add_test(unit_defs)
7568

7669
kmeans_add_test(unit_gemm)
77-
target_link_libraries(time_queries PRIVATE flat_blas)
70+
target_link_libraries(unit_gemm PRIVATE flat_blas)
7871

7972
kmeans_add_test(unit_ivf_index)
8073

src/include/test/gen_partitioned.cc

Lines changed: 0 additions & 104 deletions
This file was deleted.

src/include/test/gen_queries.cc

Lines changed: 0 additions & 80 deletions
This file was deleted.

0 commit comments

Comments
 (0)