harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Ivanov" <ivavladi...@gmail.com>
Subject Re: [classlib] [testing] Coverage (was Re: 37% of total test execution time is spent in a single test)
Date Mon, 21 Aug 2006 04:23:41 GMT
Coverage for swing module was added.

 thanks, Vladimir


On 8/18/06, Tim Ellison <t.p.ellison@gmail.com> wrote:
>
> Thanks for updating the numbers Vladimir.  Overall I think they look
> quite respectable.
>
> There are a few places where it looks like we could use more test
> coverage.  It's a great way to contribute to Harmony (he said, hinting
> to the lurkers <g>).  Pick a favourite package, kick the wheels of
> Harmony using JUnit tests -- then contribute them for the greater good.
>
> Regards,
> Tim
>
> Vladimir Ivanov wrote:
> > The coverage information was updated on the wiki. Also, coverage for
> > accessibility, awt, instrument and sound modules was added. I have a
> > problem
> > to calculate coverage for swing module (test run hang up) so issue 564
> will
> > be updated later.
> >
> > thanks, Vladimir
> >
> >
> > On 6/20/06, Paulex Yang < paulex.yang@gmail.com> wrote:
> >>
> >> Vladimir Ivanov wrote:
> >> > Classes from exclude list are now specified as green.
> >> > I've update wiki (http://wiki.apache.org/harmony/Coverage_information
> )
> >> > and
> >> > coverage pages.
> >> >
> >> Great, thank you, Vladimir!
> >> > Thanks,
> >> >  Vladimir
> >> >
> >> >
> >> > On 6/16/06, Paulex Yang <paulex.yang@gmail.com> wrote:
> >> >>
> >> >> Vladimir
> >> >>
> >> >> Vladimir Ivanov wrote:
> >> >> >>
> >> >> >> The current reports don't provide code source linking. Are
you
> >> >> going to
> >> >>
> >> >> >> add it?
> >> >> >
> >> >> >
> >> >> > There were no information for 'security' and 'auth' modules, but,
> I
> >> >> have
> >> >> > updated the pages and now there is source code linking for all
> >> >> modules.
> >> >> >
> >> >> > One more issue to discuss: excluded classes present in the
> coverage
> >> >> table
> >> >> > now with 0 coverage. May be it is more convenient do not have
> these
> >> >> > classes
> >> >> > in coverage tables at all? In this case one won't wonder why the
> >> class
> >> >> > has 0
> >> >> > coverage - go to exclude list to look at the class and decide
> >> whether
> >> >> the
> >> >> > class is really untested or just excluded from coverage, instead,
> >> all
> >> >> > really
> >> >> > uncovered classes will be shown with 0 coverage, if a class is
> >> >> missed in
> >> >> > coverage table – it is in exclude list.
> >> >> +1 for current "0 coverage" is not convenient. But if we can remove
> >> them
> >> >> from the report, how about just mark them in another way? say, mark
> >> the
> >> >> excluded class with different background colors?
> >> >>
> >> >> At least people don't need to care about two documents for one
> >> package.
> >> >> > Thanks,
> >> >> >   Vladimir
> >> >> >
> >> >> >
> >> >> >
> >> >> >> Now I have 2 questions/ issues to discuss:
> >> >> >> > 1) preferable VM to calculate coverage (seems, the exclude
list
>
> >> >> is a
> >> >> >> > little
> >> >> >> > bit different for j9 and drlvm)
> >> >> >>
> >> >> >>
> >> >> >> If the only difference is the exclude list then I'd suggest
> >> using VM
> >> >> >> with
> >> >> >> the shortest one. :-)
> >> >> >>
> >> >> >> 2) preferable sorting mode for results (by name, by blocks
or by
> >> >> >> methods)
> >> >> >>
> >> >> >>
> >> >> >> IMHO, sorting by name looks more natural.
> >> >> >>
> >> >> >> Thanks,
> >> >> >> Stepan.
> >> >> >>
> >> >> >> Thanks,
> >> >> >> > Vladimir
> >> >> >> >
> >> >> >> > <SNIP>
> >> >> >> >
> >> >> >>
> >> >> >>
> >> >> >> ------------------------------------------------------
> >> >> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> >> >> To unsubscribe, e-mail:
> >> harmony-dev-unsubscribe@incubator.apache.org
> >> >> >> For additional commands, e-mail:
> >> >> harmony-dev-help@incubator.apache.org
> >> >> >>
> >> >> >>
> >> >> >
> >> >>
> >> >>
> >> >> --
> >> >> Paulex Yang
> >> >> China Software Development Lab
> >> >> IBM
> >> >>
> >> >>
> >> >>
> >> >>
> ---------------------------------------------------------------------
> >> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> >> For additional commands, e-mail:
> harmony-dev-help@incubator.apache.org
> >> >>
> >> >>
> >> >
> >>
> >>
> >> --
> >> Paulex Yang
> >> China Software Development Lab
> >> IBM
> >>
> >>
> >>
> >> ---------------------------------------------------------------------
> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >>
> >>
> >
>
> --
>
> Tim Ellison (t.p.ellison@gmail.com )
> IBM Java technology centre, UK.
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message