From 0ecf462b8e3ae90633a8c5875c15e9e63d896b03 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Mon, 7 Sep 2015 13:55:25 +0300 Subject: Explicitly test archive name in shutil.make_archive() tests to expose failure details in issue25018. --- Lib/test/test_shutil.py | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/Lib/test/test_shutil.py b/Lib/test/test_shutil.py index dcec65b..6143be6 100644 --- a/Lib/test/test_shutil.py +++ b/Lib/test/test_shutil.py @@ -385,10 +385,10 @@ class TestShutil(unittest.TestCase): base_name = os.path.join(tmpdir2, 'archive') # working with relative paths to avoid tar warnings - make_archive(splitdrive(base_name)[1], 'gztar', root_dir, '.') + tarball = make_archive(splitdrive(base_name)[1], 'gztar', root_dir, '.') # check if the compressed tarball was created - tarball = base_name + '.tar.gz' + self.assertEqual(tarball, base_name + '.tar.gz') self.assertTrue(os.path.isfile(tarball)) self.assertTrue(tarfile.is_tarfile(tarball)) with tarfile.open(tarball, 'r:gz') as tf: @@ -397,9 +397,8 @@ class TestShutil(unittest.TestCase): './sub', './sub/file3', './sub2']) # trying an uncompressed one - base_name = os.path.join(tmpdir2, 'archive') - make_archive(splitdrive(base_name)[1], 'tar', root_dir, '.') - tarball = base_name + '.tar' + tarball = make_archive(splitdrive(base_name)[1], 'tar', root_dir, '.') + self.assertEqual(tarball, base_name + '.tar') self.assertTrue(os.path.isfile(tarball)) self.assertTrue(tarfile.is_tarfile(tarball)) with tarfile.open(tarball, 'r') as tf: @@ -434,10 +433,10 @@ class TestShutil(unittest.TestCase): def test_tarfile_vs_tar(self): root_dir, base_dir = self._create_files() base_name = os.path.join(self.mkdtemp(), 'archive') - make_archive(base_name, 'gztar', root_dir, base_dir) + tarball = make_archive(base_name, 'gztar', root_dir, base_dir) # check if the compressed tarball was created - tarball = base_name + '.tar.gz' + self.assertEqual(tarball, base_name + '.tar.gz') self.assertTrue(os.path.isfile(tarball)) # now create another tarball using `tar` @@ -451,13 +450,14 @@ class TestShutil(unittest.TestCase): self.assertEqual(self._tarinfo(tarball), self._tarinfo(tarball2)) # trying an uncompressed one - make_archive(base_name, 'tar', root_dir, base_dir) - tarball = base_name + '.tar' + tarball = make_archive(base_name, 'tar', root_dir, base_dir) + self.assertEqual(tarball, base_name + '.tar') self.assertTrue(os.path.isfile(tarball)) # now for a dry_run - make_archive(base_name, 'tar', root_dir, base_dir, dry_run=True) - tarball = base_name + '.tar' + tarball = make_archive(base_name, 'tar', root_dir, base_dir, + dry_run=True) + self.assertEqual(tarball, base_name + '.tar') self.assertTrue(os.path.isfile(tarball)) @unittest.skipUnless(zlib, "Requires zlib") -- cgit v0.12