Refactored the code

Signed-off-by: Sriram Ganesh <srignsh22@gmail.com>

Author: rgsriram

server/src/test/java/org/opensearch/cluster/metadata/IndexMetadataTests.java

@@ -618,15 +618,13 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
 
         IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.get(emptySettings);
 
-        // Test whitespace-only value (should be allowed)
         final Settings whitespaceSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), true)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "   ")
             .build();
 
         IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.get(whitespaceSettings);
 
-        // Test valid value when remote store is enabled
         final Settings validSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), true)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer-node-1")
@@ -635,7 +633,6 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
         String value = IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.get(validSettings);
         assertEquals("writer-node-1", value);
 
-        // Test that setting throws exception when remote store is disabled
         final Settings disabledSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), false)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer-node-1")
@@ -646,7 +643,6 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
         });
         assertTrue(e.getMessage().contains("can only be set when"));
 
-        // Test that setting throws exception when remote store is not set
         final Settings noRemoteStoreSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer-node-1")
             .build();
@@ -657,7 +653,6 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
         );
         assertTrue(e.getMessage().contains("can only be set when"));
 
-        // Test that setting throws exception when path contains forward slash
         final Settings invalidPathSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), true)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer/node")
@@ -669,7 +664,6 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
         );
         assertTrue(e.getMessage().contains("cannot contain path separators"));
 
-        // Test that setting throws exception when path contains backslash
         final Settings backslashSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), true)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer\\node")
@@ -681,7 +675,6 @@ public void testRemoteStoreSegmentPathPrefixValidation() {
         );
         assertTrue(e.getMessage().contains("cannot contain path separators"));
 
-        // Test that setting throws exception when path contains colon
         final Settings colonSettings = Settings.builder()
             .put(IndexMetadata.INDEX_REMOTE_STORE_ENABLED_SETTING.getKey(), true)
             .put(IndexMetadata.INDEX_REMOTE_STORE_SEGMENT_PATH_PREFIX.getKey(), "writer:node")