This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author cbc
Recipients cbc, chris.jerdonek, eli.bendersky
Date 2012-07-31.18:06:55
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1343758018.09.0.603602674726.issue15454@psf.upfronthosting.co.za>
In-reply-to
Content
At the PyOhio sprints, I noticed this issue and found it interesting for low hanging fruit. First I went to make tests for this enhancement, only to find that report(), report_partial_closure(), and report_full_closure did not already have tests.

In adding tests for these methods, I also found that dircmp is only tested for one directory level, leaving out coverage for a pretty wide swath of dircmp.

The first patch I'm submitting here is just for the report methods for the level of dircmp testing which already exists. A subsequent patch will test the report methods a couple of directory levels deep to get full coverage for report_full_closure().

Even if this enhancement is not accepted, the report methods do need tests. I'm putting this patch here in advance of any other work to get feedback on the flavor of tests I have given the report methods on the existing one directory level of dircmp testing before proceeding with any deeper testing.

The tests in this patch all pass with the existing filecmp code without adding any significant execution time to test_filecmp.

Any deeper testing  of dircmp will also have to not be necessarily comprehensive, as the permutations of test inputs start to become too time-consuming. setUp and tearDown may also need some refactoring for deeper testing.

Should adding tests for the report methods be a separate issue from this enhancement?
History
Date User Action Args
2012-07-31 18:06:58cbcsetrecipients: + cbc, eli.bendersky, chris.jerdonek
2012-07-31 18:06:58cbcsetmessageid: <1343758018.09.0.603602674726.issue15454@psf.upfronthosting.co.za>
2012-07-31 18:06:57cbclinkissue15454 messages
2012-07-31 18:06:56cbccreate