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

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


The following commit(s) were added to refs/heads/main by this push:
     new 05c15e8db73 Tests: SolrJettyTestRule: harden cleanup
05c15e8db73 is described below

commit 05c15e8db7372639fa3cb2e650366eec0acfe41a
Author: David Smiley <[email protected]>
AuthorDate: Fri Nov 28 01:15:12 2025 -0500

    Tests: SolrJettyTestRule: harden cleanup
    
    If we don't insist `jetty` get removed, then one test failure of jetty not 
stopping can lead to follow-on tests failing as they can't start
---
 .../src/java/org/apache/solr/util/SolrJettyTestRule.java             | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/solr/test-framework/src/java/org/apache/solr/util/SolrJettyTestRule.java 
b/solr/test-framework/src/java/org/apache/solr/util/SolrJettyTestRule.java
index 061bdaa0d5f..467c6bb2b1b 100644
--- a/solr/test-framework/src/java/org/apache/solr/util/SolrJettyTestRule.java
+++ b/solr/test-framework/src/java/org/apache/solr/util/SolrJettyTestRule.java
@@ -55,9 +55,10 @@ public class SolrJettyTestRule extends SolrClientTestRule {
         throw e;
       } catch (Exception e) {
         throw new RuntimeException(e);
+      } finally {
+        jetty = null;
+        enableProxy = false;
       }
-      jetty = null;
-      enableProxy = false;
     }
   }
 

Reply via email to