Beefy Boxes and Bandwidth Generously Provided by pair Networks
Your skill will accomplish
what the force of many cannot
 
PerlMonks  

Re^3: Mocking isa under Test::Deep (sentinel)

by BrowserUk (Pope)
on May 03, 2016 at 04:26 UTC ( #1162060=note: print w/replies, xml ) Need Help??


in reply to Re^2: Mocking isa under Test::Deep (sentinel)
in thread Mocking isa under Test::Deep

A more accurate summary would be ...

I'll take your word on that. I just looked through it all again and I can see that my original conclusion was less than the full story.

That said, the breaking change is to Test::Deeply; appears (to me) to be digging into the guts of perl (via Scalar::Util::blessed()) to prevent their users from using fairly ordinary OO techniques to bypass their checks; and breaking stuff that worked for their users, all for the sake of the unjustified* and nebulous goal of 'improvement' of their own code base.

That's akin to Stanley embossing a weak point on the shafts of their screwdrivers to prevent users from using them as levers.

(*As in, not explained anywhere I could find.)


With the rise and rise of 'Social' network sites: 'Computers are making people easier to use everyday'
Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
"Science is about questioning the status quo. Questioning authority". I knew I was on the right track :)
In the absence of evidence, opinion is indistinguishable from prejudice.
  • Comment on Re^3: Mocking isa under Test::Deep (sentinel)

Replies are listed 'Best First'.
Re^4: Mocking isa under Test::Deep (blessed vs isa)
by tye (Sage) on May 03, 2016 at 17:26 UTC
    appears (to me) to be digging into the guts of perl (via Scalar::Util::blessed()) to prevent their users from using fairly ordinary OO techniques to bypass their checks

    Actually not. You could remove the use of Scalar::Util::blessed() and the code would function identically. Test::Deep is using isa() to enable users to extend the Test::Deep::Cmp functionality using ordinary OO techniques.

    If they had actually used blessed() in a way that prevents such extending then the "breaking change" would have been avoided. Replace the new code:

    if (! $Expects and Scalar::Util::blessed($d1) and $d1->isa("Test::Deep +::Cmp"))

    with code that looks too closely under the covers:

    if (! $Expects and Scalar::Util::blessed($d1) eq "Test::Deep::Cmp")

    and the fact that isa() was intentionally broken in order to do a sloppy hack to subvert draconian type checking would not have mattered.

    - tye        

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1162060]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others perusing the Monastery: (4)
As of 2019-10-20 21:45 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Notices?