[
https://issues.apache.org/jira/browse/FLINK-12646?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Till Rohrmann resolved FLINK-12646.
-----------------------------------
Resolution: Fixed
Fix Version/s: 1.8.1
1.7.3
Fixed via
1.9.0: 7ee00fd31c5d373d376fd4d8aa498fc8fe267edb
1.8.1: 5dec698fcdf5503b608d6cc0757c40054a46b2f9
1.7.3: dce00e173ac6cda9541d4378dcddeb827857c44c
> Fix broken tests of RestClientTest
> ----------------------------------
>
> Key: FLINK-12646
> URL: https://issues.apache.org/jira/browse/FLINK-12646
> Project: Flink
> Issue Type: Bug
> Components: Runtime / REST
> Reporter: Victor Wong
> Assignee: Victor Wong
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.7.3, 1.9.0, 1.8.1
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> In
> {code:java}
> org.apache.flink.runtime.rest.RestClientTest#testConnectionTimeout
> {code}
> , we use a "unroutableIp" with a value of "10.255.255.1" for test.
> But sometimes this IP is reachable in a private network of a company, which
> is the case for me. As a result, this test failed with a following exception:
>
> {code:java}
> java.lang.AssertionError: Expected: an instance of
> org.apache.flink.shaded.netty4.io.netty.channel.ConnectTimeoutException but:
> <org.apache.flink.shaded.netty4.io.netty.channel.AbstractChannel$AnnotatedConnectException:
> Connection refused: /10.255.255.1:80> is a
> org.apache.flink.shaded.netty4.io.netty.channel.AbstractChannel$AnnotatedConnectException
> at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20) at
> org.junit.Assert.assertThat(Assert.java:956) at
> org.junit.Assert.assertThat(Assert.java:923) at
> org.apache.flink.runtime.rest.RestClientTest.testConnectionTimeout(RestClientTest.java:76)
> ...
> {code}
>
>
> Can we change the `unroutableIp` to a reserved IP address, i.e "240.0.0.0",
> which is described as _Reserved for future use_ in
> [wikipedia|https://en.wikipedia.org/wiki/Reserved_IP_addresses]
> Or change the assertion?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
