Skip to content

Commit 558650b

Browse files
authored
fix test error (#137078)
1 parent 4a53551 commit 558650b

File tree

2 files changed

+6
-9
lines changed

2 files changed

+6
-9
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -354,9 +354,6 @@ tests:
354354
- class: org.elasticsearch.search.CCSDuelIT
355355
method: testTermsAggsWithProfile
356356
issue: https://github.com/elastic/elasticsearch/issues/132880
357-
- class: org.elasticsearch.xpack.search.AsyncSearchErrorTraceIT
358-
method: testAsyncSearchFailingQueryErrorTraceDefault
359-
issue: https://github.com/elastic/elasticsearch/issues/133010
360357
- class: org.elasticsearch.xpack.security.authc.AuthenticationServiceTests
361358
method: testInvalidToken
362359
issue: https://github.com/elastic/elasticsearch/issues/133328

x-pack/plugin/async-search/src/internalClusterTest/java/org/elasticsearch/xpack/search/AsyncSearchErrorTraceIT.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ public void testAsyncSearchFailingQueryErrorTraceDefault() throws Exception {
7676
createAsyncRequest.addParameter("keep_on_completion", "true");
7777
createAsyncRequest.addParameter("wait_for_completion_timeout", "0ms");
7878
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncRequest);
79-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
79+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
8080
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
8181
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
8282
awaitAsyncRequestDoneRunning(getAsyncRequest);
@@ -103,7 +103,7 @@ public void testAsyncSearchFailingQueryErrorTraceTrue() throws Exception {
103103
createAsyncRequest.addParameter("keep_on_completion", "true");
104104
createAsyncRequest.addParameter("wait_for_completion_timeout", "0ms");
105105
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncRequest);
106-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
106+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
107107
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
108108
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
109109
getAsyncRequest.addParameter("error_trace", "true");
@@ -131,7 +131,7 @@ public void testAsyncSearchFailingQueryErrorTraceFalse() throws Exception {
131131
createAsyncRequest.addParameter("keep_on_completion", "true");
132132
createAsyncRequest.addParameter("wait_for_completion_timeout", "0ms");
133133
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncRequest);
134-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
134+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
135135
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
136136
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
137137
getAsyncRequest.addParameter("error_trace", "false");
@@ -172,7 +172,7 @@ public void testDataNodeLogsStackTrace() throws Exception {
172172
try (var mockLog = MockLog.capture(SearchService.class)) {
173173
ErrorTraceHelper.addSeenLoggingExpectations(numShards, mockLog, errorTriggeringIndex);
174174
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncRequest);
175-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
175+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
176176
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
177177
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
178178
// Use the same value of error_trace as the search request
@@ -206,7 +206,7 @@ public void testAsyncSearchFailingQueryErrorTraceFalseOnSubmitAndTrueOnGet() thr
206206
createAsyncSearchRequest.addParameter("keep_on_completion", "true");
207207
createAsyncSearchRequest.addParameter("wait_for_completion_timeout", "0ms");
208208
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncSearchRequest);
209-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
209+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
210210
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
211211
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
212212
getAsyncRequest.addParameter("error_trace", "true");
@@ -234,7 +234,7 @@ public void testAsyncSearchFailingQueryErrorTraceTrueOnSubmitAndFalseOnGet() thr
234234
createAsyncSearchRequest.addParameter("keep_on_completion", "true");
235235
createAsyncSearchRequest.addParameter("wait_for_completion_timeout", "0ms");
236236
Map<String, Object> createAsyncResponseEntity = performRequestAndGetResponseEntity(createAsyncSearchRequest);
237-
if (createAsyncResponseEntity.get("is_running").equals("true")) {
237+
if (Boolean.TRUE.equals(createAsyncResponseEntity.get("is_running"))) {
238238
String asyncExecutionId = (String) createAsyncResponseEntity.get("id");
239239
Request getAsyncRequest = new Request("GET", "/_async_search/" + asyncExecutionId);
240240
getAsyncRequest.addParameter("error_trace", "false");

0 commit comments

Comments
 (0)