wido commented on code in PR #8389:
URL: https://github.com/apache/cloudstack/pull/8389#discussion_r1692685516
##########
utils/src/test/java/com/cloud/utils/UriUtilsTest.java:
##########
@@ -273,4 +273,16 @@ public void testIsUrlForCompressedFile() {
Assert.assertTrue(UriUtils.isUrlForCompressedFile("https://abc.com/xyz.gz";));
Assert.assertFalse(UriUtils.isUrlForCompressedFile("http://abc.com/xyz.qcow2";));
}
+
+ @Test
+ public void validateUrl() {
+ Pair<String, Integer> url1 =
UriUtils.validateUrl("https://www.cloudstack.org";);
+ Assert.assertEquals(url1.first(), "www.cloudstack.org");
+
+ Pair<String, Integer> url2 =
UriUtils.validateUrl("https://www.apache.org";);
+ Assert.assertEquals(url2.first(), "www.apache.org");
+
+ Pair<String, Integer> url3 =
UriUtils.validateUrl("https://ipv6.google.com";);
+ Assert.assertEquals(url3.first(), "ipv6.google.com");
Review Comment:
> @wido could you write the test such that it can skip if host does not have
ipv6 enabled? But still work in env like Github Actions where ipv6 is available
That's something you can't detect. I think we would have to catch the
IllegalArgumentException in this case
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
