[ https://issues.apache.org/jira/browse/CLOUDSTACK-8981?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14971457#comment-14971457 ]
ASF GitHub Bot commented on CLOUDSTACK-8981: -------------------------------------------- Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/965#issuecomment-150646084 Compilation and testing went successfull. As it only involved a change of a unit test, I hadn't ran integration tests. ``` Running com.cloud.utils.rest.BasicRestClientTest 2015-10-23 18:51:01,522 DEBUG [utils.rest.BasicRestClient] (main:) Executig GET request on https://fakelocalhost.fakelocaldomain:443/path 2015-10-23 18:51:01,525 DEBUG [utils.rest.BasicRestClient] (main:) Executig GET request on https://fakelocalhost.fakelocaldomain:443/path 2015-10-23 18:51:01,584 DEBUG [utils.rest.BasicRestClient] (main:) Executig GET request on https://fakelocalhost.fakelocaldomain:443/path Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.083 sec - in com.cloud.utils.rest.BasicRestClientTest ``` Ping @DaanHoogland @remibergsma LGTM :+1: > test fails when port is reachable > --------------------------------- > > Key: CLOUDSTACK-8981 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8981 > Project: CloudStack > Issue Type: Test > Security Level: Public(Anyone can view this level - this is the > default.) > Reporter: Daan Hoogland > > BasicRestClientTest.testExecuteRequestWhenClientThrowsIOException fails when > the address that is tested against can be reached because of a developers > machine configuration. A clear message is needed when the exception is not > thrown. -- This message was sent by Atlassian JIRA (v6.3.4#6332)