[squeak-dev] The Inbox: Chronology-Tests-dtl.15.mcz
David T. Lewis
lewis at mail.msen.com
Thu Jan 17 04:13:00 UTC 2019
The test is not intended to advocate for or against caching #getSeconds.
It provides coverage for expected behavior. So if you did decide to cache
#getSeconds, then you would still want this test to pass.
Dave
On Wed, Jan 16, 2019 at 08:55:32PM -0600, Chris Muller wrote:
> That is not a reason to not cache #getSeconds. Invalidation is a
> standard feature of any cache. For the case you mention, the app
> would simply invalidate its cache, after it had benefitted from
> billions and billions of accesses to it... :)
>
> On Sat, Jan 12, 2019 at 9:52 AM <commits at source.squeak.org> wrote:
> >
> > David T. Lewis uploaded a new version of Chronology-Tests to project The Inbox:
> > http://source.squeak.org/inbox/Chronology-Tests-dtl.15.mcz
> >
> > ==================== Summary ====================
> >
> > Name: Chronology-Tests-dtl.15
> > Author: dtl
> > Time: 12 January 2019, 10:52:06.856687 am
> > UUID: 93d70832-de59-4f2a-8516-b98262941370
> > Ancestors: Chronology-Tests-dtl.14
> >
> > Add testGetSeconds to verify that getSeconds represents whole seconds in the local time zone. For a given instance of DateAndTime, changing the time zone offset changes the local representation, but does not affect magnitude (time since a defiined epoch). Therefore, if time zone offset changes, the asSeconds value should not change, and the getSeconds value should change to reflect local timezone.
> >
> > =============== Diff against Chronology-Tests-dtl.14 ===============
> >
> > Item was added:
> > + ----- Method: DateAndTimeTest>>testGetSeconds (in category 'Tests') -----
> > + testGetSeconds
> > + "Verify that getSeconds represents whole seconds in the local time zone. For
> > + a given instance of DateAndTime, changing the time zone offset changes the
> > + local representation, but does not affect magnitude (time since a defiined
> > + epoch). Therefore, if time zone offset changes, the asSeconds value should
> > + not change, and the getSeconds value should change to reflect local timezone."
> > +
> > + | dt s1 id stSeconds seconds1 seconds2 |
> > + s1 := '2019-01-12T10:07:05.18743-05:00'.
> > + dt := s1 asDateAndTime.
> > + self assert: 18000 seconds negated equals: dt offset.
> > + seconds1 := dt getSeconds.
> > + self assert: 36425 equals: seconds1.
> > + id := dt identityHash.
> > + stSeconds := dt asSeconds.
> > + dt makeUTC. "make the receiver's timezone GMT, do not change magnitude"
> > + self assert: id equals: dt identityHash. "same object, not a copy"
> > + self assert: '2019-01-12T15:07:05.18743+00:00' equals: dt asString.
> > + self assert: stSeconds equals: dt asSeconds. "magnitude unchanged"
> > + self assert: '2019-01-12T10:07:05.18743-05:00' asDateAndTime equals: dt. "still equal"
> > + seconds2 := dt getSeconds.
> > + self deny: seconds1 equals: seconds2.
> > + self assert: 54425 equals: seconds2.
> > +
> > + !
> >
> >
>
More information about the Squeak-dev
mailing list
|