tflobbe commented on pull request #2411:
URL: https://github.com/apache/lucene-solr/pull/2411#issuecomment-786327686
I don't think this will work well. Gradle runs multiple tests on the same
JVM, static state will span multiple tests (so, depending on the order of
tests, this could conflict with TestConfigSetsAPI or other classes testing auth
or causing other inconsistencies).
How about replacing this code in `RoutedAliasUpdateProcessorTest`:
```
assertEquals(0, new ConfigSetAdminRequest.Create()
.setConfigSetName(configName)
.setBaseConfigSetName("_default")
.process(getSolrClient()).getStatus());
```
With something like:
```
new ZkConfigManager(cluster.getZkClient()).copyConfigDir("_default",
configName);
```
Would that work for you? You wouldn't be using the Configsets API, but I
think that's OK, it's not what this class is trying to test?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
