Update tests from JUnit 4 to 5. Closes #79.

Project: http://git-wip-us.apache.org/repos/asf/commons-text/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-text/commit/49db4cb3
Tree: http://git-wip-us.apache.org/repos/asf/commons-text/tree/49db4cb3
Diff: http://git-wip-us.apache.org/repos/asf/commons-text/diff/49db4cb3

Branch: refs/heads/master
Commit: 49db4cb38baa24348023258924e63e9dac786f00
Parents: 6f06de9
Author: Pascal Schumacher <pascalschumac...@apache.org>
Authored: Sun Apr 8 09:42:14 2018 -0600
Committer: Gary Gregory <garydgreg...@gmail.com>
Committed: Sun Apr 8 09:42:14 2018 -0600

----------------------------------------------------------------------
 pom.xml                                         |   32 +-
 .../commons/text/AlphabetConverterTest.java     |   87 +-
 .../org/apache/commons/text/CaseUtilsTest.java  |    2 +-
 .../commons/text/CharacterPredicatesTest.java   |    2 +-
 .../commons/text/CompositeFormatTest.java       |    8 +-
 .../commons/text/ExtendedMessageFormatTest.java |  126 +-
 .../commons/text/FormattableUtilsTest.java      |   14 +-
 .../commons/text/RandomStringGeneratorTest.java |   68 +-
 .../text/StrBuilderAppendInsertTest.java        |    2 +-
 .../org/apache/commons/text/StrBuilderTest.java |  111 +-
 .../org/apache/commons/text/StrLookupTest.java  |   14 +-
 .../org/apache/commons/text/StrMatcherTest.java |    2 +-
 .../apache/commons/text/StrSubstitutorTest.java | 1664 +++++++++---------
 .../apache/commons/text/StrTokenizerTest.java   |   79 +-
 .../commons/text/StringEscapeUtilsTest.java     |  187 +-
 .../commons/text/StringSubstitutorTest.java     |   39 +-
 ...titutorWithInterpolatorStringLookupTest.java |  146 +-
 .../commons/text/StringTokenizerTest.java       |   79 +-
 .../text/TextStringBuilderAppendInsertTest.java |    2 +-
 .../commons/text/TextStringBuilderTest.java     |  111 +-
 .../org/apache/commons/text/WordUtilsTest.java  |   15 +-
 .../text/diff/ReplacementsFinderTest.java       |   60 +-
 .../text/diff/StringsComparatorTest.java        |   10 +-
 .../text/lookup/DateStringLookupTest.java       |  100 +-
 .../EnvironmentVariableStringLookupTest.java    |   74 +-
 .../lookup/InterpolatorStringLookupTest.java    |  196 +--
 .../lookup/JavaPlatformStringLookupTest.java    |   63 +-
 .../text/lookup/LocalHostStringLookupTest.java  |   92 +-
 .../text/lookup/MapStringLookupTest.java        |   74 +-
 .../text/lookup/NullStringLookupTest.java       |   60 +-
 .../lookup/ResourceBundleStringLookupTest.java  |   70 +-
 .../lookup/SystemPropertyStringLookupTest.java  |   62 +-
 .../commons/text/matcher/StringMatcherTest.java |    2 +-
 .../matcher/StringSubstitutorGetSetTest.java    |   18 +-
 .../text/similarity/CosineDistanceTest.java     |    6 +-
 .../text/similarity/CosineSimilarityTest.java   |   21 +-
 .../commons/text/similarity/FuzzyScoreTest.java |   27 +-
 .../text/similarity/HammingDistanceTest.java    |   19 +-
 .../text/similarity/JaccardDistanceTest.java    |   56 +-
 .../text/similarity/JaccardSimilarityTest.java  |   56 +-
 .../similarity/JaroWinklerDistanceTest.java     |   29 +-
 .../LevenshteinDetailedDistanceTest.java        |   37 +-
 .../similarity/LevenshteinDistanceTest.java     |   27 +-
 .../text/similarity/LevenshteinResultsTest.java |    2 +-
 .../LongestCommonSubsequenceDistanceTest.java   |   19 +-
 .../LongestCommonSubsequenceTest.java           |   49 +-
 .../ParameterizedEditDistanceFromTest.java      |   84 +-
 .../ParameterizedLevenshteinDistanceTest.java   |  121 +-
 .../ParameterizedSimilarityScoreFromTest.java   |   54 +-
 .../similarity/SimilarityScoreFromTest.java     |    7 +-
 .../text/similarity/StringMetricFromTest.java   |   18 +-
 .../text/translate/AggregateTranslatorTest.java |    2 +-
 .../text/translate/CodePointTranslatorTest.java |    2 +-
 .../text/translate/CsvTranslatorsTest.java      |    2 +-
 .../text/translate/EntityArraysTest.java        |    2 +-
 .../text/translate/JavaUnicodeEscaperTest.java  |    2 +-
 .../text/translate/LookupTranslatorTest.java    |    7 +-
 .../translate/NumericEntityEscaperTest.java     |    2 +-
 .../translate/NumericEntityUnescaperTest.java   |    2 +-
 .../text/translate/OctalUnescaperTest.java      |    2 +-
 .../translate/SinglePassTranslatorTest.java     |   15 +-
 .../text/translate/UnicodeEscaperTest.java      |    2 +-
 .../text/translate/UnicodeUnescaperTest.java    |    2 +-
 .../UnicodeUnpairedSurrogateRemoverTest.java    |    2 +-
 64 files changed, 2185 insertions(+), 2162 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 8f872f9..241330e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -48,6 +48,9 @@
     
<commons.distSvnStagingUrl>scm:svn:https://dist.apache.org/repos/dist/dev/commons/text/</commons.distSvnStagingUrl>
     <commons.release.isDistModule>true</commons.release.isDistModule>
 
+    <junit.version>5.1.0</junit.version>
+    <junit.platform.version>1.1.0</junit.platform.version>
+
     <checkstyle.plugin.version>3.0.0</checkstyle.plugin.version>
     <checkstyle.version>8.9</checkstyle.version>
   </properties>
@@ -60,15 +63,21 @@
     </dependency>
     <!-- testing -->
     <dependency>
-      <groupId>junit</groupId>
-      <artifactId>junit</artifactId>
-      <version>4.12</version>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-engine</artifactId>
+      <version>${junit.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.junit.jupiter</groupId>
+      <artifactId>junit-jupiter-params</artifactId>
+      <version>${junit.version}</version>
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>org.hamcrest</groupId>
-      <artifactId>hamcrest-all</artifactId>
-      <version>1.3</version>
+      <groupId>org.junit.platform</groupId>
+      <artifactId>junit-platform-launcher</artifactId>
+      <version>${junit.platform.version}</version>
       <scope>test</scope>
     </dependency>
     <dependency>
@@ -99,6 +108,17 @@
     </pluginManagement>
     <plugins>
       <plugin>
+        <artifactId>maven-surefire-plugin</artifactId>
+        <version>2.19</version>
+          <dependencies>
+             <dependency>
+               <groupId>org.junit.platform</groupId>
+               <artifactId>junit-platform-surefire-provider</artifactId>
+               <version>${junit.platform.version}</version>
+           </dependency>
+        </dependencies>
+      </plugin>
+      <plugin>
         <artifactId>maven-checkstyle-plugin</artifactId>
         <version>${checkstyle.plugin.version}</version>
         <configuration>

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/AlphabetConverterTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/AlphabetConverterTest.java 
b/src/test/java/org/apache/commons/text/AlphabetConverterTest.java
index 7eb8887..1ce32a7 100644
--- a/src/test/java/org/apache/commons/text/AlphabetConverterTest.java
+++ b/src/test/java/org/apache/commons/text/AlphabetConverterTest.java
@@ -22,11 +22,11 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.rules.ExpectedException;
+import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static 
org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 /**
  * Unit tests for {@link AlphabetConverter}.
@@ -55,14 +55,11 @@ public class AlphabetConverterTest {
             108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 
121, 122};
     private static Integer[] doNotEncodeCodepoints = {32, 97, 98, 99}; // 
space, a, b, c
 
-    @Rule
-    public ExpectedException thrown = ExpectedException.none();
-
     @Test
-    public void encodeFailureTest() throws UnsupportedEncodingException {
-        thrown.expect(UnsupportedEncodingException.class);
-        thrown.expectMessage("Couldn't find encoding for '3'");
-        test(binary, numbers, empty, "3");
+    public void encodeFailureTest() {
+        assertThatThrownBy(() -> {
+            test(binary, numbers, empty, "3");
+        
}).isInstanceOf(UnsupportedEncodingException.class).hasMessage("Couldn't find 
encoding for '3' in 3");
     }
 
     @Test
@@ -117,23 +114,21 @@ public class AlphabetConverterTest {
     @Test
     public void unexpectedEndwhileDecodingTest() throws 
UnsupportedEncodingException {
         final String toDecode = "00d01d0";
-
-        thrown.expect(UnsupportedEncodingException.class);
-        thrown.expectMessage("Unexpected end of string while decoding " + 
toDecode);
-
-        final AlphabetConverter ac = createJavadocExample();
-        ac.decode(toDecode);
+        assertThatThrownBy(() -> {
+            final AlphabetConverter ac = createJavadocExample();
+            ac.decode(toDecode);
+        }).isInstanceOf(UnsupportedEncodingException.class).hasMessage(
+                "Unexpected end of string while decoding " + toDecode);
     }
 
     @Test
     public void unexpectedStringWhileDecodingTest() throws 
UnsupportedEncodingException {
         final String toDecode = "00XX";
-
-        thrown.expect(UnsupportedEncodingException.class);
-        thrown.expectMessage("Unexpected string without decoding (XX) in " + 
toDecode);
-
-        final AlphabetConverter ac = createJavadocExample();
-        ac.decode(toDecode);
+        assertThatThrownBy(() -> {
+            final AlphabetConverter ac = createJavadocExample();
+            ac.decode(toDecode);
+        }).isInstanceOf(UnsupportedEncodingException.class).hasMessage(
+                "Unexpected string without decoding (XX) in " + toDecode);
     }
 
     /*
@@ -156,39 +151,37 @@ public class AlphabetConverterTest {
 
     @Test
     public void noEncodingLettersTest() {
-        thrown.expect(IllegalArgumentException.class);
-        thrown.expectMessage(
+        assertThatThrownBy(() -> {
+            AlphabetConverter.createConverterFromChars(englishAndNumbers, 
numbers, numbers);
+        }).isInstanceOf(IllegalArgumentException.class).hasMessage(
                 "Must have at least two encoding characters (excluding those 
in the 'do not encode' list), but has 0");
-
-        AlphabetConverter.createConverterFromChars(englishAndNumbers, numbers, 
numbers);
     }
 
     @Test
     public void onlyOneEncodingLettersTest() {
-        thrown.expect(IllegalArgumentException.class);
-        thrown.expectMessage(
-                "Must have at least two encoding characters (excluding those 
in the 'do not encode' list), but has 1");
-
-        final Character[] numbersPlusUnderscore = Arrays.copyOf(numbers, 
numbers.length + 1);
-        numbersPlusUnderscore[numbersPlusUnderscore.length - 1] = '_';
+        assertThatThrownBy(() -> {
+            final Character[] numbersPlusUnderscore = Arrays.copyOf(numbers, 
numbers.length + 1);
+            numbersPlusUnderscore[numbersPlusUnderscore.length - 1] = '_';
 
-        AlphabetConverter.createConverterFromChars(englishAndNumbers, 
numbersPlusUnderscore, numbers);
+            AlphabetConverter.createConverterFromChars(englishAndNumbers, 
numbersPlusUnderscore, numbers);
+        }).isInstanceOf(IllegalArgumentException.class).hasMessage(
+                "Must have at least two encoding characters (excluding those 
in the 'do not encode' list), but has 1");
     }
 
     @Test
     public void missingDoNotEncodeLettersFromEncodingTest() {
-        thrown.expect(IllegalArgumentException.class);
-        thrown.expectMessage("Can not use 'do not encode' list because 
encoding alphabet does not contain");
-
-        AlphabetConverter.createConverterFromChars(englishAndNumbers, 
lowerCaseEnglish, numbers);
+        assertThatThrownBy(() -> {
+            AlphabetConverter.createConverterFromChars(englishAndNumbers, 
lowerCaseEnglish, numbers);
+        }).isInstanceOf(IllegalArgumentException.class).hasMessage(
+                "Can not use 'do not encode' list because encoding alphabet 
does not contain '0'");
     }
 
     @Test
     public void missingDoNotEncodeLettersFromOriginalTest() {
-        thrown.expect(IllegalArgumentException.class);
-        thrown.expectMessage("Can not use 'do not encode' list because 
original alphabet does not contain");
-
-        AlphabetConverter.createConverterFromChars(lowerCaseEnglish, 
englishAndNumbers, numbers);
+        assertThatThrownBy(() -> {
+            AlphabetConverter.createConverterFromChars(lowerCaseEnglish, 
englishAndNumbers, numbers);
+        }).isInstanceOf(IllegalArgumentException.class).hasMessage(
+                "Can not use 'do not encode' list because original alphabet 
does not contain '0'");
     }
 
     private void test(final Character[] originalChars, final Character[] 
encodingChars,
@@ -229,13 +222,15 @@ public class AlphabetConverterTest {
         }
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testCreateConverterFromCharsWithNullAndNull() {
-        final Character[] characterArray = new Character[2];
-        characterArray[0] = '$';
-        characterArray[1] = characterArray[0];
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            final Character[] characterArray = new Character[2];
+            characterArray[0] = '$';
+            characterArray[1] = characterArray[0];
 
-        AlphabetConverter.createConverterFromChars(characterArray, null, null);
+            AlphabetConverter.createConverterFromChars(characterArray, null, 
null);
+        });
     }
 
     @Test

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/CaseUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/CaseUtilsTest.java 
b/src/test/java/org/apache/commons/text/CaseUtilsTest.java
index 33ee577..c3f4353 100644
--- a/src/test/java/org/apache/commons/text/CaseUtilsTest.java
+++ b/src/test/java/org/apache/commons/text/CaseUtilsTest.java
@@ -16,7 +16,7 @@
  */
 package org.apache.commons.text;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Modifier;

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/CharacterPredicatesTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/CharacterPredicatesTest.java 
b/src/test/java/org/apache/commons/text/CharacterPredicatesTest.java
index e16ac13..53879e4 100644
--- a/src/test/java/org/apache/commons/text/CharacterPredicatesTest.java
+++ b/src/test/java/org/apache/commons/text/CharacterPredicatesTest.java
@@ -16,7 +16,7 @@
  */
 package org.apache.commons.text;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
 

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/CompositeFormatTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/CompositeFormatTest.java 
b/src/test/java/org/apache/commons/text/CompositeFormatTest.java
index 0130f9b..24b3586 100644
--- a/src/test/java/org/apache/commons/text/CompositeFormatTest.java
+++ b/src/test/java/org/apache/commons/text/CompositeFormatTest.java
@@ -17,7 +17,7 @@
 
 package org.apache.commons.text;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.text.FieldPosition;
 import java.text.Format;
@@ -25,7 +25,7 @@ import java.text.ParsePosition;
 import java.text.SimpleDateFormat;
 import java.util.Locale;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests for {@link CompositeFormat}.
@@ -70,8 +70,8 @@ public class CompositeFormatTest {
 
         composite.parseObject("", null);
         composite.format(new Object(), new StringBuffer(), null);
-        assertEquals("Parser get method incorrectly implemented", parser, 
composite.getParser());
-        assertEquals("Formatter get method incorrectly implemented", 
formatter, composite.getFormatter());
+        assertEquals(parser, composite.getParser(), "Parser get method 
incorrectly implemented");
+        assertEquals(formatter, composite.getFormatter(), "Formatter get 
method incorrectly implemented");
     }
 
     @Test

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/ExtendedMessageFormatTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/text/ExtendedMessageFormatTest.java 
b/src/test/java/org/apache/commons/text/ExtendedMessageFormatTest.java
index 8770bc4..0563826 100644
--- a/src/test/java/org/apache/commons/text/ExtendedMessageFormatTest.java
+++ b/src/test/java/org/apache/commons/text/ExtendedMessageFormatTest.java
@@ -16,9 +16,11 @@
  */
 package org.apache.commons.text;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
+import static 
org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.text.DateFormat;
 import java.text.FieldPosition;
@@ -34,8 +36,8 @@ import java.util.HashSet;
 import java.util.Locale;
 import java.util.Map;
 
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 
 /**
  * Test case for {@link ExtendedMessageFormat}.
@@ -44,7 +46,7 @@ public class ExtendedMessageFormatTest {
 
     private final Map<String, FormatFactory> registry = new HashMap<>();
 
-    @Before
+    @BeforeEach
     public void setUp() {
         registry.put("lower", new LowerCaseFormatFactory());
         registry.put("upper", new UpperCaseFormatFactory());
@@ -57,7 +59,7 @@ public class ExtendedMessageFormatTest {
     public void testExtendedFormats() {
         final String pattern = "Lower: {0,lower} Upper: {1,upper}";
         final ExtendedMessageFormat emf = new ExtendedMessageFormat(pattern, 
registry);
-        assertEquals("TOPATTERN", pattern, emf.toPattern());
+        assertEquals(pattern, emf.toPattern(), "TOPATTERN");
         assertEquals("Lower: foo Upper: BAR", emf.format(new Object[] {"foo", 
"bar"}));
         assertEquals("Lower: foo Upper: BAR", emf.format(new Object[] {"Foo", 
"Bar"}));
         assertEquals("Lower: foo Upper: BAR", emf.format(new Object[] {"FOO", 
"BAR"}));
@@ -152,8 +154,8 @@ public class ExtendedMessageFormatTest {
             expected.append(df.format(args[1]));
             expected.append(" Salary: ");
             expected.append(nf.format(args[2]));
-            assertEquals("pattern comparison for locale " + locale, 
expectedPattern, emf.toPattern());
-            assertEquals(String.valueOf(locale), expected.toString(), 
emf.format(args));
+            assertEquals(expectedPattern, emf.toPattern(), "pattern comparison 
for locale " + locale);
+            assertEquals(expected.toString(), emf.format(args), 
String.valueOf(locale));
         }
     }
 
@@ -278,8 +280,8 @@ public class ExtendedMessageFormatTest {
             final MessageFormat dateDefault = createMessageFormat("{0,date}", 
locale);
             final String pattern = "{0,date,short}";
             final ExtendedMessageFormat dateShort = new 
ExtendedMessageFormat(pattern, locale, dateRegistry);
-            assertEquals("overridden date,short format", 
dateDefault.format(args), dateShort.format(args));
-            assertEquals("overridden date,short pattern", pattern, 
dateShort.toPattern());
+            assertEquals(dateDefault.format(args), dateShort.format(args), 
"overridden date,short format");
+            assertEquals(pattern, dateShort.toPattern(), "overridden 
date,short pattern");
         }
     }
 
@@ -290,11 +292,11 @@ public class ExtendedMessageFormatTest {
     public void testBuiltInNumberFormat() {
         final Object[] args = new Object[] {Double.valueOf("6543.21")};
         final Locale[] availableLocales = NumberFormat.getAvailableLocales();
-        checkBuiltInFormat("1: {0,number}",            args, availableLocales);
-        checkBuiltInFormat("2: {0,number,integer}",    args, availableLocales);
-        checkBuiltInFormat("3: {0,number,currency}",   args, availableLocales);
-        checkBuiltInFormat("4: {0,number,percent}",    args, availableLocales);
-        checkBuiltInFormat("5: {0,number,00000.000}",  args, availableLocales);
+        checkBuiltInFormat("1: {0,number}",           args, availableLocales);
+        checkBuiltInFormat("2: {0,number,integer}",   args, availableLocales);
+        checkBuiltInFormat("3: {0,number,currency}",  args, availableLocales);
+        checkBuiltInFormat("4: {0,number,percent}",   args, availableLocales);
+        checkBuiltInFormat("5: {0,number,00000.000}", args, availableLocales);
     }
 
     /**
@@ -313,35 +315,35 @@ public class ExtendedMessageFormatTest {
         ExtendedMessageFormat other = null;
 
         // Same object
-        assertTrue("same, equals()",   emf.equals(emf));
-        assertTrue("same, hashcode()", emf.hashCode() == emf.hashCode());
+        assertTrue(emf.equals(emf), "same, equals()");
+        assertEquals(emf.hashCode(), emf.hashCode(), "same, hashcode()");
 
-        assertFalse("null, equals", emf.equals(null));
+        assertFalse(emf.equals(null), "null, equals");
 
         // Equal Object
         other = new ExtendedMessageFormat(pattern, Locale.US, fmtRegistry);
-        assertTrue("equal, equals()",   emf.equals(other));
-        assertTrue("equal, hashcode()", emf.hashCode() == other.hashCode());
+        assertTrue(emf.equals(other), "equal, equals()");
+        assertTrue(emf.hashCode() == other.hashCode(), "equal, hashcode()");
 
         // Different Class
         other = new OtherExtendedMessageFormat(pattern, Locale.US, 
fmtRegistry);
-        assertFalse("class, equals()",  emf.equals(other));
-        assertTrue("class, hashcode()", emf.hashCode() == other.hashCode()); 
// same hashcode
+        assertFalse(emf.equals(other), "class, equals()");
+        assertTrue(emf.hashCode() == other.hashCode(), "class, hashcode()"); 
// same hashcode
 
         // Different pattern
         other = new ExtendedMessageFormat("X" + pattern, Locale.US, 
fmtRegistry);
-        assertFalse("pattern, equals()",   emf.equals(other));
-        assertFalse("pattern, hashcode()", emf.hashCode() == other.hashCode());
+        assertFalse(emf.equals(other), "pattern, equals()");
+        assertFalse(emf.hashCode() == other.hashCode(), "pattern, hashcode()");
 
         // Different registry
         other = new ExtendedMessageFormat(pattern, Locale.US, otherRegitry);
-        assertFalse("registry, equals()",   emf.equals(other));
-        assertFalse("registry, hashcode()", emf.hashCode() == 
other.hashCode());
+        assertFalse(emf.equals(other), "registry, equals()");
+        assertFalse(emf.hashCode() == other.hashCode(), "registry, 
hashcode()");
 
         // Different Locale
         other = new ExtendedMessageFormat(pattern, Locale.FRANCE, fmtRegistry);
-        assertFalse("locale, equals()",  emf.equals(other));
-        assertTrue("locale, hashcode()", emf.hashCode() == other.hashCode()); 
// same hashcode
+        assertFalse(emf.equals(other), "locale, equals()");
+        assertTrue(emf.hashCode() == other.hashCode(), "locale, hashcode()"); 
// same hashcode
     }
 
     /**
@@ -393,8 +395,8 @@ public class ExtendedMessageFormatTest {
         } else {
             emf = new ExtendedMessageFormat(pattern, locale);
         }
-        assertEquals("format "    + buffer.toString(), mf.format(args), 
emf.format(args));
-        assertEquals("toPattern " + buffer.toString(), mf.toPattern(), 
emf.toPattern());
+        assertEquals(mf.format(args), emf.format(args), "format "    + 
buffer.toString());
+        assertEquals(mf.toPattern(), emf.toPattern(), "toPattern " + 
buffer.toString());
     }
 
     /**
@@ -412,53 +414,71 @@ public class ExtendedMessageFormatTest {
         return result;
     }
 
-    @Test(expected = UnsupportedOperationException.class)
+    @Test
     public void testSetFormatIsUnsupported() {
-        final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
-        emf.setFormat(0, new LowerCaseFormat());
+        
assertThatExceptionOfType(UnsupportedOperationException.class).isThrownBy(() -> 
{
+            final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
+            emf.setFormat(0, new LowerCaseFormat());
+        });
     }
 
-    @Test(expected = UnsupportedOperationException.class)
+    @Test
     public void testSetFormatByArgumentIndexIsUnsupported() {
-        final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
-        emf.setFormatByArgumentIndex(0, new LowerCaseFormat());
+        
assertThatExceptionOfType(UnsupportedOperationException.class).isThrownBy(() -> 
{
+            final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
+            emf.setFormatByArgumentIndex(0, new LowerCaseFormat());
+        });
     }
 
-    @Test(expected = UnsupportedOperationException.class)
+    @Test
     public void testSetFormatsIsUnsupported() {
-        final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
-        emf.setFormats(new Format[]{new LowerCaseFormat(), new 
UpperCaseFormat()});
+        
assertThatExceptionOfType(UnsupportedOperationException.class).isThrownBy(() -> 
{
+            final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
+            emf.setFormats(new Format[] {new LowerCaseFormat(), new 
UpperCaseFormat()});
+        });
     }
 
-    @Test(expected = UnsupportedOperationException.class)
+    @Test
     public void testSetFormatsByArgumentIndex() {
-        final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
-        emf.setFormatsByArgumentIndex(new Format[]{new LowerCaseFormat(), new 
UpperCaseFormat()});
+        
assertThatExceptionOfType(UnsupportedOperationException.class).isThrownBy(() -> 
{
+            final ExtendedMessageFormat emf = new ExtendedMessageFormat("");
+            emf.setFormatsByArgumentIndex(new Format[] {new LowerCaseFormat(), 
new UpperCaseFormat()});
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void 
testFailsToCreateExtendedMessageFormatTakingTwoArgumentsThrowsIllegalArgumentExceptionOne()
 {
-        new ExtendedMessageFormat("agdXdkR;T1{9 ^,LzXf?", new HashMap<String, 
FormatFactory>());
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new ExtendedMessageFormat("agdXdkR;T1{9 ^,LzXf?", new 
HashMap<String, FormatFactory>());
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void 
testFailsToCreateExtendedMessageFormatTakingTwoArgumentsThrowsIllegalArgumentExceptionTwo()
 {
-        new ExtendedMessageFormat("a5XdkR;T1{9 ,LzXf?", new HashMap<String, 
FormatFactory>());
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new ExtendedMessageFormat("a5XdkR;T1{9 ,LzXf?", new 
HashMap<String, FormatFactory>());
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void 
testFailsToCreateExtendedMessageFormatTakingTwoArgumentsThrowsIllegalArgumentExceptionThree()
 {
-        new ExtendedMessageFormat("9jLh_D9{ ", new HashMap<String, 
FormatFactory>());
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new ExtendedMessageFormat("9jLh_D9{ ", new HashMap<String, 
FormatFactory>());
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void 
testFailsToCreateExtendedMessageFormatTakingTwoArgumentsThrowsIllegalArgumentExceptionFour()
 {
-        new ExtendedMessageFormat("RD,nXhM{}{", new HashMap<String, 
FormatFactory>());
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new ExtendedMessageFormat("RD,nXhM{}{", new HashMap<String, 
FormatFactory>());
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void 
testFailsToCreateExtendedMessageFormatTakingTwoArgumentsThrowsIllegalArgumentExceptionFive()
 {
-        new ExtendedMessageFormat("j/[_D9{0,\"&'+0o", new HashMap<String, 
FormatFactory>());
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new ExtendedMessageFormat("j/[_D9{0,\"&'+0o", new HashMap<String, 
FormatFactory>());
+        });
     }
 
     @Test

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/FormattableUtilsTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/FormattableUtilsTest.java 
b/src/test/java/org/apache/commons/text/FormattableUtilsTest.java
index 7741523..273cea1 100644
--- a/src/test/java/org/apache/commons/text/FormattableUtilsTest.java
+++ b/src/test/java/org/apache/commons/text/FormattableUtilsTest.java
@@ -18,11 +18,13 @@ package org.apache.commons.text;
 
 import static java.util.FormattableFlags.LEFT_JUSTIFY;
 import static org.assertj.core.api.Assertions.assertThat;
+import static 
org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
+import static org.assertj.core.api.Assertions.assertThatNullPointerException;
 
 import java.util.Formattable;
 import java.util.Formatter;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests {@link FormattableUtils}.
@@ -107,9 +109,11 @@ public class FormattableUtilsTest {
             .isEqualTo("+*   ");
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testIllegalEllipsis() {
-        FormattableUtils.append("foo", new Formatter(), 0, -1, 1, "xx");
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            FormattableUtils.append("foo", new Formatter(), 0, -1, 1, "xx");
+        });
     }
 
     @Test
@@ -146,9 +150,9 @@ public class FormattableUtilsTest {
             .isEqualTo("+*___");
     }
 
-    @Test(expected = NullPointerException.class)
+    @Test
     public void testAppendWithNullFormatterAndIntsThrowsNullPointerException() 
{
-        FormattableUtils.append("", null, 0, 0, 0, '}');
+        assertThatNullPointerException().isThrownBy(() -> 
FormattableUtils.append("", null, 0, 0, 0, '}'));
     }
 
     static class SimplestFormattable implements Formattable {

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/RandomStringGeneratorTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/text/RandomStringGeneratorTest.java 
b/src/test/java/org/apache/commons/text/RandomStringGeneratorTest.java
index f6c40b1..7805b89 100644
--- a/src/test/java/org/apache/commons/text/RandomStringGeneratorTest.java
+++ b/src/test/java/org/apache/commons/text/RandomStringGeneratorTest.java
@@ -16,14 +16,12 @@
  */
 package org.apache.commons.text;
 
-import static org.hamcrest.Matchers.allOf;
-import static org.hamcrest.Matchers.greaterThanOrEqualTo;
-import static org.hamcrest.Matchers.lessThanOrEqualTo;
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.junit.Assert.assertThat;
+import static 
org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
+import static org.assertj.core.api.Assertions.assertThatNullPointerException;
 import static org.assertj.core.api.Assertions.fail;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Tests for {@link RandomStringGenerator}
@@ -44,22 +42,28 @@ public class RandomStringGeneratorTest {
         }
     };
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testInvalidLength() {
-        final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
-        generator.generate(-1);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
+            generator.generate(-1);
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testGenerateMinMaxLengthInvalidLength() {
-        final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
-        generator.generate(-1, 0);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
+            generator.generate(-1, 0);
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testGenerateMinMaxLengthMinGreaterThanMax() {
-        final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
-        generator.generate(1, 0);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
+            generator.generate(1, 0);
+        });
     }
 
     private static int codePointLength(final String s) {
@@ -80,17 +84,21 @@ public class RandomStringGeneratorTest {
         final int maxLength = 3;
         final RandomStringGenerator generator = new 
RandomStringGenerator.Builder().build();
         final String str = generator.generate(minLength, maxLength);
-        assertThat(codePointLength(str), allOf(greaterThanOrEqualTo(0), 
lessThanOrEqualTo(3)));
+        assertThat(codePointLength(str)).isBetween(0, 3);
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testBadMinimumCodePoint() {
-        new RandomStringGenerator.Builder().withinRange(-1, 1);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new RandomStringGenerator.Builder().withinRange(-1, 1);
+        });
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testBadMaximumCodePoint() {
-        new RandomStringGenerator.Builder().withinRange(0, 
Character.MAX_CODE_POINT + 1);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new RandomStringGenerator.Builder().withinRange(0, 
Character.MAX_CODE_POINT + 1);
+        });
     }
 
     @Test
@@ -216,9 +224,11 @@ public class RandomStringGeneratorTest {
         } while (i < str.length());
     }
 
-    @Test(expected = IllegalArgumentException.class)
+    @Test
     public void testBadMinAndMax() {
-        new RandomStringGenerator.Builder().withinRange(2, 1);
+        assertThatIllegalArgumentException().isThrownBy(() -> {
+            new RandomStringGenerator.Builder().withinRange(2, 1);
+        });
     }
 
     @Test
@@ -279,13 +289,15 @@ public class RandomStringGeneratorTest {
         }
     }
 
-    @Test(expected = NullPointerException.class)
+    @Test
     public void testGenerateTakingIntThrowsNullPointerException() {
-        final RandomStringGenerator.Builder randomStringGeneratorBuilder = new 
RandomStringGenerator.Builder();
-        final CharacterPredicate[] characterPredicateArray = new 
CharacterPredicate[2];
-        randomStringGeneratorBuilder.filteredBy(characterPredicateArray);
-        final RandomStringGenerator randomStringGenerator = 
randomStringGeneratorBuilder.build();
-
-        randomStringGenerator.generate(18);
+        assertThatNullPointerException().isThrownBy(() -> {
+            final RandomStringGenerator.Builder randomStringGeneratorBuilder = 
new RandomStringGenerator.Builder();
+            final CharacterPredicate[] characterPredicateArray = new 
CharacterPredicate[2];
+            randomStringGeneratorBuilder.filteredBy(characterPredicateArray);
+            final RandomStringGenerator randomStringGenerator = 
randomStringGeneratorBuilder.build();
+
+            randomStringGenerator.generate(18);
+        });
     }
 }

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/StrBuilderAppendInsertTest.java
----------------------------------------------------------------------
diff --git 
a/src/test/java/org/apache/commons/text/StrBuilderAppendInsertTest.java 
b/src/test/java/org/apache/commons/text/StrBuilderAppendInsertTest.java
index e1cfad4..416e0ed 100644
--- a/src/test/java/org/apache/commons/text/StrBuilderAppendInsertTest.java
+++ b/src/test/java/org/apache/commons/text/StrBuilderAppendInsertTest.java
@@ -26,7 +26,7 @@ import java.util.Collection;
 import java.util.Collections;
 import java.util.Iterator;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests for {@link StrBuilder}.

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/StrBuilderTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/StrBuilderTest.java 
b/src/test/java/org/apache/commons/text/StrBuilderTest.java
index 528588a..a7fafe2 100644
--- a/src/test/java/org/apache/commons/text/StrBuilderTest.java
+++ b/src/test/java/org/apache/commons/text/StrBuilderTest.java
@@ -17,14 +17,15 @@
 
 package org.apache.commons.text;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNotSame;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertSame;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNotSame;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertSame;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.fail;
 
 import java.io.IOException;
 import java.io.Reader;
@@ -36,7 +37,7 @@ import java.nio.CharBuffer;
 import java.nio.charset.Charset;
 import java.util.Arrays;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests for {@link StrBuilder}.
@@ -482,13 +483,13 @@ public class StrBuilderTest {
         assertEquals(0, sb.toCharArray().length);
 
         char[] a = sb.toCharArray();
-        assertNotNull("toCharArray() result is null", a);
-        assertEquals("toCharArray() result is too large", 0, a.length);
+        assertNotNull(a, "toCharArray() result is null");
+        assertEquals(0, a.length, "toCharArray() result is too large");
 
         sb.append("junit");
         a = sb.toCharArray();
-        assertEquals("toCharArray() result incorrect length", 5, a.length);
-        assertTrue("toCharArray() result does not match", 
Arrays.equals("junit".toCharArray(), a));
+        assertEquals(5, a.length, "toCharArray() result incorrect length");
+        assertTrue(Arrays.equals("junit".toCharArray(), a), "toCharArray() 
result does not match");
     }
 
     @Test
@@ -498,19 +499,19 @@ public class StrBuilderTest {
 
         sb.append("junit");
         char[] a = sb.toCharArray(0, 20); // too large test
-        assertEquals("toCharArray(int,int) result incorrect length", 5, 
a.length);
-        assertTrue("toCharArray(int,int) result does not match", 
Arrays.equals("junit".toCharArray(), a));
+        assertEquals(5, a.length, "toCharArray(int,int) result incorrect 
length");
+        assertTrue(Arrays.equals("junit".toCharArray(), a), 
"toCharArray(int,int) result does not match");
 
         a = sb.toCharArray(0, 4);
-        assertEquals("toCharArray(int,int) result incorrect length", 4, 
a.length);
-        assertTrue("toCharArray(int,int) result does not match", 
Arrays.equals("juni".toCharArray(), a));
+        assertEquals(4, a.length, "toCharArray(int,int) result incorrect 
length");
+        assertTrue(Arrays.equals("juni".toCharArray(), a), 
"toCharArray(int,int) result does not match");
 
         a = sb.toCharArray(0, 4);
-        assertEquals("toCharArray(int,int) result incorrect length", 4, 
a.length);
-        assertTrue("toCharArray(int,int) result does not match", 
Arrays.equals("juni".toCharArray(), a));
+        assertEquals(4, a.length, "toCharArray(int,int) result incorrect 
length");
+        assertTrue(Arrays.equals("juni".toCharArray(), a), 
"toCharArray(int,int) result does not match");
 
         a = sb.toCharArray(0, 1);
-        assertNotNull("toCharArray(int,int) result is null", a);
+        assertNotNull(a, "toCharArray(int,int) result is null");
 
         try {
             sb.toCharArray(-1, 5);
@@ -1045,7 +1046,7 @@ public class StrBuilderTest {
         sb = new StrBuilder("aaxaaaayaa");
         try {
             sb.replace(StrMatcher.stringMatcher("aa"), "-", 11, sb.length(), 
-1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         assertEquals("aaxaaaayaa", sb.toString());
@@ -1053,7 +1054,7 @@ public class StrBuilderTest {
         sb = new StrBuilder("aaxaaaayaa");
         try {
             sb.replace(StrMatcher.stringMatcher("aa"), "-", -1, sb.length(), 
-1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         assertEquals("aaxaaaayaa", sb.toString());
@@ -1108,7 +1109,7 @@ public class StrBuilderTest {
         sb = new StrBuilder("aaxaaaayaa");
         try {
             sb.replace(StrMatcher.stringMatcher("aa"), "-", 2, 1, -1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         assertEquals("aaxaaaayaa", sb.toString());
@@ -1216,28 +1217,28 @@ public class StrBuilderTest {
         // Start index is negative
         try {
             sb.subSequence(-1, 5);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
         // End index is negative
         try {
             sb.subSequence(2, -1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
         // End index greater than length()
         try {
             sb.subSequence(2, sb.length() + 1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
         // Start index greater then end index
         try {
             sb.subSequence(3, 2);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
@@ -1257,13 +1258,13 @@ public class StrBuilderTest {
         assertEquals("hello goodbye".substring(0), sb.substring(0));
         try {
             sb.substring(-1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
         try {
             sb.substring(15);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
@@ -1282,13 +1283,13 @@ public class StrBuilderTest {
 
         try {
             sb.substring(-1, 5);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
 
         try {
             sb.substring(15, 20);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException e) {
         }
     }
@@ -1765,27 +1766,27 @@ public class StrBuilderTest {
         array = new char[3];
         try {
             reader.read(array, -1, 0);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         try {
             reader.read(array, 0, -1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         try {
             reader.read(array, 100, 1);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         try {
             reader.read(array, 0, 100);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
         try {
             reader.read(array, Integer.MAX_VALUE, Integer.MAX_VALUE);
-            fail();
+            fail("Exception expected!");
         } catch (final IndexOutOfBoundsException ex) {
         }
 
@@ -1957,8 +1958,8 @@ public class StrBuilderTest {
     public void testLang295() {
         final StrBuilder sb = new StrBuilder("onetwothree");
         sb.deleteFirst("three");
-        assertFalse("The contains(char) method is looking beyond the end of 
the string", sb.contains('h'));
-        assertEquals("The indexOf(char) method is looking beyond the end of 
the string", -1, sb.indexOf('h'));
+        assertFalse(sb.contains('h'), "The contains(char) method is looking 
beyond the end of the string");
+        assertEquals(-1, sb.indexOf('h'), "The indexOf(char) method is looking 
beyond the end of the string");
     }
 
     // -----------------------------------------------------------------------
@@ -1966,14 +1967,14 @@ public class StrBuilderTest {
     public void testLang412Right() {
         final StrBuilder sb = new StrBuilder();
         sb.appendFixedWidthPadRight(null, 10, '*');
-        assertEquals("Failed to invoke appendFixedWidthPadRight correctly", 
"**********", sb.toString());
+        assertEquals("**********", sb.toString(), "Failed to invoke 
appendFixedWidthPadRight correctly");
     }
 
     @Test
     public void testLang412Left() {
         final StrBuilder sb = new StrBuilder();
         sb.appendFixedWidthPadLeft(null, 10, '*');
-        assertEquals("Failed to invoke appendFixedWidthPadLeft correctly", 
"**********", sb.toString());
+        assertEquals("**********", sb.toString(), "Failed to invoke 
appendFixedWidthPadLeft correctly");
     }
 
     @Test
@@ -2122,27 +2123,33 @@ public class StrBuilderTest {
         assertEquals("c" + System.lineSeparator(), 
sb1.appendln(ch).toString());
     }
 
-    @Test(expected = StringIndexOutOfBoundsException.class)
+    @Test
     public void 
testAppendTakingTwoIntsWithZeroThrowsStringIndexOutOfBoundsException() {
-        final Charset charset = Charset.defaultCharset();
-        final ByteBuffer byteBuffer = charset.encode("end < start");
-        final CharBuffer charBuffer = charset.decode(byteBuffer);
+        
assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() 
-> {
+            final Charset charset = Charset.defaultCharset();
+            final ByteBuffer byteBuffer = charset.encode("end < start");
+            final CharBuffer charBuffer = charset.decode(byteBuffer);
 
-        new StrBuilder(630).append(charBuffer, 0, 630);
+            new StrBuilder(630).append(charBuffer, 0, 630);
+        });
     }
 
-    @Test(expected = StringIndexOutOfBoundsException.class)
+    @Test
     public void 
testAppendTakingTwoIntsWithIndexOutOfBoundsThrowsStringIndexOutOfBoundsExceptionTwo()
 {
-        final Charset charset = Charset.defaultCharset();
-        final ByteBuffer byteBuffer = charset.encode("asdf");
-        final CharBuffer charBuffer = charset.decode(byteBuffer);
+        
assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() 
-> {
+            final Charset charset = Charset.defaultCharset();
+            final ByteBuffer byteBuffer = charset.encode("asdf");
+            final CharBuffer charBuffer = charset.decode(byteBuffer);
 
-        new StrBuilder().append(charBuffer, 933, 654);
+            new StrBuilder().append(charBuffer, 933, 654);
+        });
     }
 
-    @Test(expected = StringIndexOutOfBoundsException.class)
+    @Test
     public void testDeleteCharAtWithNegative() {
-        new StrBuilder().deleteCharAt((-1258));
+        
assertThatExceptionOfType(StringIndexOutOfBoundsException.class).isThrownBy(() 
-> {
+            new StrBuilder().deleteCharAt((-1258));
+        });
     }
 
 }

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/StrLookupTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/StrLookupTest.java 
b/src/test/java/org/apache/commons/text/StrLookupTest.java
index 62330e7..c765b5a 100644
--- a/src/test/java/org/apache/commons/text/StrLookupTest.java
+++ b/src/test/java/org/apache/commons/text/StrLookupTest.java
@@ -16,16 +16,16 @@
  */
 package org.apache.commons.text;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.fail;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Properties;
 import java.util.ResourceBundle;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Test class for {@link StrLookup}.
@@ -50,7 +50,7 @@ public class StrLookupTest  {
         assertNull(StrLookup.systemPropertiesLookup().lookup("other"));
         try {
             StrLookup.systemPropertiesLookup().lookup(null);
-            fail();
+            fail("Exception expected!");
         } catch (final NullPointerException ex) {
             // expected
         }
@@ -72,7 +72,7 @@ public class StrLookupTest  {
         newProps.setProperty(osName, newOsName);
         System.setProperties(newProps);
         try {
-            assertEquals("Changed properties not detected", newOsName, 
sysLookup.lookup(osName));
+            assertEquals(newOsName, sysLookup.lookup(osName), "Changed 
properties not detected");
         } finally {
             System.setProperties(oldProperties);
         }
@@ -91,7 +91,7 @@ public class StrLookupTest  {
         final StrLookup<String> sysLookup = StrLookup.systemPropertiesLookup();
         System.setProperty(osName, newOsName);
         try {
-            assertEquals("Changed properties not detected", newOsName, 
sysLookup.lookup(osName));
+            assertEquals(newOsName, sysLookup.lookup(osName), "Changed 
properties not detected");
         } finally {
             System.setProperty(osName, oldOs);
         }

http://git-wip-us.apache.org/repos/asf/commons-text/blob/49db4cb3/src/test/java/org/apache/commons/text/StrMatcherTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/commons/text/StrMatcherTest.java 
b/src/test/java/org/apache/commons/text/StrMatcherTest.java
index 22278f8..f35647f 100644
--- a/src/test/java/org/apache/commons/text/StrMatcherTest.java
+++ b/src/test/java/org/apache/commons/text/StrMatcherTest.java
@@ -18,7 +18,7 @@ package org.apache.commons.text;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests for {@link StrMatcher}.

Reply via email to