Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorian Waibel2015-05-22 05:03:33 -0400
committerFlorian Waibel2015-05-22 05:03:33 -0400
commitb19a18b79d458453b5170bbb3c167d536ef970d9 (patch)
treeec82402c335568fecde7df9f96af189662e7e17d
parent63feb26539190c8cb963c11121649c6842244e41 (diff)
downloadorg.eclipse.virgo.kernel-b19a18b79d458453b5170bbb3c167d536ef970d9.tar.gz
org.eclipse.virgo.kernel-b19a18b79d458453b5170bbb3c167d536ef970d9.tar.xz
org.eclipse.virgo.kernel-b19a18b79d458453b5170bbb3c167d536ef970d9.zip
Marks inactive test with @Ignore and adds TODO for review
-rw-r--r--org.eclipse.virgo.kernel.deployer/src/test/java/org/eclipse/virgo/kernel/install/artifact/internal/FileMovingArtifactStoreTests.java34
1 files changed, 18 insertions, 16 deletions
diff --git a/org.eclipse.virgo.kernel.deployer/src/test/java/org/eclipse/virgo/kernel/install/artifact/internal/FileMovingArtifactStoreTests.java b/org.eclipse.virgo.kernel.deployer/src/test/java/org/eclipse/virgo/kernel/install/artifact/internal/FileMovingArtifactStoreTests.java
index 55ee1372..2b5c57c0 100644
--- a/org.eclipse.virgo.kernel.deployer/src/test/java/org/eclipse/virgo/kernel/install/artifact/internal/FileMovingArtifactStoreTests.java
+++ b/org.eclipse.virgo.kernel.deployer/src/test/java/org/eclipse/virgo/kernel/install/artifact/internal/FileMovingArtifactStoreTests.java
@@ -19,14 +19,15 @@ import java.io.File;
import org.eclipse.virgo.util.io.PathReference;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
public class FileMovingArtifactStoreTests {
-
+
private static final String TEST_PATH = "build/fileMovingArtifactStoreTests/";
-
+
private static final String TEST_FILENAME = "some.jar";
-
+
private ArtifactStore artifactHistory;
@Before
@@ -60,7 +61,7 @@ public class FileMovingArtifactStoreTests {
checkPath(c);
assertTrue(original.equals(c));
}
-
+
@Test
public void testRepeatedStash() {
PathReference original = this.artifactHistory.getCurrentPath();
@@ -72,7 +73,7 @@ public class FileMovingArtifactStoreTests {
assertTrue(original.equals(next));
assertTrue(original.equals(last));
}
-
+
@Test
public void testFileDeletionOnUnstash() {
this.artifactHistory.save();
@@ -103,7 +104,7 @@ public class FileMovingArtifactStoreTests {
this.artifactHistory.save();
assertFalse(c.exists());
}
-
+
@Test
public void testDirectoryDeletionOnDoubleStash() {
PathReference c = this.artifactHistory.getCurrentPath();
@@ -114,36 +115,38 @@ public class FileMovingArtifactStoreTests {
this.artifactHistory.save();
assertFalse(c.exists());
}
-
- @Test(expected=IllegalStateException.class)
+
+ @Test(expected = IllegalStateException.class)
public void testBadUnstash() {
this.artifactHistory.restore();
}
- @Test(expected=IllegalStateException.class)
+ @Test(expected = IllegalStateException.class)
public void testDoubleUnstash() {
this.artifactHistory.save();
this.artifactHistory.save();
this.artifactHistory.restore();
this.artifactHistory.restore();
}
-
- @Test(expected=IllegalArgumentException.class)
+
+ @Test(expected = IllegalArgumentException.class)
public void testNullConstructorPath() {
new FileMovingArtifactStore(null);
}
-
- @Test(expected=IllegalArgumentException.class)
+
+ @Test(expected = IllegalArgumentException.class)
public void testEmptyConstructorPath() {
new FileMovingArtifactStore(new PathReference(""));
}
-
+
+ @Test
+ @Ignore("review the inactive / failing test")
+ // TODO - review the (from the beginning) inactive test
public void testDirectorylessConstructorPath() {
ArtifactStore ph = new FileMovingArtifactStore(new PathReference("a"));
assertEquals("a", ph.getCurrentPath().getName());
}
-
private void checkPath(PathReference c) {
File file = c.toFile();
assertTrue(file.toURI().toString().indexOf(TEST_PATH) != -1);
@@ -155,4 +158,3 @@ public class FileMovingArtifactStoreTests {
}
}
-

Back to the top