commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Julius Davies <juliusdav...@gmail.com>
Subject Re: [codec] 1.4 release: code coverage help needed
Date Wed, 15 Jul 2009 21:41:21 GMT
Hi, Gary,

CODEC-78 created and patch attached to address this.

I used IntelliJ's EMMA.  Hopefully Code Coverage results are similar!

yours,

Julius


On Wed, Jul 15, 2009 at 12:59 PM, Gary
Gregory<GGregory@seagullsoftware.com> wrote:
> Taking a quick look at the Cobertura coverage line/branch reports:
>
> Base64InputStream 85%/76%
> Base64OutputStream 88%/70%
>
> Gary
>
>> -----Original Message-----
>> From: Julius Davies [mailto:juliusdavies@gmail.com]
>> Sent: Wednesday, July 15, 2009 12:55 PM
>> To: Commons Developers List
>> Subject: Re: [codec] 1.4 release: code coverage help needed
>>
>> Hi, Gary,
>>
>> I think the streaming Base64 is adequately exercised by the tests
>> inside Base64InputStreamTest.java and Base64OutputStreamTest.java.
>> It's also worth noting that all the original Base64 operations are now
>> implemented via the streaming, so the original unit tests from 1.3
>> provide a good workout, too.
>>
>> I might add a couple more unit tests over the next few days and attach
>> them to Jira tickets.
>>
>>
>> yours,
>>
>> Julius
>>
>>
>> On Wed, Jul 15, 2009 at 12:32 PM, Gary
>> Gregory<GGregory@seagullsoftware.com> wrote:
>> > Hello All:
>> >
>> > I would like to ask your help to improve test coverage especially for
>> the new code. Perhaps the author of the new streaming Base64 classes would
>> like to help here since they know the code the best.
>> >
>> > If you are a committer, commit, if not please submit a patch.
>> >
>> > Gary
>> >

-- 
yours,

Julius Davies
250-592-2284 (Home)
250-893-4579 (Mobile)
http://juliusdavies.ca/logging.html

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message