Fixing Rest Converters Tests after x-pack removal (#54)

Signed-off-by: Peter Nied <petern@amazon.com>
This commit is contained in:
Sarat Vemulapalli 2021-02-06 14:06:51 -08:00 committed by Peter Nied
parent f2a90be773
commit ae46de3125
1 changed files with 0 additions and 47 deletions

View File

@ -199,13 +199,6 @@ public class RequestConvertersTests extends ESTestCase {
expectedParams.put("realtime", "false"); expectedParams.put("realtime", "false");
} }
} }
if (randomBoolean()) {
boolean refresh = randomBoolean();
getRequest.refresh(refresh);
if (refresh) {
expectedParams.put("refresh", "true");
}
}
Request request = RequestConverters.sourceExists(getRequest); Request request = RequestConverters.sourceExists(getRequest);
assertEquals(HttpHead.METHOD_NAME, request.getMethod()); assertEquals(HttpHead.METHOD_NAME, request.getMethod());
String type = getRequest.type(); String type = getRequest.type();
@ -242,13 +235,6 @@ public class RequestConvertersTests extends ESTestCase {
expectedParams.put("realtime", "false"); expectedParams.put("realtime", "false");
} }
} }
if (randomBoolean()) {
boolean refresh = randomBoolean();
getRequest.refresh(refresh);
if (refresh) {
expectedParams.put("refresh", "true");
}
}
Request request = RequestConverters.getSource(getRequest); Request request = RequestConverters.getSource(getRequest);
assertEquals(HttpGet.METHOD_NAME, request.getMethod()); assertEquals(HttpGet.METHOD_NAME, request.getMethod());
assertEquals("/" + index + "/_source/" + id, request.getEndpoint()); assertEquals("/" + index + "/_source/" + id, request.getEndpoint());
@ -271,12 +257,6 @@ public class RequestConvertersTests extends ESTestCase {
expectedParams.put("realtime", "false"); expectedParams.put("realtime", "false");
} }
} }
if (randomBoolean()) {
multiGetRequest.refresh(randomBoolean());
if (multiGetRequest.refresh()) {
expectedParams.put("refresh", "true");
}
}
int numberOfRequests = randomIntBetween(0, 32); int numberOfRequests = randomIntBetween(0, 32);
for (int i = 0; i < numberOfRequests; i++) { for (int i = 0; i < numberOfRequests; i++) {
@ -327,7 +307,6 @@ public class RequestConvertersTests extends ESTestCase {
Map<String, String> expectedParams = new HashMap<>(); Map<String, String> expectedParams = new HashMap<>();
setRandomTimeout(deleteRequest::timeout, ReplicationRequest.DEFAULT_TIMEOUT, expectedParams); setRandomTimeout(deleteRequest::timeout, ReplicationRequest.DEFAULT_TIMEOUT, expectedParams);
setRandomRefreshPolicy(deleteRequest::setRefreshPolicy, expectedParams);
setRandomVersion(deleteRequest, expectedParams); setRandomVersion(deleteRequest, expectedParams);
setRandomVersionType(deleteRequest::versionType, expectedParams); setRandomVersionType(deleteRequest::versionType, expectedParams);
setRandomIfSeqNoAndTerm(deleteRequest, expectedParams); setRandomIfSeqNoAndTerm(deleteRequest, expectedParams);
@ -391,13 +370,6 @@ public class RequestConvertersTests extends ESTestCase {
expectedParams.put("realtime", "false"); expectedParams.put("realtime", "false");
} }
} }
if (randomBoolean()) {
boolean refresh = randomBoolean();
getRequest.refresh(refresh);
if (refresh) {
expectedParams.put("refresh", "true");
}
}
if (randomBoolean()) { if (randomBoolean()) {
long version = randomLong(); long version = randomLong();
getRequest.version(version); getRequest.version(version);
@ -721,7 +693,6 @@ public class RequestConvertersTests extends ESTestCase {
} }
setRandomTimeout(indexRequest::timeout, ReplicationRequest.DEFAULT_TIMEOUT, expectedParams); setRandomTimeout(indexRequest::timeout, ReplicationRequest.DEFAULT_TIMEOUT, expectedParams);
setRandomRefreshPolicy(indexRequest::setRefreshPolicy, expectedParams);
// There is some logic around _create endpoint and version/version type // There is some logic around _create endpoint and version/version type
if (indexRequest.opType() == DocWriteRequest.OpType.CREATE) { if (indexRequest.opType() == DocWriteRequest.OpType.CREATE) {
@ -858,13 +829,6 @@ public class RequestConvertersTests extends ESTestCase {
} else { } else {
expectedParams.put("timeout", ReplicationRequest.DEFAULT_TIMEOUT.getStringRep()); expectedParams.put("timeout", ReplicationRequest.DEFAULT_TIMEOUT.getStringRep());
} }
if (randomBoolean()) {
WriteRequest.RefreshPolicy refreshPolicy = randomFrom(WriteRequest.RefreshPolicy.values());
updateRequest.setRefreshPolicy(refreshPolicy);
if (refreshPolicy != WriteRequest.RefreshPolicy.NONE) {
expectedParams.put("refresh", refreshPolicy.getValue());
}
}
setRandomWaitForActiveShards(updateRequest::waitForActiveShards, expectedParams); setRandomWaitForActiveShards(updateRequest::waitForActiveShards, expectedParams);
setRandomIfSeqNoAndTerm(updateRequest, new HashMap<>()); // if* params are passed in the body setRandomIfSeqNoAndTerm(updateRequest, new HashMap<>()); // if* params are passed in the body
if (randomBoolean()) { if (randomBoolean()) {
@ -967,7 +931,6 @@ public class RequestConvertersTests extends ESTestCase {
expectedParams.put("timeout", BulkShardRequest.DEFAULT_TIMEOUT.getStringRep()); expectedParams.put("timeout", BulkShardRequest.DEFAULT_TIMEOUT.getStringRep());
} }
setRandomRefreshPolicy(bulkRequest::setRefreshPolicy, expectedParams);
XContentType xContentType = randomFrom(XContentType.JSON, XContentType.SMILE); XContentType xContentType = randomFrom(XContentType.JSON, XContentType.SMILE);
@ -2240,16 +2203,6 @@ public class RequestConvertersTests extends ESTestCase {
} }
} }
private static void setRandomRefreshPolicy(Consumer<WriteRequest.RefreshPolicy> setter, Map<String, String> expectedParams) {
if (randomBoolean()) {
WriteRequest.RefreshPolicy refreshPolicy = randomFrom(WriteRequest.RefreshPolicy.values());
setter.accept(refreshPolicy);
if (refreshPolicy != WriteRequest.RefreshPolicy.NONE) {
expectedParams.put("refresh", refreshPolicy.getValue());
}
}
}
private static void setRandomVersion(DocWriteRequest<?> request, Map<String, String> expectedParams) { private static void setRandomVersion(DocWriteRequest<?> request, Map<String, String> expectedParams) {
if (randomBoolean()) { if (randomBoolean()) {
long version = randomFrom(Versions.MATCH_ANY, Versions.MATCH_DELETED, Versions.NOT_FOUND, randomNonNegativeLong()); long version = randomFrom(Versions.MATCH_ANY, Versions.MATCH_DELETED, Versions.NOT_FOUND, randomNonNegativeLong());