This is an automated email from the ASF dual-hosted git repository. garydgregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-text.git
commit 4c73a71062a7bede27b344d6445252e8f43cbf31 Author: Gary Gregory <[email protected]> AuthorDate: Tue Jun 2 07:31:16 2026 -0400 NumericEntityUnescaperTest: Use less vertical space --- .../text/translate/NumericEntityUnescaperTest.java | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/src/test/java/org/apache/commons/text/translate/NumericEntityUnescaperTest.java b/src/test/java/org/apache/commons/text/translate/NumericEntityUnescaperTest.java index e3205e6e..d98ef7a7 100644 --- a/src/test/java/org/apache/commons/text/translate/NumericEntityUnescaperTest.java +++ b/src/test/java/org/apache/commons/text/translate/NumericEntityUnescaperTest.java @@ -25,28 +25,25 @@ import org.junit.jupiter.api.Test; /** * Tests {@link NumericEntityUnescaper}. */ -class NumericEntityUnescaperTest { +class NumericEntityUnescaperTest { @Test void testCreatesNumericEntityUnescaperOne() { final NumericEntityUnescaper.OPTION[] numericEntityUnescaperOPTIONArray = {}; - final NumericEntityUnescaper numericEntityUnescaper = - new NumericEntityUnescaper(numericEntityUnescaperOPTIONArray); + final NumericEntityUnescaper numericEntityUnescaper = new NumericEntityUnescaper(numericEntityUnescaperOPTIONArray); assertEquals("2|y|O7y`&#uVWj", numericEntityUnescaper.translate("2|y|O7y`&#uVWj")); } @Test void testCreatesNumericEntityUnescaperTwo() { final NumericEntityUnescaper.OPTION[] numericEntityUnescaperOPTIONArray = {}; - final NumericEntityUnescaper numericEntityUnescaper = - new NumericEntityUnescaper(numericEntityUnescaperOPTIONArray); + final NumericEntityUnescaper numericEntityUnescaper = new NumericEntityUnescaper(numericEntityUnescaperOPTIONArray); assertEquals("Ws2v8|O=7NR&#cB", numericEntityUnescaper.translate("Ws2v8|O=7NR&#cB")); } @Test void testOutOfBounds() { final NumericEntityUnescaper neu = new NumericEntityUnescaper(); - assertEquals("Test &", neu.translate("Test &"), "Failed to ignore when last character is &"); assertEquals("Test &#", neu.translate("Test &#"), "Failed to ignore when last character is &"); assertEquals("Test &#x", neu.translate("Test &#x"), "Failed to ignore when last character is &"); @@ -56,7 +53,6 @@ class NumericEntityUnescaperTest { @Test void testOutOfRangeCodePoint() { final NumericEntityUnescaper neu = new NumericEntityUnescaper(); - assertEquals("�", neu.translate("�"), "Failed to ignore code point above 0x10FFFF"); assertEquals("�", neu.translate("�"), "Failed to ignore code point above 0x10FFFF"); assertEquals("�", neu.translate("�"), "Failed to ignore code point above 0x10FFFF"); @@ -67,7 +63,6 @@ class NumericEntityUnescaperTest { final NumericEntityUnescaper neu = new NumericEntityUnescaper(); final String input = "𐰢"; final String expected = "\uD803\uDC22"; - final String result = neu.translate(input); assertEquals(expected, result, "Failed to unescape numeric entities supplementary characters"); } @@ -78,22 +73,17 @@ class NumericEntityUnescaperTest { NumericEntityUnescaper neu = new NumericEntityUnescaper(NumericEntityUnescaper.OPTION.semiColonOptional); String input = "Test 0 not test"; String expected = "Test \u0030 not test"; - String result = neu.translate(input); assertEquals(expected, result, "Failed to support unfinished entities (i.e. missing semicolon)"); - // ignore it neu = new NumericEntityUnescaper(); input = "Test 0 not test"; expected = input; - result = neu.translate(input); assertEquals(expected, result, "Failed to ignore unfinished entities (i.e. missing semicolon)"); - // fail it neu = new NumericEntityUnescaper(NumericEntityUnescaper.OPTION.errorIfNoSemiColon); input = "Test 0 not test"; - try { result = neu.translate(input); fail("IllegalArgumentException expected"); @@ -101,5 +91,4 @@ class NumericEntityUnescaperTest { // expected } } - }
