[Patches] [ python-Patches-440170 ] Tests for fileinput module
noreply@sourceforge.net
noreply@sourceforge.net
Wed, 11 Jul 2001 15:15:23 -0700
Patches item #440170, was opened at 2001-07-10 12:29
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=440170&group_id=5470
Category: library
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Nick Mathewson (nickm)
Assigned to: Tim Peters (tim_one)
Summary: Tests for fileinput module
Initial Comment:
Inspired by a Post by Tim to c.l.py this morning, I've
decided to try my hand at writing real test cases for
some of the modules formerly tested only by
test_sundry.py. This is my second attempt: a set of
tests for the fileinput.py module.
----------------------------------------------------------------------
>Comment By: Nick Mathewson (nickm)
Date: 2001-07-11 15:15
Message:
Logged In: YES
user_id=499
Tim says that tempfile is bad in unit tests, and TESTFN is
good. I've uploaded a new version of test_fileinput.py that
uses TESTFN.
----------------------------------------------------------------------
Comment By: Nick Mathewson (nickm)
Date: 2001-07-10 22:09
Message:
Logged In: YES
user_id=499
I've updated the tests for fileinput.py to do the right
thing, and obey Guido's style guidelines. I don't know
whether Sourceforge will let me replace the old ones,
though. :/
----------------------------------------------------------------------
Comment By: Tim Peters (tim_one)
Date: 2001-07-10 21:14
Message:
Logged In: YES
user_id=31435
As with the uu test, please change this too not to need an
expected-output file (make the print stmts conditional,
under "if test_support.verbose:").
Note that Guido's style guide requires a single space after
commas (in argument lists and tuples specifically). So e.g.
def runTests(t1, t2, t3, t4, bs=0, round=0):
not
def runTests(t1,t2,t3,t4,bs=0,round=0):
----------------------------------------------------------------------
You can respond by visiting:
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=440170&group_id=5470