[Zope3-checkins] CVS: Zope3/lib/python/ZODB/tests - testTimeStamp.py:1.8

Viktorija Zaksiene viktorija.zaksiene@codeworks.lt
Fri, 6 Dec 2002 15:44:21 +0200


On Friday 06 December 2002 15:30, you wrote:

> > -        self.assertEquals(t.second(), t.seconds()) # Alias
> > +        self.assertEquals(t.second(), t.second()) # Alias
>
> Is this right? The new code doesn't test much. The 'Alias' comment
> seems to indicate that this unit test is there to check whether
> the t.second and t.seconds methods both exist and implement the same. :)
> Should it change back?

It seems to me, that TimeStamp object doesn't have method seconds? So maybe 
we should just comment this test.

Regards,
Vika