[test] Increase code coverage for class FlussConfigUtils (#914)

Author: wangfengpro

fluss-common/src/main/java/com/alibaba/fluss/config/FlussConfigUtils.java

@@ -17,6 +17,7 @@
 package com.alibaba.fluss.config;
 
 import com.alibaba.fluss.annotation.Internal;
+import com.alibaba.fluss.annotation.VisibleForTesting;
 
 import java.lang.reflect.Field;
 import java.util.HashMap;
@@ -36,7 +37,8 @@ public class FlussConfigUtils {
         CLIENT_OPTIONS = extractConfigOptions("client.");
     }
 
-    private static Map<String, ConfigOption<?>> extractConfigOptions(String prefix) {
+    @VisibleForTesting
+    static Map<String, ConfigOption<?>> extractConfigOptions(String prefix) {
         Map<String, ConfigOption<?>> options = new HashMap<>();
         Field[] fields = ConfigOptions.class.getFields();
         // use Java reflection to collect all options matches the prefix

fluss-common/src/test/java/com/alibaba/fluss/config/FlussConfigUtilsTest.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2025 Alibaba Group Holding Ltd.
+ *
+ * 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.
+ */
+
+package com.alibaba.fluss.config;
+
+import org.junit.jupiter.api.Test;
+
+import java.util.Map;
+
+import static com.alibaba.fluss.config.FlussConfigUtils.CLIENT_OPTIONS;
+import static com.alibaba.fluss.config.FlussConfigUtils.TABLE_OPTIONS;
+import static com.alibaba.fluss.config.FlussConfigUtils.extractConfigOptions;
+import static org.assertj.core.api.Assertions.assertThat;
+
+/** Test for {@link FlussConfigUtils}. */
+class FlussConfigUtilsTest {
+
+    @Test
+    void testExtractOptions() {
+        Map<String, ConfigOption<?>> tableOptions = extractConfigOptions("table.");
+        assertThat(tableOptions).isNotEmpty();
+        tableOptions.forEach(
+                (k, v) -> {
+                    assertThat(k).startsWith("table.");
+                    assertThat(v.key()).startsWith("table.");
+                });
+        assertThat(tableOptions.size()).isEqualTo(TABLE_OPTIONS.size());
+
+        Map<String, ConfigOption<?>> clientOptions = extractConfigOptions("client.");
+        assertThat(clientOptions).isNotEmpty();
+        clientOptions.forEach(
+                (k, v) -> {
+                    assertThat(k).startsWith("client.");
+                    assertThat(v.key()).startsWith("client.");
+                });
+        assertThat(clientOptions.size()).isEqualTo(CLIENT_OPTIONS.size());
+    }
+}

fluss-test-coverage/pom.xml

@@ -263,7 +263,6 @@
                                         <exclude>com.alibaba.fluss.row.encode.RowEncoder</exclude>
                                         <exclude>com.alibaba.fluss.cluster.ServerNode</exclude>
                                         <exclude>com.alibaba.fluss.utils.crc.Crc32C</exclude>
-                                        <exclude>com.alibaba.fluss.config.FlussConfigUtils</exclude>
                                         <exclude>com.alibaba.fluss.table.*</exclude>
                                         <exclude>com.alibaba.fluss.record.*</exclude>
                                         <exclude>com.alibaba.fluss.kv.*</exclude>