Test pom exclusions don't work anymore, now using @ignore

josemduarte · josemduarte · commit 7ad223ab58c7 · 2018-03-10T22:11:07.000-08:00
diff --git a/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/ecod/EcodInstallationTest.java b/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/ecod/EcodInstallationTest.java
@@ -268,7 +268,7 @@ public void testVersion() throws IOException {
 	 * Hierarchical field warnings are expected for versions prior to develop68.
 	 * @throws IOException
 	 */
-	//@Ignore // Very slow parsing test
+	@Ignore // Very slow parsing test
 	@Test
 	public void testAllVersions() throws IOException {
 		// List all versions
diff --git a/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/io/TestAtomCachePerformance.java b/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/io/TestAtomCachePerformance.java
@@ -25,6 +25,7 @@
 import org.biojava.nbio.structure.align.util.AtomCache;
 import org.biojava.nbio.structure.io.LocalPDBDirectory.FetchBehavior;
 import org.junit.BeforeClass;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import java.io.IOException;
@@ -35,7 +36,7 @@
 /**
  * A test to check the performance of AtomCache downloading
  *
- * By default it is excluded from executing by main biojava pom.
+ * By default it is ignored.
  * To execute use:
  * <pre>
  * mvn -Dtest=TestAtomCachePerformance test
@@ -68,6 +69,7 @@ public static void setUpBeforeClass() {
 		cache.setFetchBehavior(FetchBehavior.FORCE_DOWNLOAD);
 	}
 
+	@Ignore
 	@Test
 	public void testDownload() throws IOException, StructureException {
 		System.out.println("Starting performance test for "+PDB_IDS.length+" PDB ids");
diff --git a/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/io/TestLongPdbVsMmCifParsing.java b/biojava-integrationtest/src/test/java/org/biojava/nbio/structure/test/io/TestLongPdbVsMmCifParsing.java
@@ -29,6 +29,7 @@
 import org.junit.After;
 import org.junit.BeforeClass;
 import org.junit.ComparisonFailure;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -43,7 +44,7 @@
  * A test to make sure both PDB and mmCIF parsers can parse
  * properly large samples of the PDB.
  *
- * Will take very long to run, thus they are excluded by default in the pom.
+ * Will take very long to run, thus they are ignored by default.
  * To run them use, for the 1000 entries one:
  * <pre>
  * mvn -Dtest=TestLongPdbVsMmCifParsing#testLongPdbVsMmCif test
@@ -106,6 +107,7 @@ public static void setUpBeforeClass() {
 		cache.setObsoleteBehavior(ObsoleteBehavior.THROW_EXCEPTION);
 	}
 
+	@Ignore
 	@Test
 	public void testLongPdbVsMmCif() throws IOException, StructureException {
 
@@ -115,6 +117,7 @@ public void testLongPdbVsMmCif() throws IOException, StructureException {
 
 	}
 
+	@Ignore
 	@Test
 	public void testVeryLongPdbVsMmCif() throws IOException, StructureException {
 
@@ -124,6 +127,7 @@ public void testVeryLongPdbVsMmCif() throws IOException, StructureException {
 
 	}
 
+	@Ignore
 	@Test
 	public void testSingle() throws IOException, StructureException {
 		testAll(Arrays.asList("4kro"));
diff --git a/pom.xml b/pom.xml
@@ -362,14 +362,6 @@
 					<argLine>-Xmx1500M</argLine>
 					<reuseForks>false</reuseForks>
 					<forkCount>1</forkCount>
-
-					<excludes>
-						<!-- we exclude by default the long parsing tests, they will only be 
-							executed if explicitly called -->
-						<exclude>**/TestLongPdbVsMmCifParsing.java</exclude>
-						<exclude>**/TestAtomCachePerformance.java</exclude>
-						<exclude>**/EcodInstallationTest.java</exclude>
-					</excludes>
 				</configuration>
 			</plugin>
 
