chromatic wrote:
Are the extra pass() calls at the end to fix the Devel::Cover oddity?
Where a pass() explicitly mentions Devel::Cover, it is there either
because I experienced the D::C oddity with that file, or because that
file is structured much like others where I have experienced the problem.
(For those wondering about the D::C oddity: http://tinyurl.com/2zh9l6.)
If they
absolutely have to be there (and I'm personally comfortable with less than
100% coverage if there are known bugs with D::C), they should probably have
comments to prevent otherwise well-meaning people like me from refactoring
code that's not immediately obvious.
You will only experience this problem if you are running the tests with
Devel::Cover turned on. I would not want you to refrain from
refactoring code or tests simply to keep Devel::Cover happy -- that's my
problem.
Does that clarify things?
kid51
