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;
}
}