diff --git a/presto-native-execution/presto_cpp/main/common/tests/ConfigTest.cpp b/presto-native-execution/presto_cpp/main/common/tests/ConfigTest.cpp index 45db7673ceb6..a6ba91c4fb9a 100644 --- a/presto-native-execution/presto_cpp/main/common/tests/ConfigTest.cpp +++ b/presto-native-execution/presto_cpp/main/common/tests/ConfigTest.cpp @@ -246,4 +246,12 @@ TEST_F(ConfigTest, parseInvalid) { testInvalid("===\n", "property pair '===' has empty key"); } +TEST_F(ConfigTest, optionalNodeId) { + NodeConfig config; + auto nodeId = config.nodeId(); + // Same value must be returned. + EXPECT_EQ(nodeId, config.nodeId()); + EXPECT_EQ(nodeId, config.nodeId()); +} + } // namespace facebook::presto::test diff --git a/presto-native-execution/presto_cpp/main/tests/CMakeLists.txt b/presto-native-execution/presto_cpp/main/tests/CMakeLists.txt index 2f03319363cc..aea7e163fa48 100644 --- a/presto-native-execution/presto_cpp/main/tests/CMakeLists.txt +++ b/presto-native-execution/presto_cpp/main/tests/CMakeLists.txt @@ -13,7 +13,6 @@ add_executable( presto_server_test AnnouncerTest.cpp CoordinatorDiscovererTest.cpp - ConfigsTest.cpp HttpServerWrapper.cpp MutableConfigs.cpp PeriodicMemoryCheckerTest.cpp diff --git a/presto-native-execution/presto_cpp/main/tests/ConfigsTest.cpp b/presto-native-execution/presto_cpp/main/tests/ConfigsTest.cpp deleted file mode 100644 index be121181a70f..000000000000 --- a/presto-native-execution/presto_cpp/main/tests/ConfigsTest.cpp +++ /dev/null @@ -1,31 +0,0 @@ -/* - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -#include "presto_cpp/main/common/Configs.h" - -#include - -namespace facebook::presto { -namespace { - -TEST(NodeConfig, optionalNodeId) { - NodeConfig config; - auto nodeId = config.nodeId(); - // Same value must be returned. - EXPECT_EQ(nodeId, config.nodeId()); - EXPECT_EQ(nodeId, config.nodeId()); -} - -} // namespace -} // namespace facebook::presto