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


The following commit(s) were added to refs/heads/master by this push:
     new 511031644 Better test failure message
511031644 is described below

commit 511031644d28dd68c974dd73df7dab8b0dae86a0
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Thu Aug 10 09:31:41 2023 -0400

    Better test failure message
---
 .../commons/lang3/time/FastDateParser_TimeZoneStrategyTest.java  | 9 +++++----
 1 file changed, 5 insertions(+), 4 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 a6be1eed7..34a96d342 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
@@ -48,8 +48,8 @@ public class FastDateParser_TimeZoneStrategyTest extends 
AbstractLangTest {
         final FastDateParser parser = new FastDateParser("z", tzDefault, 
locale);
         final String[][] zones = 
DateFormatSymbols.getInstance(locale).getZoneStrings();
         for (final String[] zone : zones) {
-            for (int t = 1; t < zone.length; ++t) {
-                final String tzDisplay = zone[t];
+            for (int zIndex = 1; zIndex < zone.length; ++zIndex) {
+                final String tzDisplay = zone[zIndex];
                 if (tzDisplay == null) {
                     break;
                 }
@@ -57,8 +57,9 @@ public class FastDateParser_TimeZoneStrategyTest extends 
AbstractLangTest {
                 try {
                     parser.parse(tzDisplay);
                 } catch (ParseException e) {
-                    fail(String.format("%s: with tzDefault = %s, locale = %s, 
zones[][] size = '%s', zone[] size = '%s', tzDisplay = '%s', parser = '%s'", e,
-                            tzDefault, locale, zones.length, zone.length, 
tzDisplay, parser.toStringAll()), e);
+                    fail(String.format(
+                            "%s: with tzDefault = %s, locale = %s, zones[][] 
size = '%s', zone[] size = '%s', zIndex = %,d, tzDisplay = '%s', parser = 
'%s'", e,
+                            tzDefault, locale, zones.length, zone.length, 
zIndex, tzDisplay, parser.toStringAll()), e);
                 }
             }
         }

Reply via email to