From 4dd6499f06daa3abcdcbe5efd703a7387e4cf254 Mon Sep 17 00:00:00 2001 From: Minhan Cao Date: Tue, 3 Dec 2024 10:23:32 -0800 Subject: [PATCH] build: Removed GTest::gtest_main from CMakeLists.txt for velox_simple_aggregate_test (#11668) Summary: Removed GTest::gtest_main from CMakeLists.txt for velox_simple_aggregate_test. ``` add_executable(velox_simple_aggregate_test SimpleAggregateAdapterTest.cpp Main.cpp) ``` Since Main.cpp is already included here, GTest::gtest_main is not needed in target_link_libraries. Pull Request resolved: https://github.com/facebookincubator/velox/pull/11668 Reviewed By: kagamiori Differential Revision: D66690701 Pulled By: xiaoxmeng fbshipit-source-id: 51c5536de45bfd15abdf7a83f46c95f09846a976 --- velox/exec/tests/CMakeLists.txt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/velox/exec/tests/CMakeLists.txt b/velox/exec/tests/CMakeLists.txt index 2b3461c6cc19..ccd1d9caa81d 100644 --- a/velox/exec/tests/CMakeLists.txt +++ b/velox/exec/tests/CMakeLists.txt @@ -284,8 +284,9 @@ target_link_libraries( velox_simple_aggregate velox_exec velox_functions_aggregates_test_lib - GTest::gtest - GTest::gtest_main) + GTest::gtest) + +add_test(velox_simple_aggregate_test velox_simple_aggregate_test) if(VELOX_ENABLE_BENCHMARKS) add_library(velox_spiller_join_benchmark_base JoinSpillInputBenchmarkBase.cpp