Revert "Safer Wait for Snapshot Success in ClusterPrivilegeTests"

This reverts commit b3871f3.

Author: original-brownbear

x-pack/plugin/security/src/test/java/org/elasticsearch/integration/ClusterPrivilegeTests.java

@@ -6,14 +6,12 @@
 package org.elasticsearch.integration;
 
 import org.elasticsearch.action.admin.cluster.snapshots.status.SnapshotsStatusResponse;
-import org.elasticsearch.action.admin.cluster.state.ClusterStateRequest;
 import org.elasticsearch.client.Request;
 import org.elasticsearch.cluster.SnapshotsInProgress;
 import org.elasticsearch.common.Strings;
 import org.elasticsearch.common.settings.SecureString;
 import org.elasticsearch.common.settings.Settings;
 import org.elasticsearch.xpack.core.security.authc.support.Hasher;
-import org.hamcrest.Matchers;
 import org.junit.AfterClass;
 import org.junit.BeforeClass;
 
@@ -206,12 +204,5 @@ private void waitForSnapshotToFinish(String repo, String snapshot) throws Except
             SnapshotsStatusResponse response = client().admin().cluster().prepareSnapshotStatus(repo).setSnapshots(snapshot).get();
             assertThat(response.getSnapshots().get(0).getState(), is(SnapshotsInProgress.State.SUCCESS));
         });
-        // The status of the snapshot in the repository can become SUCCESS before it is fully finalized in the cluster state so wait for
-        // it to disappear from the cluster state as well
-        assertBusy(() -> {
-            SnapshotsInProgress response =
-                client().admin().cluster().state(new ClusterStateRequest()).get().getState().custom(SnapshotsInProgress.TYPE);
-            assertThat(response.entries(), Matchers.empty());
-        });
     }
 }