nit0906 commented on code in PR #568:
URL: https://github.com/apache/jackrabbit-oak/pull/568#discussion_r877707256
##########
oak-search-elastic/src/test/java/org/apache/jackrabbit/oak/plugins/index/elastic/ElasticIndexProviderServiceTest.java:
##########
@@ -95,6 +96,8 @@ public void defaultSetup() {
@Test
public void withElasticSetup() throws Exception {
+ // Skips this test when a single Elasticsearch cluster is re-used
among test executionsgit , see the ElasticTestServer main.
+ assumeTrue(elasticRule.useDocker());
Review Comment:
The assumeTrue(elasticRule.useDocker()) statement should also be removed
from both the tests in which it was added. Otherwise they will only run on a
docker based setup.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
