From 31213b6dfb8abd8302ee01e429c23a22f65f0d2f Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Tue, 21 Dec 2021 11:44:44 +0000 Subject: [PATCH] Bug 29403: Fix unit test for bug 29718 Content-Type: text/plain; charset=utf-8 Including a 'T' is actually valid for ISO date format, I was being naughty and writing the test to follow our code rather than writing the test to conform to the ISO spec first time around. Now we are properly supporting 'T' in ISO dates with bug 29718, we need to update the test here to properly reflect an invalid ISO date string. Signed-off-by: Jonathan Druart Signed-off-by: Marcel de Rooy The commit message is misleading here. Look at the Z in the code changes. --- t/DateUtils.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/DateUtils.t b/t/DateUtils.t index 3aae15162e..447d75ffed 100755 --- a/t/DateUtils.t +++ b/t/DateUtils.t @@ -143,7 +143,7 @@ eval { like( $@, qr/.*does not match the date format \(rfc3339\).*/, 'dt_from_string should die when passed a bad rfc3339 date string' ); eval { - $dt0 = dt_from_string('2021-11-03T10:16:59+00:00', 'iso'); + $dt0 = dt_from_string('2021-11-03T10:16:59Z+00:00', 'iso'); }; like( $@, qr/.*does not match the date format \(iso\).*/, 'dt_from_string should die when passed a bad iso date string' ); -- 2.20.1