commit f56db508e25035309e58e60e75ecbc5ac75ef278
author pjmcarthur <92818583+pjmcarthur@users.noreply.github.com> Tue Mar 19 21:39:57 2024 -0700
committer GitHub <noreply@github.com> Wed Mar 20 00:39:57 2024 -0400
tree 28adeda2065c277928bf615442945e814faa5f7b
parent faa1001ae68b5ffd0bc32c77162a7ace39f66511

SOLR-16403: Fix DeleteShardTest (#2359)

Simple mistake; test failed sometimes

Co-authored-by: Paul McArthur <pmcarthur-apache@proton.me>

solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java

diff --git a/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java b/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java
index 91ab353..f363bb5 100644
--- a/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java
@@ -63,7 +63,7 @@
             CollectionAdminRequest.deleteShard(collection, "shard1")
                 .process(cluster.getSolrClient()));
 
-    ShardTestUtil.setSliceState(cluster, collection, "shard1", Slice.State.INACTIVE);
+    setSliceState(collection, "shard1", Slice.State.INACTIVE);
 
     // Can delete an INACTIVE shard
     CollectionAdminRequest.deleteShard(collection, "shard1").process(cluster.getSolrClient());
@@ -71,7 +71,7 @@
         "Expected 'shard1' to be removed", collection, (n, c) -> c.getSlice("shard1") == null);
 
     // Can delete a shard under construction
-    ShardTestUtil.setSliceState(cluster, collection, "shard2", Slice.State.CONSTRUCTION);
+    setSliceState(collection, "shard2", Slice.State.CONSTRUCTION);
     CollectionAdminRequest.deleteShard(collection, "shard2").process(cluster.getSolrClient());
     waitForState(
         "Expected 'shard2' to be removed", collection, (n, c) -> c.getSlice("shard2") == null);