(solr) branch branch_9x updated: Fix confusing assertion messages

[psalagnac]

This is an automated email from the ASF dual-hosted git repository.

psalagnac pushed a commit to branch branch_9x
in repository https://gitbox.apache.org/repos/asf/solr.git


The following commit(s) were added to refs/heads/branch_9x by this push:
     new 68651f6d34d Fix confusing assertion messages
68651f6d34d is described below

commit 68651f6d34d071c7ff61c2f41c0b500222095fbf
Author: Pierre Salagnac <psalag...@apache.org>
AuthorDate: Thu Sep 18 14:28:42 2025 +0200

    Fix confusing assertion messages
---
 solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java         | 4 ++--
 .../src/test/org/apache/solr/cloud/OverseerModifyCollectionTest.java  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

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 53acec02971..d3af55427c2 100644
--- a/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java
@@ -121,8 +121,8 @@ public class DeleteShardTest extends SolrCloudTestCase {
 
     assertEquals(1, getCollectionState(collection).getActiveSlices().size());
     assertTrue(
-        "Instance directory still exists", 
FileUtils.fileExists(coreStatus.getInstanceDirectory()));
-    assertTrue("Data directory still exists", 
FileUtils.fileExists(coreStatus.getDataDirectory()));
+        "Instance directory not deleted", 
FileUtils.fileExists(coreStatus.getInstanceDirectory()));
+    assertTrue("Data directory not deleted", 
FileUtils.fileExists(coreStatus.getDataDirectory()));
   }
 
   private void setSliceState(String collectionName, String shardId, 
Slice.State state)
diff --git 
a/solr/core/src/test/org/apache/solr/cloud/OverseerModifyCollectionTest.java 
b/solr/core/src/test/org/apache/solr/cloud/OverseerModifyCollectionTest.java
index e9cb4d775ba..bae0a0cafb2 100644
--- a/solr/core/src/test/org/apache/solr/cloud/OverseerModifyCollectionTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/OverseerModifyCollectionTest.java
@@ -46,7 +46,7 @@ public class OverseerModifyCollectionTest extends 
SolrCloudTestCase {
         CollectionAdminRequest.modifyCollection(
                 collName, Collections.singletonMap("collection.configName", 
"conf2"))
             .processAndWait(cluster.getSolrClient(), DEFAULT_TIMEOUT);
-    assertEquals(requestStatusState, RequestStatusState.COMPLETED);
+    assertEquals(RequestStatusState.COMPLETED, requestStatusState);
 
     String configName =
         
cluster.getSolrClient().getClusterStateProvider().getCollection(collName).getConfigName();