From 7724ffe071501bcf39ea827a7adcec370c8318ba Mon Sep 17 00:00:00 2001 From: Nik Everett Date: Tue, 20 Oct 2020 13:15:00 -0400 Subject: [PATCH] Make agg test less confusing We have an `@After` annotated method in `AggregatorTestCase` that cleans up releasibles. But it was `private`! Confusingly, it seemed to be working! I'm not sure why. This makes it public which is a little more sensible. --- .../elasticsearch/search/aggregations/AggregatorTestCase.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/framework/src/main/java/org/elasticsearch/search/aggregations/AggregatorTestCase.java b/test/framework/src/main/java/org/elasticsearch/search/aggregations/AggregatorTestCase.java index 743f1833a5131..77a4be0ff9c1e 100644 --- a/test/framework/src/main/java/org/elasticsearch/search/aggregations/AggregatorTestCase.java +++ b/test/framework/src/main/java/org/elasticsearch/search/aggregations/AggregatorTestCase.java @@ -844,7 +844,7 @@ public IndexAnalyzers getIndexAnalyzers() { } @After - private void cleanupReleasables() { + public void cleanupReleasables() { Releasables.close(releasables); releasables.clear(); }