[Python-bugs-list] [ python-Bugs-540720 ] test_email fails on Mac in summer

noreply@sourceforge.net noreply@sourceforge.net
Mon, 22 Apr 2002 09:25:49 -0700


Bugs item #540720, was opened at 2002-04-07 17:55
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=540720&group_id=5470

Category: Python Library
Group: Platform-specific
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Jack Jansen (jackjansen)
Assigned to: Barry Warsaw (bwarsaw)
Summary: test_email fails on Mac in summer

Initial Comment:
Yes, I kid you not:-)

The test of the date/time  parser in test_email tries to cater for the fact that the Mac uses localtime (at least, before OSX), but it fails to cater for DST. So, test_email passes in winter, but fails in summer.

I think the whole idea of comparing based on a numeric value is not so good. Why not try a roundtrip, or compare against another date coder (time.asctime)?

----------------------------------------------------------------------

>Comment By: Barry Warsaw (bwarsaw)
Date: 2002-04-22 12:25

Message:
Logged In: YES 
user_id=12800

I believe the patch resolved this issue.

----------------------------------------------------------------------

Comment By: Barry Warsaw (bwarsaw)
Date: 2002-04-15 13:12

Message:
Logged In: YES 
user_id=12800

Actually, forget the patch.  I integrated email 2.0 into
Python 2.3's cvs and this included the proposed fix.  Take a
look at test_formatdate() and test_formatdate_localtime() in
Lib/test/test_email.py and let me know what you think.

----------------------------------------------------------------------

Comment By: Jack Jansen (jackjansen)
Date: 2002-04-09 17:49

Message:
Logged In: YES 
user_id=45365

Barry,
I can't apply the patch, it seems to be against a different version of test_email than what's in the repository (I tried both the trunk and the 22 maintainance branch).

----------------------------------------------------------------------

Comment By: Barry Warsaw (bwarsaw)
Date: 2002-04-08 02:15

Message:
Logged In: YES 
user_id=12800

What do you think of the attached patch?  Is this still a
useful test to include?

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=105470&aid=540720&group_id=5470