Bug #117204 for Finance-Bank-Postbank_de: t/04-form-accountstatement.t started to fail

Thu Aug 25 02:47:23 2016 SREZIC [...] cpan.org - Ticket created

Subject:

t/04-form-accountstatement.t started to fail

On my smoker machines the t/04-form-accountstatement.t test started to fail: ... # Failed test 'Download to file works' # at t/04-form-accountstatement.t line 85. Wide character in print at /opt/perl-5.24.0/lib/site_perl/5.24.0/Test2/Formatter/TAP.pm line 105. # Structures begin differing at: # $got->[8] = '"Überweisung";"111111/DE05370501981000000000 Finanzkasse 3991234 Steuernummer 00703434 ";"Petra Pfiffig";"Finanzkasse Köln-Süd";"-328,75 €";"5.314,05 €"' # $expected->[8] = '"Überweisung";"111111/DE05370501981000000000 Finanzkasse 3991234 Steuernummer 00703434 ";"Petra Pfiffig";"Finanzkasse Köln-Süd";"-¤ 328,75";"¤ 5.314,05"' # Looks like you failed 1 test of 8. t/04-form-accountstatement.t ...... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/8 subtests ...

Thu Aug 25 03:19:15 2016 webmaster [...] corion.net - Correspondence added

Subject:	Re: [rt.cpan.org #117204] t/04-form-accountstatement.t started to fail
Date:	Thu, 25 Aug 2016 10:18:53 +0300
To:	bug-Finance-Bank-Postbank_de [...] rt.cpan.org
From:	Max Maischein <webmaster [...] corion.net>

Hello Slaven, Thanks for the report! That means Postbank changed their CSV statement format again... Looks like I have to make the parsing and value conversion even more resilient... ( 328,75€ vs €328,75 ) Thanks again for cheking, -max Am 25. August 2016 09:47:34 OESZ, schrieb Slaven_Rezic via RT <bug-Finance-Bank-Postbank_de@rt.cpan.org>: Show quoted text

>Thu Aug 25 02:47:23 2016: Request 117204 was acted upon. >Transaction: Ticket created by SREZIC > Queue: Finance-Bank-Postbank_de > Subject: t/04-form-accountstatement.t started to fail > Broken in: 0.40 > Severity: (no value) > Owner: Nobody > Requestors: SREZIC@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=117204 > > > >On my smoker machines the t/04-form-accountstatement.t test started to >fail: > >... ># Failed test 'Download to file works' ># at t/04-form-accountstatement.t line 85. >Wide character in print at >/opt/perl-5.24.0/lib/site_perl/5.24.0/Test2/Formatter/TAP.pm line 105. ># Structures begin differing at: ># $got->[8] = '"Überweisung";"111111/DE05370501981000000000 >Finanzkasse 3991234 Steuernummer 00703434 ";"Petra >Pfiffig";"Finanzkasse Köln-Süd";"-328,75 €";"5.314,05 €"' ># $expected->[8] = '"Überweisung";"111111/DE05370501981000000000 >Finanzkasse 3991234 Steuernummer 00703434 ";"Petra >Pfiffig";"Finanzkasse Köln-Süd";"-¤ 328,75";"¤ 5.314,05"' ># Looks like you failed 1 test of 8. >t/04-form-accountstatement.t ...... >Dubious, test returned 1 (wstat 256, 0x100) >Failed 1/8 subtests >...

-- Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.

Thu Aug 25 03:19:16 2016 The RT System itself - Status changed from 'new' to 'open'

Tue Nov 22 00:52:16 2016 SREZIC [...] cpan.org - Correspondence added

RT-Send-CC:

webmaster [...] corion.net

On 2016-08-25 03:19:15, webmaster@corion.net wrote: Show quoted text

> Hello Slaven, > > Thanks for the report! > > That means Postbank changed their CSV statement format again... Looks > like I have to make the parsing and value conversion even more > resilient... ( 328,75€ vs €328,75 ) > > Thanks again for cheking, > -max > > Am 25. August 2016 09:47:34 OESZ, schrieb Slaven_Rezic via RT <bug- > Finance-Bank-Postbank_de@rt.cpan.org>:

> > Thu Aug 25 02:47:23 2016: Request 117204 was acted upon. > > Transaction: Ticket created by SREZIC > > Queue: Finance-Bank-Postbank_de > > Subject: t/04-form-accountstatement.t started to fail > > Broken in: 0.40 > > Severity: (no value) > > Owner: Nobody > > Requestors: SREZIC@cpan.org > > Status: new > > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=117204 > > > > > > > On my smoker machines the t/04-form-accountstatement.t test started > > to > > fail: > > > > ... > > # Failed test 'Download to file works' > > # at t/04-form-accountstatement.t line 85. > > Wide character in print at > > /opt/perl-5.24.0/lib/site_perl/5.24.0/Test2/Formatter/TAP.pm line > > 105. > > # Structures begin differing at: > > # $got->[8] = '"Überweisung";"111111/DE05370501981000000000 > > Finanzkasse 3991234 Steuernummer 00703434 ";"Petra > > Pfiffig";"Finanzkasse Köln-Süd";"-328,75 €";"5.314,05 €"' > > # $expected->[8] = '"Überweisung";"111111/DE05370501981000000000 > > Finanzkasse 3991234 Steuernummer 00703434 ";"Petra > > Pfiffig";"Finanzkasse Köln-Süd";"-¤ 328,75";"¤ 5.314,05"' > > # Looks like you failed 1 test of 8. > > t/04-form-accountstatement.t ...... > > Dubious, test returned 1 (wstat 256, 0x100) > > Failed 1/8 subtests > > ...

It seems it changed again: # Failed test 'Download to file works' # at t/04-form-accountstatement.t line 88. # Structures begin differing at: # $got->[8] = '"Überweisung";"111111/DE05370501981000000000 Finanzkasse 3991234 Steuernummer 00703434 ";"Petra Pfiffig";"Finanzkasse Köln-Süd";"-328,75 ";"5.314,05 "' # $expected->[8] = '"Überweisung";"111111/DE05370501981000000000 Finanzkasse 3991234 Steuernummer 00703434 ";"Petra Pfiffig";"Finanzkasse Köln-Süd";"-¤ 328,75";"¤ 5.314,05"' # Looks like you failed 1 test of 8. t/04-form-accountstatement.t ...... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/8 subtests

Tue Nov 22 00:52:24 2016 SREZIC [...] cpan.org - Broken in 0.43 added

Tue Nov 22 04:30:12 2016 corion [...] corion.net - Correspondence added

Subject:	Re: [rt.cpan.org #117204] t/04-form-accountstatement.t started to fail
Date:	Tue, 22 Nov 2016 10:29:41 +0100
To:	bug-Finance-Bank-Postbank_de [...] rt.cpan.org
From:	Max Maischein <corion [...] corion.net>

Hello Slaven, thank you very much for reporting this! I've pushed out one more change in the expected data to acommodate this change. Maybe I should make the check more resilient and not insist on a one-to-one identity... -max

Mon Jul 16 15:26:09 2018 corion [...] cpan.org - Correspondence added

Fixed in between, and at least with 0.50 Thanks again for reporting this

Mon Jul 16 15:26:10 2018 corion [...] cpan.org - Status changed from 'open' to 'resolved'