From 4be5c2e56cd99d7d1bf6a129cf26c3cbc1e2e926 Mon Sep 17 00:00:00 2001 From: gabrascher Date: Sat, 16 Apr 2016 15:35:17 -0300 Subject: [PATCH] Test fails in Widows as the file separator "/" is different from "\" File separator in windows is different from linux (the expected in the test); thus, the test *com.cloud.utils.SwiftUtilTest.testSplitSwiftPath()* will fail in windows. The problem is that the input of the test is *"container/object"* but the tested method uses the *File.separator* (that depends from the OS), in the windows the tested method (*com.cloud.utils.SwiftUtil.splitSwiftPath(String)*) looks for a "\" in windows systems, resulting in an empty string and consequently a failure in the test. Some solutions: - the simple way is to create a string `String input = "container" + File.separator + "object";`, thus independent of the OS, the test will succeed. - a tricky solution is to mock the final static variable *File.separator* and return "/". I picked the easy way. --- utils/src/test/java/com/cloud/utils/SwiftUtilTest.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/utils/src/test/java/com/cloud/utils/SwiftUtilTest.java b/utils/src/test/java/com/cloud/utils/SwiftUtilTest.java index 55f7a3479ec..20c1623d822 100644 --- a/utils/src/test/java/com/cloud/utils/SwiftUtilTest.java +++ b/utils/src/test/java/com/cloud/utils/SwiftUtilTest.java @@ -19,9 +19,11 @@ package com.cloud.utils; + import org.junit.Test; import org.mockito.Mockito; +import java.io.File; import java.net.URL; import java.security.InvalidKeyException; import java.security.NoSuchAlgorithmException; @@ -32,7 +34,6 @@ import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; import static org.mockito.Mockito.when; - public class SwiftUtilTest { @Test @@ -72,7 +73,7 @@ public class SwiftUtilTest { @Test public void testSplitSwiftPath(){ - String input = "container/object"; + String input = "container" + File.separator + "object"; String[] output = SwiftUtil.splitSwiftPath(input); String[] expected = {"container", "object"};