Skip Menu |
 

This queue is for tickets about the Archive-Zip CPAN distribution.

Report information
The Basics
Id: 110087
Status: open
Priority: 0/
Queue: Archive-Zip

People
Owner: Nobody in particular
Requestors: mst [...] shadowcat.co.uk
Cc:
AdminCc:

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



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>
Download (untitled) / with headers
text/plain 771b
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.
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>
Download (untitled) / with headers
text/plain 1.3k
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. >


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.