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

[Stephan Richter]

On Friday 06 December 2002 08:44, Viktorija Zaksiene wrote:
> 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.

Or better nuke it. :-) If I understood Jeremy's comment correctly, then the 
API changed and only second() survived as the true method.

Regards,
Stephan
-- 
Stephan Richter
CBU Physics & Chemistry (B.S.) / Tufts Physics (Ph.D. student)
Web2k - Web Software Design, Development and Training