Bug #69020 for IO-Async: t/21stream-4encoding.t failure

Wed Jun 22 05:02:42 2011 jquelin [...] cpan.org - Ticket created

Subject:

t/21stream-4encoding.t failure

$ make test [...] t/21stream-2write.t .......... ok t/21stream-3split.t .......... ok Nothing was ready after 10 second wait; called at t/21stream-4encoding.t line 93 # Looks like you planned 7 tests but ran 3. # Looks like your test exited with 255 just after 3. t/21stream-4encoding.t ....... Dubious, test returned 255 (wstat 65280, 0xff00) Failed 4/7 subtests t/22timer-absolute.t ......... ok t/22timer-countdown.t ........ ok [...] Test Summary Report ------------------- t/21stream-4encoding.t (Wstat: 65280 Tests: 3 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 7 tests but ran 3. Files=54, Tests=1339, 88 wallclock secs ( 0.36 usr 0.11 sys + 5.31 cusr 1.05 csys = 6.83 CPU) Result: FAIL Failed 1/54 test programs. 0/1339 subtests failed.

Wed Jun 22 09:16:00 2011 leonerd-cpan [...] leonerd.org.uk - Correspondence added

On Wed Jun 22 05:02:42 2011, JQUELIN wrote: Show quoted text

> $ make test > [...] > t/21stream-2write.t .......... ok > t/21stream-3split.t .......... ok > Nothing was ready after 10 second wait; called at t/21stream-

4encoding.t Show quoted text

> line 93

Ah yes, I'd noticed that one on the smokers results: http://matrix.cpantesters.org/?dist=IO-Async+0.41 Seems like something in 5.13 has changed about the way Encoding works. Once I've managed to hack myself up a perl 5.14 testing machine somehow I'll take a look into it. Meanwhile, if you want you can force install it over this test; the rest of the tests are happy, so it's just the new 'encoding' feature of IO::Async::Stream that appears not to be working. -- Paul Evans

Wed Jun 22 09:16:08 2011 The RT System itself - Status changed from 'new' to 'open'

Mon Jun 27 08:17:22 2011 leonerd-cpan [...] leonerd.org.uk - Correspondence added

Seems to be a slight change of error-handling behaviour between perl 5.13.1 and 5.13.2. The attached patch changes the test to send an invalid sequence that's still detected in 5.13.2 and above. Will be in 0.42. -- Paul Evans

Subject:

rt69020.patch

=== modified file 't/21stream-4encoding.t' --- t/21stream-4encoding.t 2011-04-14 15:23:21 +0000 +++ t/21stream-4encoding.t 2011-06-26 00:08:51 +0000 @@ -86,13 +86,13 @@ is( $read, "\x{10b}", 'Partial UTF-8 character visible after completion' ); - # 0xfe is never a valid UTF-8 byte - $wr->syswrite( "\xfe" ); + # An invalid sequence + $wr->syswrite( "\xc4!" ); $read = ""; wait_for { length $read }; - is( $read, "\x{fffd}", 'Invalid UTF-8 byte yields U+FFFD' ); + is( $read, "\x{fffd}!", 'Invalid UTF-8 byte yields U+FFFD' ); $loop->remove( $stream ); }

Mon Jun 27 08:17:23 2011 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'patched'

Wed Jun 29 08:03:28 2011 leonerd-cpan [...] leonerd.org.uk - Correspondence added

Now released in 0.42 -- Paul Evans

Wed Jun 29 08:03:29 2011 The RT System itself - Status changed from 'patched' to 'open'

Wed Jun 29 08:03:29 2011 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'resolved'

Wed Jun 29 08:03:39 2011 leonerd-cpan [...] leonerd.org.uk - Fixed in 0.42 added

Wed Jun 29 08:03:39 2011 leonerd-cpan [...] leonerd.org.uk - Broken in 0.41 added