Skip to content

Commit 0e84c31

Browse files
committed
make benchmark unit test easier to run with startlocal
1 parent 70e1018 commit 0e84c31

5 files changed

Lines changed: 21 additions & 9 deletions

File tree

java-client/src/test/java/co/elastic/clients/elasticsearch/ElasticsearchTestClient.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,21 +57,23 @@ private static JsonpMapper mapper(JsonpMapper mapper) {
5757
return mapper != null ? mapper : new JsonbJsonpMapper();
5858
}
5959

60-
public static ElasticsearchClient createClient(String url, @Nullable JsonpMapper mapper, @Nullable SSLContext sslContext) {
60+
public static ElasticsearchClient createClient(String url, @Nullable JsonpMapper mapper, @Nullable SSLContext sslContext, @Nullable String password) {
6161
System.out.println("Using a " + flavor + " client");
6262

63+
String pwd = password == null ? "changeme" : password;
64+
6365
return ElasticsearchClient.of(b -> b
6466
.host(url)
6567
.jsonMapper(mapper(mapper))
66-
.usernameAndPassword("elastic", "changeme")
68+
.usernameAndPassword("elastic", pwd)
6769
.sslContext(sslContext)
6870
.transportFactory(flavor.transportFactory())
6971
);
7072
}
7173

7274
public static ElasticsearchClient createClient(HttpServer server, @Nullable JsonpMapper mapper) {
7375
var address = server.getAddress();
74-
return createClient("http://" + address.getHostString() + ":" + address.getPort(), mapper, null);
76+
return createClient("http://" + address.getHostString() + ":" + address.getPort(), mapper, null, null);
7577
}
7678

7779
public static Function<ElasticsearchTransportConfig, ElasticsearchTransport> transportFactory() {

java-client/src/test/java/co/elastic/clients/elasticsearch/ElasticsearchTestServer.java

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,10 @@
3030
import org.testcontainers.elasticsearch.ElasticsearchContainer;
3131
import org.testcontainers.images.builder.ImageFromDockerfile;
3232
import org.testcontainers.shaded.org.apache.commons.io.IOUtils;
33+
import org.testcontainers.shaded.org.bouncycastle.cert.ocsp.Req;
3334
import org.testcontainers.utility.DockerImageName;
3435

36+
import javax.annotation.Nullable;
3537
import javax.net.ssl.SSLContext;
3638
import java.io.IOException;
3739
import java.io.InputStream;
@@ -59,15 +61,20 @@ public class ElasticsearchTestServer implements AutoCloseable {
5961
".co/elasticsearch/latest/master.json";
6062

6163
public static synchronized ElasticsearchTestServer global() {
64+
return global(null);
65+
}
66+
67+
public static synchronized ElasticsearchTestServer global(@Nullable String password) {
6268
if (global == null) {
6369

64-
// Try localhost:9200
70+
// Try localhost:9200 with set password
6571
try {
72+
String pwd = password == null ? "changeme" : password;
6673
String localUrl = "http://localhost:9200";
6774
HttpURLConnection connection = (HttpURLConnection) new URL(localUrl).openConnection();
6875
connection.setRequestProperty("Authorization", "Basic " +
6976
Base64.getEncoder().encodeToString(("elastic" +
70-
":changeme").getBytes(StandardCharsets.UTF_8)));
77+
":" + pwd).getBytes(StandardCharsets.UTF_8)));
7178

7279
try (InputStream input = connection.getInputStream()) {
7380
String content = IOUtils.toString(input, StandardCharsets.UTF_8);
@@ -103,7 +110,7 @@ public ElasticsearchTestServer(String... plugins) {
103110
protected void setup(String url, SSLContext sslContext) {
104111
this.url = url;
105112
this.sslContext = sslContext;
106-
this.client = ElasticsearchTestClient.createClient(url, null, sslContext);
113+
this.client = ElasticsearchTestClient.createClient(url, null, sslContext, null);
107114
}
108115

109116
private Version selectLatestVersion(Version version) {

java-client/src/test/java/co/elastic/clients/elasticsearch/_helpers/esql/EsqlAdapterEndToEndTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public class EsqlAdapterEndToEndTest extends Assertions {
5555
@BeforeAll
5656
public static void setup() throws Exception {
5757
var server = ElasticsearchTestServer.global();
58-
esClient = ElasticsearchTestClient.createClient(server.url(), new JacksonJsonpMapper(), server.sslContext());
58+
esClient = ElasticsearchTestClient.createClient(server.url(), new JacksonJsonpMapper(), server.sslContext(), null);
5959

6060
// Make sure index is empty
6161
esClient.indices().delete(d -> d.index("employees").ignoreUnavailable(true));

java-client/src/test/java/co/elastic/clients/elasticsearch/experiments/benchmark/base64vectors/MainCommonBenchmark64.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,11 @@
2020
package co.elastic.clients.elasticsearch.experiments.benchmark.base64vectors;
2121

2222
import co.elastic.clients.elasticsearch.ElasticsearchClient;
23+
import co.elastic.clients.elasticsearch.ElasticsearchTestClient;
2324
import co.elastic.clients.elasticsearch.ElasticsearchTestServer;
2425
import co.elastic.clients.elasticsearch.core.BulkRequest;
2526
import co.elastic.clients.elasticsearch.core.bulk.BulkOperation;
27+
import co.elastic.clients.json.jackson.JacksonJsonpMapper;
2628
import com.fasterxml.jackson.databind.MappingIterator;
2729
import com.fasterxml.jackson.databind.ObjectMapper;
2830
import com.fasterxml.jackson.databind.ObjectWriter;
@@ -51,7 +53,8 @@ public class MainCommonBenchmark64 {
5153

5254
@BeforeAll
5355
public static void setup() {
54-
elasticsearchClient = ElasticsearchTestServer.global().client();
56+
var server = ElasticsearchTestServer.global();
57+
elasticsearchClient = ElasticsearchTestClient.createClient(server.url(), new JacksonJsonpMapper(), server.sslContext(), null);
5558
}
5659

5760
public static List<ElasticsearchDoc> readMultipleObjects(InputStream file) throws IOException {

java-client/src/test/java/co/elastic/clients/transport/TransportUtilsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public void testCaCertificate() throws Exception {
8383

8484
private void checkConnection(SSLContext sslContext) throws Exception {
8585
var server = ElasticsearchTestServer.global();
86-
var esClient = ElasticsearchTestClient.createClient(server.url(), null, sslContext);
86+
var esClient = ElasticsearchTestClient.createClient(server.url(), null, sslContext, null);
8787

8888
assertNotNull(esClient.info());
8989
}

0 commit comments

Comments
 (0)