Address comment, set no default value and add UT

Author: maobaolong

client-spark/spark3/src/test/java/org/apache/spark/shuffle/DelegationRssShuffleManagerTest.java

@@ -139,7 +139,7 @@ public void testTryAccessCluster() throws Exception {
 
   @Test
   public void testDefaultIncludeExcludeProperties() throws Exception {
-    CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
+    final CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
     SparkConf conf = new SparkConf();
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_INTERVAL_MS, 3000L);
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_TIMES, 3);
@@ -148,7 +148,7 @@ public void testDefaultIncludeExcludeProperties() throws Exception {
     conf.set(RssSparkConfig.RSS_COORDINATOR_QUORUM.key(), "m1:8001,m2:8002");
     conf.set("spark.rss.storage.type", StorageType.LOCALFILE.name());
     conf.set(RssSparkConfig.RSS_TEST_MODE_ENABLE, true);
-    int confInitKeyCount = conf.getAll().length;
+    final int confInitKeyCount = conf.getAll().length;
     assertCreateRssShuffleManager(conf);
 
     // default case: access cluster should include all properties in conf and an extra one.
@@ -162,7 +162,7 @@ public void testDefaultIncludeExcludeProperties() throws Exception {
 
   @Test
   public void testIncludeProperties() throws Exception {
-    CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
+    final CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
     SparkConf conf = new SparkConf();
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_INTERVAL_MS, 3000L);
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_TIMES, 3);
@@ -178,7 +178,7 @@ public void testIncludeProperties() throws Exception {
         RssSparkConfig.RSS_ACCESS_ID
             .key()
             .substring(RssSparkConfig.SPARK_RSS_CONFIG_PREFIX.length()));
-    int confInitKeyCount = conf.getAll().length;
+    final int confInitKeyCount = conf.getAll().length;
     assertCreateRssShuffleManager(conf);
 
     ArgumentCaptor<RssAccessClusterRequest> argumentCaptor =
@@ -205,7 +205,7 @@ public void testIncludeProperties() throws Exception {
 
   @Test
   public void testExcludeProperties() throws Exception {
-    CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
+    final CoordinatorClient mockClient = setupMockedRssShuffleUtils(SUCCESS);
     SparkConf conf = new SparkConf();
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_INTERVAL_MS, 3000L);
     conf.set(RssSparkConfig.RSS_CLIENT_ACCESS_RETRY_TIMES, 3);
@@ -221,7 +221,7 @@ public void testExcludeProperties() throws Exception {
         RssSparkConfig.RSS_ACCESS_ID
             .key()
             .substring(RssSparkConfig.SPARK_RSS_CONFIG_PREFIX.length()));
-    int confInitKeyCount = conf.getAll().length;
+    final int confInitKeyCount = conf.getAll().length;
     assertCreateRssShuffleManager(conf);
 
     ArgumentCaptor<RssAccessClusterRequest> argumentCaptor =