Skip to content

Commit 8dbbc80

Browse files
authored
DATAES-853 - Cleanup tests that do not delete test indices.
Original PR: spring-projects#476
1 parent 8fea655 commit 8dbbc80

File tree

5 files changed

+25
-5
lines changed

5 files changed

+25
-5
lines changed

src/test/java/org/springframework/data/elasticsearch/NestedObjectTests.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import org.apache.lucene.search.join.ScoreMode;
3535
import org.elasticsearch.index.query.BoolQueryBuilder;
3636
import org.elasticsearch.index.query.QueryBuilder;
37+
import org.junit.jupiter.api.AfterEach;
3738
import org.junit.jupiter.api.BeforeEach;
3839
import org.junit.jupiter.api.Test;
3940
import org.springframework.beans.factory.annotation.Autowired;
@@ -67,11 +68,16 @@ public class NestedObjectTests {
6768

6869
@Autowired private ElasticsearchOperations operations;
6970

71+
private final List<Class<?>> entityClasses = Arrays.asList(Book.class, Person.class, PersonMultipleLevelNested.class);
72+
7073
@BeforeEach
7174
public void before() {
72-
IndexInitializer.init(operations.indexOps(Book.class));
73-
IndexInitializer.init(operations.indexOps(Person.class));
74-
IndexInitializer.init(operations.indexOps(PersonMultipleLevelNested.class));
75+
entityClasses.stream().map(operations::indexOps).forEach(IndexInitializer::init);
76+
}
77+
78+
@AfterEach
79+
void tearDown() {
80+
entityClasses.forEach(clazz -> operations.indexOps(clazz).delete());
7581
}
7682

7783
@Test

src/test/java/org/springframework/data/elasticsearch/config/notnested/EnableElasticsearchRepositoriesTests.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import java.lang.Long;
2525
import java.util.UUID;
2626

27+
import org.junit.jupiter.api.AfterEach;
2728
import org.junit.jupiter.api.BeforeEach;
2829
import org.junit.jupiter.api.Test;
2930
import org.springframework.beans.BeansException;
@@ -42,6 +43,7 @@
4243
import org.springframework.data.elasticsearch.core.ElasticsearchOperations;
4344
import org.springframework.data.elasticsearch.core.IndexOperations;
4445
import org.springframework.data.elasticsearch.core.geo.GeoPoint;
46+
import org.springframework.data.elasticsearch.core.mapping.IndexCoordinates;
4547
import org.springframework.data.elasticsearch.junit.jupiter.ElasticsearchRestTemplateConfiguration;
4648
import org.springframework.data.elasticsearch.junit.jupiter.SpringIntegrationTest;
4749
import org.springframework.data.elasticsearch.repository.ElasticsearchRepository;
@@ -90,6 +92,12 @@ public void before() {
9092
IndexInitializer.init(indexOperations);
9193
}
9294

95+
@AfterEach
96+
void tearDown() {
97+
operations.indexOps(IndexCoordinates.of("test-index-sample-config-not-nested")).delete();
98+
operations.indexOps(IndexCoordinates.of("test-index-uuid-keyed-config-not-nested")).delete();
99+
}
100+
93101
@Test
94102
public void bootstrapsRepository() {
95103

src/test/java/org/springframework/data/elasticsearch/core/ElasticsearchTemplateTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,8 @@ private void deleteIndices() {
151151
operations.indexOps(IndexCoordinates.of(INDEX_3_NAME)).delete();
152152
operations.indexOps(SearchHitsEntity.class).delete();
153153
operations.indexOps(HighlightEntity.class).delete();
154+
operations.indexOps(OptimisticEntity.class).delete();
155+
operations.indexOps(OptimisticAndVersionedEntity.class).delete();
154156
}
155157

156158
@Test // DATAES-106

src/test/java/org/springframework/data/elasticsearch/core/ReactiveElasticsearchTemplateTests.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,9 @@ public void after() {
119119
}
120120

121121
private void deleteIndices() {
122-
TestUtils.deleteIndex(DEFAULT_INDEX, ALTERNATE_INDEX, "rx-template-test-index-this", "rx-template-test-index-that");
122+
TestUtils.deleteIndex(DEFAULT_INDEX, ALTERNATE_INDEX, "rx-template-test-index-this", "rx-template-test-index-that",
123+
"test-index-reactive-optimistic-entity-template",
124+
"test-index-reactive-optimistic-and-versioned-entity-template");
123125
}
124126

125127
@Test // DATAES-504

src/test/java/org/springframework/data/elasticsearch/core/index/MappingBuilderTests.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@
4747
import org.assertj.core.data.Percentage;
4848
import org.elasticsearch.search.suggest.completion.context.ContextMapping;
4949
import org.json.JSONException;
50+
import org.junit.jupiter.api.AfterEach;
5051
import org.junit.jupiter.api.BeforeEach;
5152
import org.junit.jupiter.api.Test;
5253
import org.springframework.beans.factory.annotation.Autowired;
@@ -91,8 +92,9 @@ public class MappingBuilderTests extends MappingContextBaseTests {
9192
@Autowired private ElasticsearchOperations operations;
9293
private IndexOperations indexOperations;
9394

95+
@AfterEach
9496
@BeforeEach
95-
public void before() {
97+
public void deleteIndices() {
9698
indexOperations = operations.indexOps(SimpleRecursiveEntity.class);
9799
indexOperations.delete();
98100
operations.indexOps(StockPrice.class).delete();

0 commit comments

Comments
 (0)