Beefy Boxes and Bandwidth Generously Provided by pair Networks
Don't ask to ask, just ask

Re^14: Cleaning up unused subroutines

by Somni (Friar)
on Oct 26, 2007 at 19:19 UTC ( #647473=note: print w/replies, xml ) Need Help??

in reply to Re^13: Cleaning up unused subroutines
in thread Cleaning up unused subroutines

You wouldn't have to know that specifically foo() wasn't used, you'd simply go through each function, disabling its explicit tests then checking your coverage.

The test suite is being used here as an automated means of testing with as many inputs as possible. Something not easily done with your original proposed solution. Your solution misses code; utilizing the test suite has a chance of missing less.

Replies are listed 'Best First'.
Re^15: Cleaning up unused subroutines
by BrowserUk (Patriarch) on Oct 26, 2007 at 19:53 UTC
    you'd simply go through each function, disabling its explicit tests then checking your coverage.

    FGS! If you comment out the tests of foo() in your test suite, then that is exactly the same as if you had no test suite.

    And, as I showed above, if you have a function call in your code that doesn't have an associated function definition, then Devel::Cover doesn't detect it.

    All you are doing it testing your tests, not your code.

    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".
    In the absence of evidence, opinion is indistinguishable from prejudice.
      All right, I'm done. I've tried what I could to explain the merits and mechanics of the idea. If you want to discuss the value in a test suite I'm sure a top-level Meditation would be more appropriate, and I'm sure the rest of PerlMonks would be more than happy to oblige you in some lively discussion.

      Good day.

Log In?

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

How do I use this?Last hourOther CB clients
Other Users?
Others lurking in the Monastery: (6)
As of 2023-11-30 07:24 GMT
Find Nodes?
    Voting Booth?

    No recent polls found