From cb4c3a5e21e1d75bdad5c95afe2ab0e274634d25 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 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. Signed-off-by: Fridolin Somers --- 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.39.2