Add tests for RemoteGlobalMetadataManager

Signed-off-by: Shivansh Arora <hishiv@amazon.com>
opensearch-project · Jun 17, 2024 · 587c566 · 587c566
1 parent a3402d1
commit 587c566
Show file tree

Hide file tree

Showing 7 changed files with 554 additions and 10 deletions.
diff --git a/server/src/test/java/org/opensearch/gateway/remote/RemoteGlobalMetadataManagerTests.java b/server/src/test/java/org/opensearch/gateway/remote/RemoteGlobalMetadataManagerTests.java
diff --git a/...er/src/test/java/org/opensearch/gateway/remote/model/RemoteCoordinationMetadataTests.java b/...er/src/test/java/org/opensearch/gateway/remote/model/RemoteCoordinationMetadataTests.java
@@ -229,7 +229,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private CoordinationMetadata getCoordinationMetadata() {
+    public static CoordinationMetadata getCoordinationMetadata() {
         return CoordinationMetadata.builder()
             .term(TERM)
             .lastAcceptedConfiguration(new VotingConfiguration(Set.of("node1")))

diff --git a/server/src/test/java/org/opensearch/gateway/remote/model/RemoteCustomMetadataTests.java b/server/src/test/java/org/opensearch/gateway/remote/model/RemoteCustomMetadataTests.java
@@ -232,7 +232,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private Custom getCustomMetadata() {
+    public static Custom getCustomMetadata() {
         return IndexGraveyard.builder().addTombstone(new Index("test-index", "3q2423")).build();
     }
 

diff --git a/server/src/test/java/org/opensearch/gateway/remote/model/RemoteGlobalMetadataTests.java b/server/src/test/java/org/opensearch/gateway/remote/model/RemoteGlobalMetadataTests.java
@@ -180,7 +180,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private Metadata getGlobalMetadata() {
+    public static Metadata getGlobalMetadata() {
         return Metadata.builder()
             .templates(
                 TemplatesMetadata.builder()

diff --git a/.../test/java/org/opensearch/gateway/remote/model/RemoteHashesOfConsistentSettingsTests.java b/.../test/java/org/opensearch/gateway/remote/model/RemoteHashesOfConsistentSettingsTests.java
@@ -185,7 +185,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private DiffableStringMap getHashesOfConsistentSettings() {
+    public static DiffableStringMap getHashesOfConsistentSettings() {
         Map<String, String> hashesOfConsistentSettings = new HashMap<>();
         hashesOfConsistentSettings.put("secure-setting-key", "secure-setting-value");
         return new DiffableStringMap(hashesOfConsistentSettings);

diff --git a/.../test/java/org/opensearch/gateway/remote/model/RemotePersistentSettingsMetadataTests.java b/.../test/java/org/opensearch/gateway/remote/model/RemotePersistentSettingsMetadataTests.java
@@ -224,7 +224,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private Settings getSettings() {
+    public static Settings getSettings() {
         return Settings.builder().put("random_index_setting_" + randomAlphaOfLength(3), randomAlphaOfLength(5)).build();
     }
 }
diff --git a/server/src/test/java/org/opensearch/gateway/remote/model/RemoteTemplatesMetadataTests.java b/server/src/test/java/org/opensearch/gateway/remote/model/RemoteTemplatesMetadataTests.java
@@ -227,7 +227,7 @@ public void testSerDe() throws IOException {
         }
     }
 
-    private TemplatesMetadata getTemplatesMetadata() {
+    public static TemplatesMetadata getTemplatesMetadata() {
         return TemplatesMetadata.builder()
             .put(
                 IndexTemplateMetadata.builder("template" + randomAlphaOfLength(3))
-Original file line number
+Diff line change
@@ Expand Up / @@ -232,7 +232,7 @@ public void testSerDe() throws IOException { @@
             }
         }
-        private Custom getCustomMetadata() {
+        public static Custom getCustomMetadata() {
             return IndexGraveyard.builder().addTombstone(new Index("test-index", "3q2423")).build();
         }
@@ Expand Down @@