I increased test coverage from 68% to 74% of filecmp.py by adding test cases in test_filecmp.py in functions phase2 & phase4 in dircmp class.

Hi Alex.

Typo in  "#Speicify subdirectories to hide".

I think you should not create WIN32 constant. Use this statement instead: @unittest.skipIf(sys.platform == "win32", "Not valid on Windows")

I think you should use addCleanup if you want to remove the directories that you create in that specific test. For example, in test_phase2_check_diff_files, you can use:
self.addCleanup(os.rmdir, check_type_dir)

That is for now.

HI Vajrasky,

I fixed the typo in a comment and removed WIN32 constant definition from the code.

I looked at the existing tearDown method, on line 79, it calls shutil.rmtree on a directory above the files/directories I'm writing my test files, so the statements and the end of each function or combining these statements in a cleanup function is redundant, so I removed them.

Alex.

I've also increased the test coverage of filecmp.py. Don't know if I should merge my patch with Alex.Volkov's patch. I'm uploading it as a separate patch.