On Mar 21, 2007, at 4:09 AM, Glynn, Eoghan wrote:
So its not a case of extra code inserted to get the coverage to 100%
(which is a bit of an unrealistic goal anyway, if we had 80%
everywhere
we'd be doing very well and I'd be delighted). Instead it's a trivial
bit of code required to allow the HTTPConduit be tested in isolation.
Understood -- I appreciate the difficulty.
BTW speaking of testing, there doesn't seem to be any tests at all in
Polar's TrustDecider patch. A unit test (possibly extending the
existing
HTTPConduitTest) is probably the least required.
Yup -- I agree. This needs a test -- Polar should provide one with
his patch.
