This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-lang.git
commit 18b7e5e8248cb1db67e495056d5713d330538a5b Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Thu Dec 5 15:49:44 2024 -0500 Better unit test failure messages --- .../commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/apache/commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java b/src/test/java/org/apache/commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java index 1f1f7aa7a..7af947769 100644 --- a/src/test/java/org/apache/commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java +++ b/src/test/java/org/apache/commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java @@ -30,6 +30,7 @@ import java.util.List; import java.util.Locale; import java.util.Objects; import java.util.TimeZone; +import java.util.concurrent.atomic.AtomicInteger; import org.apache.commons.lang3.AbstractLangTest; import org.apache.commons.lang3.ArraySorter; @@ -56,6 +57,7 @@ public class FastDateParser_TimeZoneStrategyTest extends AbstractLangTest { private static final List<Locale> Java11Failures = new ArrayList<>(); private static final List<Locale> Java17Failures = new ArrayList<>(); + private static final AtomicInteger fails = new AtomicInteger(); @AfterAll public static void afterAll() { @@ -129,7 +131,8 @@ public class FastDateParser_TimeZoneStrategyTest extends AbstractLangTest { Java17Failures.add(locale); // Mark as an assumption failure instead of a hard fail System.err.printf( - "Java %s %s - Mark as an assumption failure instead of a hard fail: locale = '%s', parse = '%s'%n", + "[%,d] Java %s %s - Mark as an assumption failure instead of a hard fail: locale = '%s', parse = '%s'%n", + fails.incrementAndGet(), SystemUtils.JAVA_VENDOR, SystemUtils.JAVA_VM_VERSION, localeStr, tzDisplay); @@ -141,7 +144,8 @@ public class FastDateParser_TimeZoneStrategyTest extends AbstractLangTest { Java11Failures.add(locale); // Mark as an assumption failure instead of a hard fail System.err.printf( - "Java %s %s - Mark as an assumption failure instead of a hard fail: locale = '%s', parse = '%s'%n", + "[%,d] Java %s %s - Mark as an assumption failure instead of a hard fail: locale = '%s', parse = '%s'%n", + fails.incrementAndGet(), SystemUtils.JAVA_VENDOR, SystemUtils.JAVA_VM_VERSION, localeStr, tzDisplay);