[native] Move optional node config test in existing ConfigTest.

prestodb · May 24, 2024 · 98e941f · 98e941f
1 parent e17d22d
commit 98e941f
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 32 deletions.
diff --git 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
@@ -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