Skip Menu |

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

Report information
The Basics
Id: 82738
Status: open
Priority: 0/
Queue: TAP-Formatter-HTML

People
Owner: steve [...] purkis.ca
Requestors: ovid [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.10



Subject: Installing this breaks Test::Harness tests
I'm trying to ship a new release of Test::Harness because it's blocking a new release of Perl, but I've hit this bug in the tests: t/compat/env_opts.t .. 1..12 ok 1 - made harness ok 2 - set color correctly ok 3 - set jobs correctly ok 4 - made harness Can't locate object method "color" via package "TAP::Formatter::HTML" at ... # Looks like you planned 12 tests but only ran 4. # Looks like your test died just after 4. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 8/12 subtests Test Summary Report ------------------- t/compat/env_opts.t (Wstat: 65280 Tests: 4 Failed: 0) Non-zero exit status: 255 Parse errors: Bad plan. You planned 12 tests but ran 4. Files=1, Tests=4, 1 wallclock secs ( 0.03 usr 0.01 sys + 0.14 cusr 0.02 csys = 0.20 CPU) Result: FAIL Cheers, Ovid
On Wed Jan 16 01:33:08 2013, OVID wrote: Show quoted text
> I'm trying to ship a new release of Test::Harness because it's > blocking a new release of Perl, but > I've hit this bug in the tests: > > t/compat/env_opts.t .. > 1..12 > ok 1 - made harness > ok 2 - set color correctly > ok 3 - set jobs correctly > ok 4 - made harness > Can't locate object method "color" via package > "TAP::Formatter::HTML" at ...
That's new :) TAP::Formatter::HTML ignores the color arg to new, and never expected anyone to call the method directly (it doesn't make sense for this kind of formatter - it's really an argument specific to the console formatter). At any rate, it's easy enough to add it in. I'll do that & ship a new release. See: https://github.com/spurkis/TAP-Formatter-HTML -Steve
Fixed in 0.10, on it's way to CPAN. Note: I confirmed by un-skipping & running only this test: Test-Harness $ prove -I../TAP-Formatter-HTML-0.10/lib -lvm t/compat/env_opts.t t/compat/env_opts.t .. 1..12 ok 1 - made harness ok 2 - set color correctly ok 3 - set jobs correctly ok 4 - made harness ok 5 - set color correctly ok 6 - set jobs correctly ok 7 - correct formatter ok 8 # skip requires TAP::Harness::Archive ... ok All tests successful.
Subject: Re: [rt.cpan.org #82738] Installing this breaks Test::Harness tests
Date: Fri, 18 Jan 2013 22:22:20 -0800 (PST)
To: "bug-TAP-Formatter-HTML [...] rt.cpan.org" <bug-TAP-Formatter-HTML [...] rt.cpan.org>
From: Ovid <curtis_ovid_poe [...] yahoo.com>
Thanks, Steve :)   Cheers, Ovid -- Twitter - http://twitter.com/OvidPerl/ Buy my book - http://bit.ly/beginning_perl Buy my other book - http://www.oreilly.com/catalog/perlhks/ Live and work overseas - http://www.overseas-exile.com/ Show quoted text
>________________________________ > From: Steve Purkis via RT <bug-TAP-Formatter-HTML@rt.cpan.org> >To: ovid@cpan.org >Sent: Friday, 18 January 2013, 12:58 >Subject: [rt.cpan.org #82738] Installing this breaks Test::Harness tests > ><URL: https://rt.cpan.org/Ticket/Display.html?id=82738 > > >Fixed in 0.10, on it's way to CPAN. > >Note: I confirmed by un-skipping & running only this test: > >Test-Harness $ prove -I../TAP-Formatter-HTML-0.10/lib -lvm t/compat/env_opts.t >t/compat/env_opts.t .. >1..12 >ok 1 - made harness >ok 2 - set color correctly >ok 3 - set jobs correctly >ok 4 - made harness >ok 5 - set color correctly >ok 6 - set jobs correctly >ok 7 - correct formatter >ok 8 # skip requires TAP::Harness::Archive >... >ok >All tests successful. > > > > >