Fix Unit test after remove state enabled assertion

Signed-off-by: Sooraj Sinha <[email protected]>
opensearch-project · Aug 31, 2023 · 99734f1 · 99734f1
1 parent 42af4dd
commit 99734f1
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/server/src/test/java/org/opensearch/gateway/remote/RemoteClusterStateServiceTests.java b/server/src/test/java/org/opensearch/gateway/remote/RemoteClusterStateServiceTests.java
@@ -44,7 +44,6 @@
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.doThrow;
 import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.when;
 
 public class RemoteClusterStateServiceTests extends OpenSearchTestCase {
@@ -80,19 +79,18 @@ public void testFailWriteFullMetadataNonClusterManagerNode() throws IOException
         Assert.assertThat(manifest, nullValue());
     }
 
-    public void testFailWriteFullMetadataWhenRemoteStateDisabled() throws IOException {
+    public void testFailInitializationWhenRemoteStateDisabled() throws IOException {
         final Settings settings = Settings.builder().build();
-        remoteClusterStateService = spy(
-            new RemoteClusterStateService(
+        assertThrows(
+            AssertionError.class,
+            () -> new RemoteClusterStateService(
                 "test-node-id",
                 repositoriesServiceSupplier,
                 settings,
                 new ClusterSettings(settings, ClusterSettings.BUILT_IN_CLUSTER_SETTINGS),
                 () -> 0L
             )
         );
-        final ClusterState clusterState = generateClusterStateWithOneIndex().nodes(nodesWithLocalNodeClusterManager()).build();
-        assertThrows(AssertionError.class, () -> remoteClusterStateService.writeFullMetadata(clusterState));
     }
 
     public void testFailWriteFullMetadataWhenRepositoryNotSet() {