mirror of https://github.com/apache/jclouds.git
Modified jclouds-filesystem tests to skip tests that are known not to work on Windows
This commit is contained in:
parent
532c4eb71f
commit
dcfbb7dde1
|
@ -19,13 +19,7 @@
|
||||||
|
|
||||||
package org.jclouds.filesystem;
|
package org.jclouds.filesystem;
|
||||||
|
|
||||||
import static org.testng.Assert.assertEquals;
|
import static org.testng.Assert.*;
|
||||||
import static org.testng.Assert.assertFalse;
|
|
||||||
import static org.testng.Assert.assertNotNull;
|
|
||||||
import static org.testng.Assert.assertNotSame;
|
|
||||||
import static org.testng.Assert.assertNull;
|
|
||||||
import static org.testng.Assert.assertTrue;
|
|
||||||
import static org.testng.Assert.fail;
|
|
||||||
|
|
||||||
import java.io.File;
|
import java.io.File;
|
||||||
import java.io.FileInputStream;
|
import java.io.FileInputStream;
|
||||||
|
@ -53,6 +47,7 @@ import org.jclouds.filesystem.reference.FilesystemConstants;
|
||||||
import org.jclouds.filesystem.utils.TestUtils;
|
import org.jclouds.filesystem.utils.TestUtils;
|
||||||
import org.testng.annotations.AfterMethod;
|
import org.testng.annotations.AfterMethod;
|
||||||
import org.testng.annotations.BeforeMethod;
|
import org.testng.annotations.BeforeMethod;
|
||||||
|
import org.testng.annotations.DataProvider;
|
||||||
import org.testng.annotations.Test;
|
import org.testng.annotations.Test;
|
||||||
|
|
||||||
import com.google.inject.CreationException;
|
import com.google.inject.CreationException;
|
||||||
|
@ -117,6 +112,11 @@ public class FilesystemAsyncBlobStoreTest {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@DataProvider
|
||||||
|
public Object[][] ignoreOnWindows() {
|
||||||
|
return (TestUtils.isWindowsOs() ? TestUtils.NO_INVOCATIONS
|
||||||
|
: TestUtils.SINGLE_NO_ARG_INVOCATION);
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Checks if context parameters are managed in the correct way
|
* Checks if context parameters are managed in the correct way
|
||||||
|
@ -429,6 +429,7 @@ public class FilesystemAsyncBlobStoreTest {
|
||||||
/**
|
/**
|
||||||
* Test of removeBlob method, with only one blob with a complex path as key
|
* Test of removeBlob method, with only one blob with a complex path as key
|
||||||
*/
|
*/
|
||||||
|
@Test(dataProvider = "ignoreOnWindows")
|
||||||
public void testRemoveBlob_ComplexBlobKey() throws IOException {
|
public void testRemoveBlob_ComplexBlobKey() throws IOException {
|
||||||
final String BLOB_KEY = TestUtils.createRandomBlobKey("aa/bb/cc/dd/", null);
|
final String BLOB_KEY = TestUtils.createRandomBlobKey("aa/bb/cc/dd/", null);
|
||||||
boolean result;
|
boolean result;
|
||||||
|
@ -463,6 +464,7 @@ public class FilesystemAsyncBlobStoreTest {
|
||||||
* when first blob is removed, not all of its key's path is removed, because
|
* when first blob is removed, not all of its key's path is removed, because
|
||||||
* it is shared with the second blob's key
|
* it is shared with the second blob's key
|
||||||
*/
|
*/
|
||||||
|
@Test(dataProvider = "ignoreOnWindows")
|
||||||
public void testRemoveBlob_TwoComplexBlobKeys() throws IOException {
|
public void testRemoveBlob_TwoComplexBlobKeys() throws IOException {
|
||||||
final String BLOB_KEY1 = TestUtils.createRandomBlobKey("aa/bb/cc/dd/", null);
|
final String BLOB_KEY1 = TestUtils.createRandomBlobKey("aa/bb/cc/dd/", null);
|
||||||
final String BLOB_KEY2 = TestUtils.createRandomBlobKey("aa/bb/ee/ff/", null);
|
final String BLOB_KEY2 = TestUtils.createRandomBlobKey("aa/bb/ee/ff/", null);
|
||||||
|
@ -761,7 +763,7 @@ public class FilesystemAsyncBlobStoreTest {
|
||||||
TestUtils.directoryExists(TARGET_CONTAINER_NAME2, false);
|
TestUtils.directoryExists(TARGET_CONTAINER_NAME2, false);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test(dataProvider = "ignoreOnWindows")
|
||||||
public void testInvalidContainerName() {
|
public void testInvalidContainerName() {
|
||||||
try {
|
try {
|
||||||
blobStore.createContainerInLocation(null, "file/system");
|
blobStore.createContainerInLocation(null, "file/system");
|
||||||
|
|
|
@ -85,6 +85,11 @@ public class FilesystemStorageStrategyImplTest {
|
||||||
TestUtils.cleanDirectoryContent(TestUtils.TARGET_BASE_DIR);
|
TestUtils.cleanDirectoryContent(TestUtils.TARGET_BASE_DIR);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@DataProvider
|
||||||
|
public Object[][] ignoreOnWindows() {
|
||||||
|
return (TestUtils.isWindowsOs() ? TestUtils.NO_INVOCATIONS
|
||||||
|
: TestUtils.SINGLE_NO_ARG_INVOCATION);
|
||||||
|
}
|
||||||
|
|
||||||
public void testCreateDirectory() {
|
public void testCreateDirectory() {
|
||||||
storageStrategy.createDirectory(CONTAINER_NAME, null);
|
storageStrategy.createDirectory(CONTAINER_NAME, null);
|
||||||
|
@ -108,6 +113,7 @@ public class FilesystemStorageStrategyImplTest {
|
||||||
storageStrategy.createDirectory(CONTAINER_NAME, null);
|
storageStrategy.createDirectory(CONTAINER_NAME, null);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test(dataProvider = "ignoreOnWindows")
|
||||||
public void testCreateDirectory_WrongDirectoryName() {
|
public void testCreateDirectory_WrongDirectoryName() {
|
||||||
try {
|
try {
|
||||||
storageStrategy.createDirectory(CONTAINER_NAME, "$%&!'`\\/");
|
storageStrategy.createDirectory(CONTAINER_NAME, "$%&!'`\\/");
|
||||||
|
|
|
@ -48,7 +48,12 @@ public class TestUtils {
|
||||||
private static int imageResourceIndex = 0;
|
private static int imageResourceIndex = 0;
|
||||||
|
|
||||||
public static final String TARGET_BASE_DIR = "." + File.separator + "target" + File.separator + "basedir" + File.separator;
|
public static final String TARGET_BASE_DIR = "." + File.separator + "target" + File.separator + "basedir" + File.separator;
|
||||||
|
public static final Object[][] NO_INVOCATIONS = new Object[0][0];
|
||||||
|
public static final Object[][] SINGLE_NO_ARG_INVOCATION = new Object[][] { new Object[0] };
|
||||||
|
|
||||||
|
public static boolean isWindowsOs() {
|
||||||
|
return System.getProperty("os.name", "").toLowerCase().contains("windows");
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Generate a random blob key simple name (with no path in the key)
|
* Generate a random blob key simple name (with no path in the key)
|
||||||
|
|
Loading…
Reference in New Issue