Bug #110087 for Archive-Zip: Test::MockModule + SUPER scare me

Fri Dec 04 15:06:18 2015 mst [...] shadowcat.co.uk - Ticket created

Subject:	Test::MockModule + SUPER scare me
Date:	Fri, 4 Dec 2015 20:05:58 +0000
To:	bugs-Archive-Zip [...] rt.cpan.org
From:	Matt S Trout <mst [...] shadowcat.co.uk>

https://api.metacpan.org/source/CHROMATIC/SUPER-1.20141117/lib/SUPER.pm I'm not amazingly fond of the idea of monkeypatching UNIVERSAL in a test suite, for something that's this crucial to CPAN in general doing the right thing. Might I suggest Sub::Override and Mock::Quick as other things you might depend on - or just saying "ach, I'm already in the coal mines" and doing it the old fashioned way with scratch classes and or 'local *Foo::bar = sub {' -- Matt S Trout - Shadowcat Systems - Perl consulting with a commit bit and a clue http://shadowcat.co.uk/blog/matt-s-trout/ http://twitter.com/shadowcat_mst/ Email me now on mst (at) shadowcat.co.uk and let's chat about how our CPAN commercial support, training and consultancy packages could help your team.

Fri Dec 04 20:32:04 2015 fred [...] redhotpenguin.com - Correspondence added

Subject:	Re: [rt.cpan.org #110087] Test::MockModule + SUPER scare me
Date:	Fri, 4 Dec 2015 17:31:07 -0800
To:	bug-Archive-Zip [...] rt.cpan.org
From:	Fred Moyer <fred [...] redhotpenguin.com>

Ok I'll give that a shot, thanks for the pointer. On Fri, Dec 4, 2015 at 12:06 PM, Matt S Trout via RT <bug-Archive-Zip@rt.cpan.org> wrote: Show quoted text

> Fri Dec 04 15:06:18 2015: Request 110087 was acted upon. > Transaction: Ticket created by mst@shadowcat.co.uk > Queue: Archive-Zip > Subject: Test::MockModule + SUPER scare me > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: mst@shadowcat.co.uk > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=110087 > > > > https://api.metacpan.org/source/CHROMATIC/SUPER-1.20141117/lib/SUPER.pm > > I'm not amazingly fond of the idea of monkeypatching UNIVERSAL in a test > suite, for something that's this crucial to CPAN in general doing the right > thing. > > Might I suggest Sub::Override and Mock::Quick as other things you might > depend on - or just saying "ach, I'm already in the coal mines" and doing > it the old fashioned way with scratch classes and or 'local *Foo::bar = sub {' > > -- > Matt S Trout - Shadowcat Systems - Perl consulting with a commit bit and a clue > > http://shadowcat.co.uk/blog/matt-s-trout/ http://twitter.com/shadowcat_mst/ > > Email me now on mst (at) shadowcat.co.uk and let's chat about how our CPAN > commercial support, training and consultancy packages could help your team. >

Fri Dec 04 20:32:05 2015 The RT System itself - Status changed from 'new' to 'open'