Skip Menu |

This queue is for tickets about the Date-Formatter CPAN distribution.

Report information
The Basics
Id: 109066
Status: open
Priority: 0/
Queue: Date-Formatter
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.11
Fixed in: (no value)

History Show all quoted text
  Tue Nov 17 03:22:45 2015 SREZIC [...] cpan.org - Ticket created
Subject: Tests fail (with latest DateTime::Locale?)
Test suite fails, probably only with recent DateTime::Locale versions (1.01, maybe also 0.95): Invalid locale code or name: fr-ca # Looks like you planned 2 tests but ran 1. # Looks like your test exited with 255 just after 1. t/70_localization.t ....................... Dubious, test returned 255 (wstat 65280, 0xff00) Failed 1/2 subtests
  Tue Dec 01 05:58:31 2015 ilmari+cpan [...] ilmari.org - Correspondence added
Subject: Patch for test failure with latest DateTime::Locale
On 2015-11-17 08:22:45, SREZIC wrote: Show quoted text
> Invalid locale code or name: fr-ca
The attached patch fixes it, by using the proper locale name "fr-CA" as documented in DateTime::Locale::Catalog.
Subject: 0001-Use-a-proper-locale-name-for-the-test.patch
From e7c40c68fdbfea3148352453ff0c916c55208621 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?D=2E=20Ilmari=20Manns=C3=A5ker?= <ilmari.mannsaker@net-a-porter.com> Date: Tue, 1 Dec 2015 10:50:26 +0000 Subject: [PATCH] Use a proper locale name for the test DateTime::Locale 1.00 uses the latest version of CLDR and does not include locales not in that data, breaking this test. --- t/70_localization.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/70_localization.t b/t/70_localization.t index 3568ef8..038eb1d 100644 --- a/t/70_localization.t +++ b/t/70_localization.t @@ -5,7 +5,7 @@ use_ok( 'Date::Formatter' ); -my $date = Date::Formatter->now( locale => 'fr_ca' ); +my $date = Date::Formatter->now( locale => 'fr-CA' ); my $day_of_week = $date->getDayOfWeek; like( $day_of_week, qr/dimanche|lundi|mardi|mercredi|jeudi|vendredi|samedi|dimanche/ ); -- 2.6.2
  Tue Dec 01 05:58:32 2015 The RT System itself - Status changed from 'new' to 'open'