Commitc6e0a6c

authored and

committed

Avoid from_timestamp() function to raise an exception when the offset is greater or lower than 24 hours.

Add tests that exercise the new behaviour

1 parent0857d33 commitc6e0a6cCopy full SHA for c6e0a6c

File tree

+21

-4

lines changed

+21

-4

lines changed

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -121,8 +121,11 @@ def dst(self, dt):`
`121`	`121`	`deffrom_timestamp(timestamp,tz_offset):`
`122`	`122`	`"""Converts a timestamp + tz_offset into an aware datetime instance."""`
`123`	`123`	`utc_dt=datetime.fromtimestamp(timestamp,utc)`
`124`		`-local_dt=utc_dt.astimezone(tzoffset(tz_offset))`
`125`		`-returnlocal_dt`
	`124`	`+try:`
	`125`	`+local_dt=utc_dt.astimezone(tzoffset(tz_offset))`
	`126`	`+returnlocal_dt`
	`127`	`+exceptValueError:`
	`128`	`+returnutc_dt`
`126`	`129`
`127`	`130`
`128`	`131`	`defparse_date(string_date):`

Lines changed: 16 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`importtempfile`
`8`	`8`	`importtime`
`9`	`9`	`fromunittestimportskipIf`
`10`		`-`
	`10`	`+fromdatetimeimportdatetime`
`11`	`11`
`12`	`12`	`importddt`
`13`	`13`
`@@ -18,7 +18,8 @@`
`18`	`18`	`utctz_to_altz,`
`19`	`19`	`verify_utctz,`
`20`	`20`	`parse_date,`
`21`		`-)`
	`21`	`+tzoffset,`
	`22`	`+from_timestamp)`
`22`	`23`	`fromgit.test.libimport (`
`23`	`24`	`TestBase,`
`24`	`25`	`assert_equal`
`@@ -260,3 +261,16 @@ def test_iterable_list(self, case):`
`260`	`261`
`261`	`262`	`self.failUnlessRaises(IndexError,ilist.__delitem__,0)`
`262`	`263`	`self.failUnlessRaises(IndexError,ilist.__delitem__,'something')`
	`264`	`+`
	`265`	`+deftest_from_timestamp(self):`
	`266`	`+# Correct offset: UTC+2, should return datetime + tzoffset(+2)`
	`267`	`+altz=utctz_to_altz('+0200')`
	`268`	`+self.assertEqual(datetime.fromtimestamp(1522827734,tzoffset(altz)),from_timestamp(1522827734,altz))`
	`269`	`+`
	`270`	`+# Wrong offset: UTC+58, should return datetime + tzoffset(UTC)`
	`271`	`+altz=utctz_to_altz('+5800')`
	`272`	`+self.assertEqual(datetime.fromtimestamp(1522827734,tzoffset(0)),from_timestamp(1522827734,altz))`
	`273`	`+`
	`274`	`+# Wrong offset: UTC-9000, should return datetime + tzoffset(UTC)`
	`275`	`+altz=utctz_to_altz('-9000')`
	`276`	`+self.assertEqual(datetime.fromtimestamp(1522827734,tzoffset(0)),from_timestamp(1522827734,altz))`

Comments

(0)