Skip to content

Commit 560eef4

Browse files
committed
fix tests
1 parent 6e2d8a3 commit 560eef4

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

java/src/org/openqa/selenium/remote/http/jdk/JdkHttpClient.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ public class JdkHttpClient implements HttpClient {
8080
private final ExecutorService executorService;
8181
private final Duration readTimeout;
8282
private final Duration connectTimeout;
83-
private final ClientConfig config;
83+
private ClientConfig config;
8484

8585
JdkHttpClient(ClientConfig config) {
8686
Objects.requireNonNull(config, "Client config must be set");
@@ -128,7 +128,7 @@ protected PasswordAuthentication getPasswordAuthentication() {
128128

129129
// Remove credentials from URL
130130
try {
131-
config =
131+
this.config =
132132
ClientConfig.defaultConfig()
133133
.baseUri(
134134
new URI(

java/test/org/openqa/selenium/remote/http/jdk/JdkHttpClientTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,12 @@
1717

1818
package org.openqa.selenium.remote.http.jdk;
1919

20-
import static org.hamcrest.MatcherAssert.assertThat;
2120

2221
import java.net.URI;
2322
import java.net.URISyntaxException;
24-
import org.junit.Test;
23+
import org.junit.jupiter.api.Test;
24+
import static org.assertj.core.api.Assertions.assertThat;
25+
import org.openqa.selenium.remote.http.ClientConfig;
2526
import org.openqa.selenium.remote.http.HttpClient;
2627
import org.openqa.selenium.remote.internal.HttpClientTestBase;
2728

@@ -39,7 +40,6 @@ void shouldStripCredentialsFromUrl() throws URISyntaxException {
3940

4041
JdkHttpClient client = new JdkHttpClient(config);
4142

42-
// Get the modified URI from the client's config
4343
URI modifiedUri = client.getBaseUri();
4444

4545
assertThat(modifiedUri.getUserInfo()).isNull();

0 commit comments

Comments
 (0)