Bug #127642 for HTTP-Request-FromCurl: Failed test '--verbose -s $url/foo/..'

Wed Nov 14 16:23:20 2018 SREZIC [...] cpan.org - Ticket created

Subject:

Failed test '--verbose -s $url/foo/..'

On CentOS7 and Debian/wheezy the following test failure happens: ... # Curl version 7.26.0 # Failed test '--verbose -s $url/foo/..' # at t/curl-identity.t line 174. # got: '//' # expected: '//foo/..' # --verbose -s $url/foo/.. # Failed test 'planned to run 34 but done_testing() expects 32' # at t/curl-identity.t line 208. # Looks like you planned 34 tests but ran 33. # Looks like you failed 2 tests of 33 run. t/curl-identity.t .. Dubious, test returned 2 (wstat 512, 0x200) Failed 3/34 subtests (1 TODO test unexpectedly succeeded) ... This was the log from a Debian/wheezy system. CentOS7 has Curl version 7.29.0 installed.

Wed Nov 14 17:06:29 2018 corion [...] corion.net - Correspondence added

Subject:	Re: [rt.cpan.org #127642] Failed test '--verbose -s $url/foo/..'
Date:	Wed, 14 Nov 2018 23:06:09 +0100
To:	bug-HTTP-Request-FromCurl [...] rt.cpan.org
From:	Max Maischein <corion [...] corion.net>

Hello Slaven! Show quoted text

> On CentOS7 and Debian/wheezy the following test failure happens: > > ... > # Curl version 7.26.0 > > # Failed test '--verbose -s $url/foo/..' > # at t/curl-identity.t line 174. > # got: '//' > # expected: '//foo/..' > # --verbose -s $url/foo/.. > > # Failed test 'planned to run 34 but done_testing() expects 32' > # at t/curl-identity.t line 208. > # Looks like you planned 34 tests but ran 33. > # Looks like you failed 2 tests of 33 run. > t/curl-identity.t .. > Dubious, test returned 2 (wstat 512, 0x200) > Failed 3/34 subtests > (1 TODO test unexpectedly succeeded) > ... > > This was the log from a Debian/wheezy system. CentOS7 has Curl version 7.29.0 installed. >

Thanks for the report. I guess that versions (much) earlier than 7.61.0 don't have the path cleanup. I've marked the failing test as TODO for curl versions prior 7.61.0. -max

Wed Nov 14 17:06:29 2018 The RT System itself - Status changed from 'new' to 'open'

Fri Nov 16 01:28:08 2018 SREZIC [...] cpan.org - Correspondence added

On 2018-11-14 17:06:29, corion@corion.net wrote: Show quoted text

> Hello Slaven!

> > On CentOS7 and Debian/wheezy the following test failure happens: > > > > ... > > # Curl version 7.26.0 > > > > # Failed test '--verbose -s $url/foo/..' > > # at t/curl-identity.t line 174. > > # got: '//' > > # expected: '//foo/..' > > # --verbose -s $url/foo/.. > > > > # Failed test 'planned to run 34 but done_testing() expects 32' > > # at t/curl-identity.t line 208. > > # Looks like you planned 34 tests but ran 33. > > # Looks like you failed 2 tests of 33 run. > > t/curl-identity.t .. > > Dubious, test returned 2 (wstat 512, 0x200) > > Failed 3/34 subtests > > (1 TODO test unexpectedly succeeded) > > ... > > > > This was the log from a Debian/wheezy system. CentOS7 has Curl > > version 7.29.0 installed. > >

> Thanks for the report. I guess that versions (much) earlier than > 7.61.0 > don't have the path cleanup. I've marked the failing test as TODO for > curl versions prior 7.61.0.

With 0.02 the plan seems to be off in these situations: ... # Curl version 7.29.0 # Failed test 'planned to run 38 but done_testing() expects 36' # at t/curl-identity.t line 223. # Looks like you planned 38 tests but ran 37. # Looks like you failed 1 test of 37 run. t/curl-identity.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 2/38 subtests (1 TODO test unexpectedly succeeded) ...

Sat Nov 24 04:11:25 2018 corion [...] cpan.org - Correspondence added

Fixed, and also restructured the test so skipping tests actually works Thanks for reporting!

Sat Nov 24 04:11:26 2018 corion [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Nov 24 04:11:26 2018 corion [...] cpan.org - Fixed in 0.04 added