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);

Reply via email to