raboof/pekko
1
0
Fork 0
Code Issues Pull requests Projects Releases 6 Packages Wiki Activity Actions 10

More reasonable DNS timeout avoiding resolution fail in test #29858

Browse source
This commit is contained in:
Johan Andrén 2020-12-07 13:54:20 +01:00
parent 2c659a046e
commit 513d6f5d06
2 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
akka-docs/src/test/java/jdocs/discovery/DnsDiscoveryDocTest.java
View file

@ -18,6 +18,7 @@ import org.scalatestplus.junit.JUnitSuite;
import java.time.Duration;
import java.util.concurrent.CompletionStage;
import java.util.concurrent.TimeUnit;
@SuppressWarnings("unused")
public class DnsDiscoveryDocTest extends JUnitSuite {
@ -42,7 +43,9 @@ public class DnsDiscoveryDocTest extends JUnitSuite {
ServiceDiscovery discovery = Discovery.get(system).discovery();
// ...
CompletionStage<ServiceDiscovery.Resolved> result =
discovery.lookup("akka.io", Duration.ofMillis(500));
discovery.lookup("akka.io", Duration.ofSeconds(2));
// #lookup-dns
result.toCompletableFuture().get(3, TimeUnit.SECONDS);
}
}

3
akka-docs/src/test/scala/docs/discovery/DnsDiscoveryDocSpec.scala
View file

@ -32,8 +32,9 @@ class DnsDiscoveryDocSpec extends AkkaSpec(DnsDiscoveryDocSpec.config) {
val discovery: ServiceDiscovery = Discovery(system).discovery
// ...
val result: Future[ServiceDiscovery.Resolved] = discovery.lookup("akka.io", resolveTimeout = 500.millis)
val result: Future[ServiceDiscovery.Resolved] = discovery.lookup("akka.io", resolveTimeout = 2.seconds)
// #lookup-dns
val resolved = result.futureValue
resolved.serviceName shouldBe "akka.io"
resolved.addresses shouldNot be(Symbol("empty"))