fix: backward compatible support for Z UTC designator in timestamps

pact-foundation · Jun 15, 2020 · 2a16da9 · 2a16da9
1 parent d87fef5
commit 2a16da9
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 6 deletions.
diff --git a/core/matchers/src/main/kotlin/au/com/dius/pact/core/matchers/MatcherExecutor.kt b/core/matchers/src/main/kotlin/au/com/dius/pact/core/matchers/MatcherExecutor.kt
@@ -324,7 +324,11 @@ fun <M : Mismatch> matchDateTime(
     emptyList()
   } else {
     try {
-      DateTimeFormatter.ofPattern(pattern).parse(safeToString(actual))
+      /**
+        Replacing 'Z' with 'X' in order to offer backward compatibility for consumers who might be
+        using ISO 8601 the UTC designator 'Z'
+       **/
+      DateTimeFormatter.ofPattern(pattern.replace('Z', 'X')).parse(safeToString(actual))
       emptyList<M>()
     } catch (e: DateTimeParseException) {
       listOf(mismatchFactory.create(expected, actual,

diff --git a/core/matchers/src/test/groovy/au/com/dius/pact/core/matchers/MatcherExecutorSpec.groovy b/core/matchers/src/test/groovy/au/com/dius/pact/core/matchers/MatcherExecutorSpec.groovy
@@ -149,6 +149,13 @@ class MatcherExecutorSpec extends Specification {
     '2014-01-01 14:00:00+10:00'                   | '2013#12#01#14#00#00'                   | "yyyy'#'MM'#'dd'#'HH'#'mm'#'ss"  || true
     '2014-01-01 14:00:00+10:00'                   | null                                    | null                             || false
     '2014-01-01T10:00+10:00[Australia/Melbourne]' | '2020-01-01T10:00+01:00[Europe/Warsaw]' | "yyyy-MM-dd'T'HH:mmXXX'['zzz']'" || true
+    '2019-11-25T13:45:00+02:00'                   | '2019-11-25T11:45:00Z'                  | "yyyy-MM-dd'T'HH:mm:ssX"         || true
+    '2019-11-25T13:45:00+02:00'                   | '2019-11-25T11:45:00Z'                  | "yyyy-MM-dd'T'HH:mm:ssZZ"        || true
+    '2019-11-25T13:45:00+02:00'                   | '2019-11-25T11:45Z'                     | "yyyy-MM-dd'T'HH:mmZZ"           || true
+    '2019-11-25T13:45:00+02:00'                   | '2019-11-25T11Z'                        | "yyyy-MM-dd'T'HHZZ"              || true
+    '2019-11-25T13:45:00+0200'                    | '2019-11-25T11:45:00Z'                  | "yyyy-MM-dd'T'HH:mm:ssZ"         || true
+    '2019-11-25T13:45:00+0200'                    | '2019-11-25T11:45Z'                     | "yyyy-MM-dd'T'HH:mmZ"            || true
+    '2019-11-25T13:45:00+0200'                    | '2019-11-25T11Z'                        | "yyyy-MM-dd'T'HHZ"               || true
 
     matcher = pattern ? new TimestampMatcher(pattern) : new TimestampMatcher()
   }
@@ -159,11 +166,11 @@ class MatcherExecutorSpec extends Specification {
     MatcherExecutorKt.domatch(matcher, path, expected, actual, mismatchFactory).empty == mustBeEmpty
 
     where:
-    expected         | actual     | pattern    || mustBeEmpty
-    '14:00:00'       | '14:00:00' | null       || true
-    '00:00'          | '14:01:02' | 'mm:ss'    || false
-    '00:00:14'       | '05:10:14' | 'ss:mm:HH' || true
-    '14:00:00+10:00' | null       | null       || false
+    expected         | actual       | pattern       || mustBeEmpty
+    '14:00:00'       | '14:00:00'   | null          || true
+    '00:00'          | '14:01:02'   | 'mm:ss'       || false
+    '00:00:14'       | '05:10:14'   | 'ss:mm:HH'    || true
+    '14:00:00+10:00' | null         | null          || false
 
     matcher = pattern ? new TimeMatcher(pattern) : new TimeMatcher()
   }