Skip Menu |

This queue is for tickets about the Tickit-Console CPAN distribution.

Report information
The Basics
Id: 101718
Status: resolved
Priority: 0/
Queue: Tickit-Console

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.07
Fixed in: 0.09



Subject: Test suite fails for almost all TZ settings
This works: $ env TZ=Europe/London ./Build test ... Result: PASS This fails: $ env TZ=Europe/Berlin ./Build test ... t/06timestamp.t .... 1/? # Failed test 'Display after tab->append_line with timestamp' # at t/06timestamp.t line 32. # Display differs on line 0 at column 0 # Got: '[22:33:09] First line' # Expected: '[21:33:09] First line' # Failed test 'Display after tab->append_line with formatted timestamp' # at t/06timestamp.t line 56. # Display differs on line 0 at column 1 # Got: '22' # Expected: '21' # Failed test 'Display after tab->append_line with datestamp' # at t/06timestamp.t line 85. # Display differs on line 1 at column 0 # Got: '[22:33] First line' # Expected: '[21:33] First line' # Failed test 'Display after tab->append_line with datestamp' # at t/06timestamp.t line 118. # Display differs on line 1 at column 0 # Got: '[15:41] Third line' # Expected: '[14:41] Third line' # Looks like you failed 4 tests of 4. t/06timestamp.t .... Dubious, test returned 4 (wstat 1024, 0x400) Failed 4/4 subtests t/99pod.t .......... ok Test Summary Report ------------------- t/06timestamp.t (Wstat: 1024 Tests: 4 Failed: 4) Failed tests: 1-4 Non-zero exit status: 4 Files=8, Tests=51, 1 wallclock secs ( 0.04 usr 0.05 sys + 0.62 cusr 0.15 csys = 0.86 CPU) Result: FAIL Failed 1/8 test programs. 4/51 subtests failed. (And it seems that any other non-UTC timezone setting fails)
On Sat Jan 24 05:36:15 2015, SREZIC wrote: Show quoted text
> This works: > > $ env TZ=Europe/London ./Build test > ... > Result: PASS > > This fails: > > $ env TZ=Europe/Berlin ./Build test > ...
Hi LeoNerd, The attached patch makes t/06timestamp.t timezone-safe. It uses too many auxiliary variables to be pretty, but it does the job. Also, I may be calling TEXT inappropriately. Cheers, Anno
Subject: patch
Download patch
application/octet-stream 5.9k

Message body not shown because it is not plain text.

Now fixed in latest release. -- Paul Evans