harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Zhang" <zhanghuang...@gmail.com>
Subject Re: [classlib] [tests] Can anyone explain what these are for?
Date Thu, 12 Oct 2006 01:04:44 GMT
On 10/11/06, Alexei Zakharov <alexei.zakharov@gmail.com> wrote:
>
> Some tests for beans contained the code like this:
>
> <---
> public void testGetArguments() {
>    // Covered in the testcases for the constructor
> }
>
> public void testGetMethodName() {
>    // Covered in the testcases for the constructor
> }
> <---
>
> I just commented it out. I could simply delete it but decided that we
> may need it in the future. When we finish with 1.5 and everybody will
> have a lot of free time - we can get back to it and think again: do we
> need to insert an additional test case here?


Yes. My suggestion is marking these tests with "TODO". It's easy for
remindering. Eventually, we'll decide whether removing the TODOs or writing
more test code there.

Thanks!



2006/10/11, Richard Liang <richard.liangyx@gmail.com>:
> On 10/11/06, Nathan Beyer <nbeyer@gmail.com> wrote:
> > Perhaps, but there are much better ways of determining that. From just
> > reading the test code to code coverage tools.
> >
> > From my analysis, these were part of the large test contribution and
> > indicated that there wasn't an explicit test for that method. In most
> > cases, there were tests for these methods, either in other classes or
> > in other methods of the class.
>
> Nathan, I agree ;-)
>
> >
> > -Nathan
> >
> > On 10/10/06, Richard Liang <richard.liangyx@gmail.com> wrote:
> > > On 10/11/06, Nathan Beyer <nbeyer@gmail.com> wrote:
> > > > I've seen a few and I've deleted any that I've come across. I would
> > > > say feel free to delete them too. I've also been deleting empty
setup
> > > > and teardown methods too.
> > >
> > > Please do not simply delete them. Maybe that means we are lack of the
> > > test for some methods, for example "getInetAddress".
> > >
> > > >
> > > > -Nathan
> > > >
> > > > On 10/10/06, Alexey Petrenko <alexey.a.petrenko@gmail.com> wrote:
> > > > > These could be a result of copy/paste tests creation.
> > > > > And I'm curious why it was written for the first time. :)
> > > > >
> > > > > SY, Alexey
> > > > >
> > > > > 2006/10/10, Mark Hindess <mark.hindess@googlemail.com>:
> > > > > >
> > > > > > I've come across a couple of tests with things like:
> > > > > >
> > > > > >         public void test_getInetAddress() {
> > > > > >                 assertTrue("Used to test", true);
> > > > > >         }
> > > > > >
> > > > > > Can anyone explain why we have these?

--
Alexei Zakharov,
Intel Middleware Product Division

---------------------------------------------------------------------
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





-- 
Best regards,
Andrew Zhang

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