Address review comments.

heuermh · heuermh · commit 71368bebfc9e · 2021-02-26T11:33:21.000-06:00
diff --git a/biojava-genome/src/test/java/org/biojava/nbio/genome/io/fastq/FastqBuilderTest.java b/biojava-genome/src/test/java/org/biojava/nbio/genome/io/fastq/FastqBuilderTest.java
@@ -20,10 +20,11 @@
  */
 package org.biojava.nbio.genome.io.fastq;
 
-
 import org.junit.Assert;
 import org.junit.Test;
 
+import org.junit.function.ThrowingRunnable;
+
 /**
  * Unit test for FastqBuilder.
  */
@@ -60,15 +61,12 @@ public void testConstructorFastq()
 	@Test
 	public void testConstructorNullFastq()
 	{
-		try
-		{
-			new FastqBuilder(null);
-			Assert.fail("builder(null) expected IllegalArgumentException");
-		}
-		catch (IllegalArgumentException e)
-		{
-			// expected
-		}
+		Assert.assertThrows(IllegalArgumentException.class, new ThrowingRunnable() {
+			@Override
+			public void run() {
+				new FastqBuilder(null);
+			}
+		});
 	}
 
 	@Test
diff --git a/biojava-genome/src/test/java/org/biojava/nbio/genome/io/fastq/FastqTest.java b/biojava-genome/src/test/java/org/biojava/nbio/genome/io/fastq/FastqTest.java
@@ -23,6 +23,7 @@
 import org.junit.Assert;
 import org.junit.Test;
 
+import org.junit.function.ThrowingRunnable;
 
 /**
  * Unit test for Fastq.
@@ -114,15 +115,12 @@ public void testBuilder()
 	@Test
 	public void testBuilderNullFastq()
 	{
-		try
-		{
-			Fastq.builder(null);
-			Assert.fail("builder(null) expected IllegalArgumentException");
-		}
-		catch (IllegalArgumentException e)
-		{
-			// expected
-		}
+		Assert.assertThrows(IllegalArgumentException.class, new ThrowingRunnable() {
+			@Override
+			public void run() {
+				Fastq.builder(null);
+			}
+		});
 	}
 
 	@Test
