You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Gary Gregory <GG...@seagullsoftware.com> on 2009/07/15 21:32:15 UTC
[codec] 1.4 release: code coverage help needed
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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [codec] 1.4 release: code coverage help needed
Posted by Julius Davies <ju...@gmail.com>.
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<GG...@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<GG...@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
RE: [codec] 1.4 release: code coverage help needed
Posted by Gary Gregory <GG...@seagullsoftware.com>.
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<GG...@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
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
> >
>
>
>
> --
> 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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [codec] 1.4 release: code coverage help needed
Posted by Julius Davies <ju...@gmail.com>.
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<GG...@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
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
--
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