Skip to content

Commit 8abfda0

Browse files
jaymodejaymode
authored
Rename assertThrows to prevent naming clash (#52651)
This commit renames ElasticsearchAssertions#assertThrows to assertRequestBuilderThrows and assertFutureThrows to avoid a naming clash with JUnit 4.13+ and static imports of these methods. Additionally, these methods have been updated to make use of expectThrows internally to avoid duplicating the logic there. Relates #51787 Backport of #52582
1 parent 3840a76 commit 8abfda0

File tree

19 files changed

+147
-145
lines changed

19 files changed

+147
-145
lines changed

‎server/src/test/java/org/elasticsearch/action/admin/cluster/node/tasks/TasksIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,9 @@
8585
import static org.elasticsearch.common.unit.TimeValue.timeValueMillis;
8686
import static org.elasticsearch.common.unit.TimeValue.timeValueSeconds;
8787
import static org.elasticsearch.http.HttpTransportSettings.SETTING_HTTP_MAX_HEADER_SIZE;
88+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertFutureThrows;
8889
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertNoFailures;
8990
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertSearchResponse;
90-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
9191
import static org.hamcrest.Matchers.allOf;
9292
import static org.hamcrest.Matchers.containsString;
9393
import static org.hamcrest.Matchers.empty;
@@ -785,7 +785,7 @@ public void testTaskStoringFailureResult() throws Exception {
785785
request.setShouldBlock(false);
786786

787787
// Start non-blocking test task that should fail
788-
assertThrows(
788+
assertFutureThrows(
789789
client().execute(TestTaskPlugin.TestTaskAction.INSTANCE, request),
790790
IllegalStateException.class
791791
);

‎server/src/test/java/org/elasticsearch/action/admin/indices/create/CreateIndexIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
import static org.elasticsearch.common.xcontent.XContentFactory.jsonBuilder;
5555
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
5656
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertBlocked;
57-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
57+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
5858
import static org.hamcrest.Matchers.allOf;
5959
import static org.hamcrest.Matchers.equalTo;
6060
import static org.hamcrest.Matchers.greaterThanOrEqualTo;
@@ -341,7 +341,7 @@ public void testFailureToCreateIndexCleansUpIndicesService() {
341341
.addAlias(new Alias("alias1").writeIndex(true))
342342
.get());
343343

344-
assertThrows(client().admin().indices().prepareCreate("test-idx-2")
344+
assertRequestBuilderThrows(client().admin().indices().prepareCreate("test-idx-2")
345345
.setSettings(settings)
346346
.addAlias(new Alias("alias1").writeIndex(true)),
347347
IllegalStateException.class);

‎server/src/test/java/org/elasticsearch/action/admin/indices/exists/IndicesExistsIT.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@
2828
import org.elasticsearch.test.InternalTestCluster;
2929

3030
import java.io.IOException;
31-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
31+
32+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
3233

3334
@ClusterScope(scope = ESIntegTestCase.Scope.TEST, numDataNodes = 0, numClientNodes = 0, transportClientRatio = 0.0,
3435
autoManageMasterNodes = false)
@@ -40,7 +41,8 @@ public void testIndexExistsWithBlocksInPlace() throws IOException {
4041
.put(GatewayService.RECOVER_AFTER_NODES_SETTING.getKey(), 99).build();
4142
String node = internalCluster().startNode(settings);
4243

43-
assertThrows(client(node).admin().indices().prepareExists("test").setMasterNodeTimeout(TimeValue.timeValueSeconds(0)),
44+
assertRequestBuilderThrows(
45+
client(node).admin().indices().prepareExists("test").setMasterNodeTimeout(TimeValue.timeValueSeconds(0)),
4446
MasterNotDiscoveredException.class);
4547

4648
internalCluster().stopRandomNode(InternalTestCluster.nameFilter(node)); // shut down node so that test properly cleans up

‎server/src/test/java/org/elasticsearch/action/termvectors/GetTermVectorsIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@
5555

5656
import static org.elasticsearch.common.xcontent.XContentFactory.jsonBuilder;
5757
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
58-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
58+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
5959
import static org.hamcrest.Matchers.equalTo;
6060
import static org.hamcrest.Matchers.notNullValue;
6161
import static org.hamcrest.Matchers.nullValue;
@@ -369,7 +369,7 @@ public void testDuelESLucene() throws Exception {
369369
for (TestConfig test : testConfigs) {
370370
TermVectorsRequestBuilder request = getRequestForConfig(test);
371371
if (test.expectedException != null) {
372-
assertThrows(request, test.expectedException);
372+
assertRequestBuilderThrows(request, test.expectedException);
373373
continue;
374374
}
375375

‎server/src/test/java/org/elasticsearch/cluster/NoMasterNodeIT.java

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353

5454
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertExists;
5555
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertHitCount;
56-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
56+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
5757
import static org.hamcrest.Matchers.equalTo;
5858
import static org.hamcrest.Matchers.greaterThan;
5959

@@ -96,35 +96,35 @@ public void testNoMasterActions() throws Exception {
9696
assertTrue(state.blocks().hasGlobalBlockWithId(NoMasterBlockService.NO_MASTER_BLOCK_ID));
9797
});
9898

99-
assertThrows(clientToMasterlessNode.prepareGet("test", "type1", "1"),
99+
assertRequestBuilderThrows(clientToMasterlessNode.prepareGet("test", "type1", "1"),
100100
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
101101
);
102102

103-
assertThrows(clientToMasterlessNode.prepareGet("no_index", "type1", "1"),
103+
assertRequestBuilderThrows(clientToMasterlessNode.prepareGet("no_index", "type1", "1"),
104104
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
105105
);
106106

107-
assertThrows(clientToMasterlessNode.prepareMultiGet().add("test", "type1", "1"),
107+
assertRequestBuilderThrows(clientToMasterlessNode.prepareMultiGet().add("test", "type1", "1"),
108108
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
109109
);
110110

111-
assertThrows(clientToMasterlessNode.prepareMultiGet().add("no_index", "type1", "1"),
111+
assertRequestBuilderThrows(clientToMasterlessNode.prepareMultiGet().add("no_index", "type1", "1"),
112112
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
113113
);
114114

115-
assertThrows(clientToMasterlessNode.admin().indices().prepareAnalyze("test", "this is a test"),
115+
assertRequestBuilderThrows(clientToMasterlessNode.admin().indices().prepareAnalyze("test", "this is a test"),
116116
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
117117
);
118118

119-
assertThrows(clientToMasterlessNode.admin().indices().prepareAnalyze("no_index", "this is a test"),
119+
assertRequestBuilderThrows(clientToMasterlessNode.admin().indices().prepareAnalyze("no_index", "this is a test"),
120120
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
121121
);
122122

123-
assertThrows(clientToMasterlessNode.prepareSearch("test").setSize(0),
123+
assertRequestBuilderThrows(clientToMasterlessNode.prepareSearch("test").setSize(0),
124124
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
125125
);
126126

127-
assertThrows(clientToMasterlessNode.prepareSearch("no_index").setSize(0),
127+
assertRequestBuilderThrows(clientToMasterlessNode.prepareSearch("no_index").setSize(0),
128128
ClusterBlockException.class, RestStatus.SERVICE_UNAVAILABLE
129129
);
130130

‎server/src/test/java/org/elasticsearch/indices/settings/UpdateSettingsIT.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
import static org.elasticsearch.cluster.metadata.IndexMetaData.SETTING_READ_ONLY;
4646
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
4747
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertBlocked;
48-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
48+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
4949
import static org.hamcrest.Matchers.containsString;
5050
import static org.hamcrest.Matchers.equalTo;
5151
import static org.hamcrest.Matchers.greaterThan;
@@ -452,7 +452,8 @@ public void testEngineGCDeletesSetting() throws Exception {
452452
client().prepareIndex("test", "type", "1").setSource("f", 1).setVersionType(VersionType.EXTERNAL).setVersion(1).get();
453453
client().prepareDelete("test", "type", "1").setVersionType(VersionType.EXTERNAL).setVersion(2).get();
454454
// delete is still in cache this should fail
455-
assertThrows(client().prepareIndex("test", "type", "1").setSource("f", 3).setVersionType(VersionType.EXTERNAL).setVersion(1),
455+
assertRequestBuilderThrows(
456+
client().prepareIndex("test", "type", "1").setSource("f", 3).setVersionType(VersionType.EXTERNAL).setVersion(1),
456457
VersionConflictEngineException.class);
457458
assertAcked(client().admin().indices().prepareUpdateSettings("test").setSettings(Settings.builder().put("index.gc_deletes", 0)));
458459

‎server/src/test/java/org/elasticsearch/indices/template/SimpleIndexTemplateIT.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@
5656
import static org.elasticsearch.index.query.QueryBuilders.termQuery;
5757
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
5858
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertHitCount;
59-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
59+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
6060
import static org.hamcrest.Matchers.anyOf;
6161
import static org.hamcrest.Matchers.containsInAnyOrder;
6262
import static org.hamcrest.Matchers.containsString;
@@ -109,7 +109,7 @@ public void testSimpleIndexTemplateTests() throws Exception {
109109
.get();
110110

111111
// test create param
112-
assertThrows(client().admin().indices().preparePutTemplate("template_2")
112+
assertRequestBuilderThrows(client().admin().indices().preparePutTemplate("template_2")
113113
.setPatterns(Collections.singletonList("test*"))
114114
.setSettings(indexSettings())
115115
.setCreate(true)
@@ -329,7 +329,7 @@ public void testThatInvalidGetIndexTemplatesFails() throws Exception {
329329
}
330330

331331
private void testExpectActionRequestValidationException(String... names) {
332-
assertThrows(client().admin().indices().prepareGetTemplates(names),
332+
assertRequestBuilderThrows(client().admin().indices().prepareGetTemplates(names),
333333
ActionRequestValidationException.class,
334334
"get template with " + Arrays.toString(names));
335335
}

‎server/src/test/java/org/elasticsearch/persistent/PersistentTasksExecutorIT.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -25,16 +25,16 @@
2525
import org.elasticsearch.common.UUIDs;
2626
import org.elasticsearch.common.settings.Settings;
2727
import org.elasticsearch.common.unit.TimeValue;
28-
import org.elasticsearch.plugins.Plugin;
29-
import org.elasticsearch.tasks.TaskId;
30-
import org.elasticsearch.tasks.TaskInfo;
31-
import org.elasticsearch.test.ESIntegTestCase;
3228
import org.elasticsearch.persistent.PersistentTasksCustomMetaData.PersistentTask;
3329
import org.elasticsearch.persistent.PersistentTasksService.WaitForPersistentTaskListener;
3430
import org.elasticsearch.persistent.TestPersistentTasksPlugin.State;
35-
import org.elasticsearch.persistent.TestPersistentTasksPlugin.TestPersistentTasksExecutor;
3631
import org.elasticsearch.persistent.TestPersistentTasksPlugin.TestParams;
32+
import org.elasticsearch.persistent.TestPersistentTasksPlugin.TestPersistentTasksExecutor;
3733
import org.elasticsearch.persistent.TestPersistentTasksPlugin.TestTasksRequestBuilder;
34+
import org.elasticsearch.plugins.Plugin;
35+
import org.elasticsearch.tasks.TaskId;
36+
import org.elasticsearch.tasks.TaskInfo;
37+
import org.elasticsearch.test.ESIntegTestCase;
3838
import org.junit.After;
3939
import org.junit.Before;
4040

@@ -43,7 +43,7 @@
4343
import java.util.List;
4444
import java.util.Objects;
4545

46-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
46+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertFutureThrows;
4747
import static org.hamcrest.Matchers.empty;
4848
import static org.hamcrest.Matchers.equalTo;
4949
import static org.hamcrest.Matchers.hasSize;
@@ -137,7 +137,7 @@ public void testPersistentActionCompletion() throws Exception {
137137
//try sending completion request with incorrect allocation id
138138
PlainActionFuture<PersistentTask<?>> failedCompletionNotificationFuture = new PlainActionFuture<>();
139139
persistentTasksService.sendCompletionRequest(taskId, Long.MAX_VALUE, null, failedCompletionNotificationFuture);
140-
assertThrows(failedCompletionNotificationFuture, ResourceNotFoundException.class);
140+
assertFutureThrows(failedCompletionNotificationFuture, ResourceNotFoundException.class);
141141
// Make sure that the task is still running
142142
assertThat(client().admin().cluster().prepareListTasks().setActions(TestPersistentTasksExecutor.NAME + "[c]")
143143
.setDetailed(true).get().getTasks().size(), equalTo(1));
@@ -245,11 +245,11 @@ public void testPersistentActionStatusUpdate() throws Exception {
245245
persistentTasksService.waitForPersistentTaskCondition(taskId,
246246
task -> false, TimeValue.timeValueMillis(10), future1);
247247

248-
assertThrows(future1, IllegalStateException.class, "timed out after 10ms");
248+
assertFutureThrows(future1, IllegalStateException.class, "timed out after 10ms");
249249

250250
PlainActionFuture<PersistentTask<?>> failedUpdateFuture = new PlainActionFuture<>();
251251
persistentTasksService.sendUpdateStateRequest(taskId, -2, new State("should fail"), failedUpdateFuture);
252-
assertThrows(failedUpdateFuture, ResourceNotFoundException.class, "the task with id " + taskId +
252+
assertFutureThrows(failedUpdateFuture, ResourceNotFoundException.class, "the task with id " + taskId +
253253
" and allocation id -2 doesn't exist");
254254

255255
// Wait for the task to disappear
@@ -273,7 +273,7 @@ public void testCreatePersistentTaskWithDuplicateId() throws Exception {
273273

274274
PlainActionFuture<PersistentTask<TestParams>> future2 = new PlainActionFuture<>();
275275
persistentTasksService.sendStartRequest(taskId, TestPersistentTasksExecutor.NAME, new TestParams("Blah"), future2);
276-
assertThrows(future2, ResourceAlreadyExistsException.class);
276+
assertFutureThrows(future2, ResourceAlreadyExistsException.class);
277277

278278
assertBusy(() -> {
279279
// Wait for the task to start

‎server/src/test/java/org/elasticsearch/repositories/blobstore/BlobStoreRepositoryCleanupIT.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
import java.util.concurrent.TimeUnit;
3636

3737
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertAcked;
38-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
38+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertFutureThrows;
3939
import static org.hamcrest.Matchers.is;
4040

4141
@ESIntegTestCase.ClusterScope(scope = ESIntegTestCase.Scope.TEST, numDataNodes = 0)
@@ -59,7 +59,7 @@ public void testRepeatCleanupsDontRemove() throws Exception {
5959
final String masterNode = startBlockedCleanup("test-repo");
6060

6161
logger.info("--> sending another cleanup");
62-
assertThrows(client().admin().cluster().prepareCleanupRepository("test-repo").execute(), IllegalStateException.class);
62+
assertFutureThrows(client().admin().cluster().prepareCleanupRepository("test-repo").execute(), IllegalStateException.class);
6363

6464
logger.info("--> ensure cleanup is still in progress");
6565
final RepositoryCleanupInProgress cleanup =

‎server/src/test/java/org/elasticsearch/search/morelikethis/MoreLikeThisIT.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@
5656
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertHitCount;
5757
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertNoFailures;
5858
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertOrderedSearchHits;
59+
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertRequestBuilderThrows;
5960
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertSearchHits;
6061
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertSearchResponse;
61-
import static org.elasticsearch.test.hamcrest.ElasticsearchAssertions.assertThrows;
6262
import static org.hamcrest.Matchers.equalTo;
6363
import static org.hamcrest.Matchers.instanceOf;
6464
import static org.hamcrest.Matchers.notNullValue;
@@ -339,12 +339,12 @@ public void testNumericField() throws Exception {
339339
assertHitCount(searchResponse, 1L);
340340

341341
// Explicit list of fields including numeric fields -> fail
342-
assertThrows(client().prepareSearch().setQuery(
342+
assertRequestBuilderThrows(client().prepareSearch().setQuery(
343343
new MoreLikeThisQueryBuilder(new String[] {"string_value", "int_value"}, null,
344344
new Item[] {new Item("test", "1")}).minTermFreq(1).minDocFreq(1)), SearchPhaseExecutionException.class);
345345

346346
// mlt query with no field -> exception because _all is not enabled)
347-
assertThrows(client().prepareSearch()
347+
assertRequestBuilderThrows(client().prepareSearch()
348348
.setQuery(moreLikeThisQuery(new String[] {"index"}).minTermFreq(1).minDocFreq(1)),
349349
SearchPhaseExecutionException.class);
350350

@@ -354,12 +354,12 @@ public void testNumericField() throws Exception {
354354
assertHitCount(searchResponse, 2L);
355355

356356
// mlt query with at least a numeric field -> fail by default
357-
assertThrows(client().prepareSearch().setQuery(
357+
assertRequestBuilderThrows(client().prepareSearch().setQuery(
358358
moreLikeThisQuery(new String[] {"string_value", "int_value"}, new String[] {"index"}, null)),
359359
SearchPhaseExecutionException.class);
360360

361361
// mlt query with at least a numeric field -> fail by command
362-
assertThrows(client().prepareSearch().setQuery(
362+
assertRequestBuilderThrows(client().prepareSearch().setQuery(
363363
moreLikeThisQuery(new String[] {"string_value", "int_value"}, new String[] {"index"}, null).failOnUnsupportedField(true)),
364364
SearchPhaseExecutionException.class);
365365

@@ -371,12 +371,12 @@ public void testNumericField() throws Exception {
371371
assertHitCount(searchResponse, 2L);
372372

373373
// mlt field query on a numeric field -> failure by default
374-
assertThrows(client().prepareSearch().setQuery(
374+
assertRequestBuilderThrows(client().prepareSearch().setQuery(
375375
moreLikeThisQuery(new String[] {"int_value"}, new String[] {"42"}, null).minTermFreq(1).minDocFreq(1)),
376376
SearchPhaseExecutionException.class);
377377

378378
// mlt field query on a numeric field -> failure by command
379-
assertThrows(client().prepareSearch().setQuery(
379+
assertRequestBuilderThrows(client().prepareSearch().setQuery(
380380
moreLikeThisQuery(new String[] {"int_value"}, new String[] {"42"}, null).minTermFreq(1).minDocFreq(1)
381381
.failOnUnsupportedField(true)),
382382
SearchPhaseExecutionException.class);

0 commit comments

Comments
 (0)