You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by ds...@apache.org on 2023/11/20 06:47:14 UTC
(solr) branch main updated: SOLR-17079: fix test; close CoreContainer Ignore OverseerCollectionConfigSetProcessorTest temporarily...
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 4a9f98d6836 SOLR-17079: fix test; close CoreContainer Ignore OverseerCollectionConfigSetProcessorTest temporarily...
4a9f98d6836 is described below
commit 4a9f98d6836c5fb9cfb2b4cdf0bb4096259fbd24
Author: David Smiley <ds...@salesforce.com>
AuthorDate: Mon Nov 20 01:46:34 2023 -0500
SOLR-17079: fix test; close CoreContainer
Ignore OverseerCollectionConfigSetProcessorTest temporarily...
---
.../apache/solr/cloud/OverseerCollectionConfigSetProcessorTest.java | 2 ++
.../solr/cluster/placement/impl/PlacementPluginIntegrationTest.java | 3 +++
2 files changed, 5 insertions(+)
diff --git a/solr/core/src/test/org/apache/solr/cloud/OverseerCollectionConfigSetProcessorTest.java b/solr/core/src/test/org/apache/solr/cloud/OverseerCollectionConfigSetProcessorTest.java
index ac3df8b4373..24140b5c3c0 100644
--- a/solr/core/src/test/org/apache/solr/cloud/OverseerCollectionConfigSetProcessorTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/OverseerCollectionConfigSetProcessorTest.java
@@ -88,6 +88,7 @@ import org.junit.After;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.BeforeClass;
+import org.junit.Ignore;
import org.junit.Test;
import org.mockito.ArgumentCaptor;
import org.mockito.Mockito;
@@ -96,6 +97,7 @@ import org.mockito.stubbing.Answer;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+@Ignore("VERY temporary, SOLR-17079")
public class OverseerCollectionConfigSetProcessorTest extends SolrTestCaseJ4 {
private static final Logger log = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
diff --git a/solr/core/src/test/org/apache/solr/cluster/placement/impl/PlacementPluginIntegrationTest.java b/solr/core/src/test/org/apache/solr/cluster/placement/impl/PlacementPluginIntegrationTest.java
index bcac9882e2c..fadc23c6b27 100644
--- a/solr/core/src/test/org/apache/solr/cluster/placement/impl/PlacementPluginIntegrationTest.java
+++ b/solr/core/src/test/org/apache/solr/cluster/placement/impl/PlacementPluginIntegrationTest.java
@@ -111,6 +111,7 @@ public class PlacementPluginIntegrationTest extends SolrCloudTestCase {
MatcherAssert.assertThat(
cc.getPlacementPluginFactory().createPluginInstance(),
instanceOf(SimplePlacementFactory.SimplePlacementPlugin.class));
+ cc.shutdown();
}
@Test
@@ -120,6 +121,7 @@ public class PlacementPluginIntegrationTest extends SolrCloudTestCase {
MatcherAssert.assertThat(
cc.getPlacementPluginFactory().createPluginInstance(),
instanceOf(RandomPlacementFactory.RandomPlacementPlugin.class));
+ cc.shutdown();
}
@Test
@@ -138,6 +140,7 @@ public class PlacementPluginIntegrationTest extends SolrCloudTestCase {
(AffinityPlacementConfig) cc.getPlacementPluginFactory().getConfig();
assertEquals(config.minimalFreeDiskGB, 10);
assertEquals(config.prioritizedFreeDiskGB, 200);
+ cc.shutdown();
}
@Test