convert old main method for insdbcparser to junit

otter606 · otter606 · commit 794de2cf89df · 2021-11-16T08:54:37.000Z
diff --git a/biojava-core/src/test/java/org/biojava/nbio/core/sequence/location/InsdcParserTest.java b/biojava-core/src/test/java/org/biojava/nbio/core/sequence/location/InsdcParserTest.java
@@ -21,14 +21,11 @@
 package org.biojava.nbio.core.sequence.location;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertNotNull;
 
-import java.util.Arrays;
-import java.util.Collection;
+
 import org.biojava.nbio.core.sequence.DataSource;
 import org.biojava.nbio.core.sequence.location.InsdcParser.complexFeaturesAppendEnum;
 import org.biojava.nbio.core.sequence.location.template.Location;
-import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.CsvSource;
 import org.slf4j.Logger;
@@ -43,7 +40,6 @@ public class InsdcParserTest {
 
 	private Logger log = LoggerFactory.getLogger(getClass());
 
-
 	/**
 	 * Test for issue #254
 	 *
@@ -64,21 +60,15 @@ public void extractAccessionTest(String data, String expected) throws Exception
 		}
 	}
 
-	@Test
-	public void testParser() {
-		String[] testStrings = { "J00194.1:100..202", "A00001.5:34..45", "43..129", "bond(55,110)",
-				"bond(34,35),join(56..80),complement(45,73)",
-				"order(complement(30,40),70..80),bond(34,35),join(56,80),complement(45..56)",
-				"join(join(complement(30,40),complement(70..80)),bond(34,35),join(56,80),complement(45..56))",
-				"complement(join(complement(2000..4000),complement(70..80)),bond(34,35),join(56,80),complement(45..56))",
-
-		};
+	@ParameterizedTest
+	@CsvSource(delimiterString = "|", value = { "J00194.1:100..202|100..202(+)", "A00001.5:34..45|34..45(+)",
+			"43..129|43..129(+)", "bond(55,110)|55..110(+)", "bond(34,35),join(56..80),complement(45,73)|34..80(.)",
+			"order(complement(30,40),70..80),bond(34,35),join(56,80),complement(45..56)|30..80(.)",
+			"join(join(complement(30,40),complement(70..80)),bond(34,35),join(56,80),complement(45..56))|30..80(.)",
+			"complement(join(complement(2000..4000),complement(70..80)),bond(34,35),join(56,80),complement(45..56))|34..4000(.)" })
+	void testParser(String header, String parsedLocation) {
 		InsdcParser p = new InsdcParser();
 		p.setComplexFeaturesAppendMode(complexFeaturesAppendEnum.HIERARCHICAL);
-
-		for (String s : testStrings) {
-			Location l = p.parse(s);
-			assertNotNull(l);
-		}
+		assertEquals(parsedLocation, p.parse(header).toString());
 	}
 }
