Beefy Boxes and Bandwidth Generously Provided by pair Networks
XP is just a number

Re^6: Universal test flag

by chromatic (Archbishop)
on Jul 19, 2012 at 19:06 UTC ( #982692=note: print w/ replies, xml ) Need Help??

in reply to Re^5: Universal test flag
in thread Universal test flag

I found tobyink's approach the best because it would even allow to run the app for integral testing with certain parts shut-off. I don't see this any different that skipping tests from the test suite.

The question is what's in control, the tests or something external.

There may be confirmation bias here, but every time I've modified my code to add a branch to distinguish between testing and deployment, it's bitten me later. I'm even leery of using separate Catalyst configurations for testing and deployment because those differences exist (I do use those configurations and usually very productively, but they introduce a risk I'm still not completely comfortable with).

As an intermediate step between "We don't have tests for this code" and "We have great tests for this code and we've refactored the code to be simple and testable and clean", debugging sections in the code are workable. I could live with them, as long as there's a plan to replace them with something better in the near future.

You wrote about management and business priorities and you're right to mention that. Getting some working tests is better than having no working tests—but even so, I just can't convince myself to have a lot of confidence in those tests until the differences between the code running under testing and the code running when deployed are minimal.

(That's why I use very, very minimal mock objects.)

Comment on Re^6: Universal test flag

Log In?

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

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (6)
As of 2014-10-20 04:26 GMT
Find Nodes?
    Voting Booth?

    For retirement, I am banking on:

    Results (72 votes), past polls