Skip to content

Commit e406790

Browse files
committed
[grid] Fixing RelayOptionsTest
1 parent 03c8b2f commit e406790

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

‎java/test/org/openqa/selenium/grid/node/relay/RelayOptionsTest.java

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

1818
package org.openqa.selenium.grid.node.relay;
1919

20-
import static org.assertj.core.api.Assertions.assertThat;
21-
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
22-
2320
import org.junit.Test;
2421
import org.openqa.selenium.Capabilities;
2522
import org.openqa.selenium.grid.config.Config;
@@ -32,9 +29,13 @@
3229
import org.openqa.selenium.remote.tracing.Tracer;
3330

3431
import java.io.StringReader;
32+
import java.time.Duration;
3533
import java.util.Collection;
3634
import java.util.Map;
3735

36+
import static org.assertj.core.api.Assertions.assertThat;
37+
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
38+
3839
@SuppressWarnings("DuplicatedCode")
3940
public class RelayOptionsTest {
4041

@@ -51,7 +52,10 @@ public void basicConfigurationIsParsedSuccessfully() {
5152
HttpClient.Factory httpClientFactory = networkOptions.getHttpClientFactory(tracer);
5253
RelayOptions relayOptions = new RelayOptions(config);
5354
Map<Capabilities, Collection<SessionFactory>>
54-
sessionFactories = relayOptions.getSessionFactories(tracer, httpClientFactory);
55+
sessionFactories = relayOptions.getSessionFactories(
56+
tracer,
57+
httpClientFactory,
58+
Duration.ofSeconds(300));
5559

5660
Capabilities chrome = sessionFactories
5761
.keySet()
@@ -105,7 +109,8 @@ public void missingConfigsThrowsConfigException() {
105109
Tracer tracer = DefaultTestTracer.createTracer();
106110
HttpClient.Factory httpClientFactory = networkOptions.getHttpClientFactory(tracer);
107111
assertThatExceptionOfType(ConfigException.class)
108-
.isThrownBy(() -> new RelayOptions(config).getSessionFactories(tracer, httpClientFactory));
112+
.isThrownBy(() -> new RelayOptions(config)
113+
.getSessionFactories(tracer, httpClientFactory, Duration.ofSeconds(300)));
109114
}
110115

111116
@Test
@@ -121,7 +126,8 @@ public void incompleteConfigsThrowsConfigException() {
121126
Tracer tracer = DefaultTestTracer.createTracer();
122127
HttpClient.Factory httpClientFactory = networkOptions.getHttpClientFactory(tracer);
123128
assertThatExceptionOfType(ConfigException.class)
124-
.isThrownBy(() -> new RelayOptions(config).getSessionFactories(tracer, httpClientFactory));
129+
.isThrownBy(() -> new RelayOptions(config)
130+
.getSessionFactories(tracer, httpClientFactory, Duration.ofSeconds(300)));
125131
}
126132

127133
}

0 commit comments

Comments
 (0)