gryf/fs-uae-wrapper
1
0
Fork 0
mirror of https://github.com/gryf/fs-uae-wrapper.git synced 2025-12-22 22:28:03 +01:00
Code Issues Projects Releases Wiki Activity

Fix for typo in attribute name

Browse Source
This commit is contained in:
gryf
2017-01-08 11:59:43 +01:00
parent bd0e3ea56a
commit 6a026ecff1
2 changed files with 12 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
fs_uae_wrapper/file_archive.py
View File

@@ -17,15 +17,15 @@ class Archive(object):
def __init__(self): def __init__(self):
self.archiver = path.which(self.ARCH) self.archiver = path.which(self.ARCH)
self._compess = self.archiver self._compress = self.archiver
self._decompess = self.archiver self._decompress = self.archiver
def create(self, arch_name, files=None): def create(self, arch_name, files=None):
""" """
Create archive. Return True on success, False otherwise. Create archive. Return True on success, False otherwise.
""" """
files = files if files else ['.'] files = files if files else ['.']
result = subprocess.call([self._compess] + self.ADD + [arch_name] result = subprocess.call([self._compress] + self.ADD + [arch_name]
+ files) + files)
if result != 0: if result != 0:
sys.stderr.write("Unable to create archive `%s'\n" % arch_name) sys.stderr.write("Unable to create archive `%s'\n" % arch_name)
@@ -40,7 +40,7 @@ class Archive(object):
sys.stderr.write("Archive `%s' doesn't exists.\n" % arch_name) sys.stderr.write("Archive `%s' doesn't exists.\n" % arch_name)
return False return False
result = subprocess.call([self._decompess] + self.EXTRACT + result = subprocess.call([self._decompress] + self.EXTRACT +
[arch_name]) [arch_name])
if result != 0: if result != 0:
sys.stderr.write("Unable to extract archive `%s'\n" % arch_name) sys.stderr.write("Unable to extract archive `%s'\n" % arch_name)
@@ -55,7 +55,7 @@ class TarArchive(Archive):
def create(self, arch_name, files=None): def create(self, arch_name, files=None):
files = files if files else sorted(os.listdir('.')) files = files if files else sorted(os.listdir('.'))
result = subprocess.call([self._compess] + self.ADD + [arch_name] + result = subprocess.call([self._compress] + self.ADD + [arch_name] +
files) files)
if result != 0: if result != 0:
sys.stderr.write("Unable to create archive `%s'\n" % arch_name) sys.stderr.write("Unable to create archive `%s'\n" % arch_name)
@@ -86,7 +86,7 @@ class ZipArchive(Archive):
def __init__(self): def __init__(self):
super(ZipArchive, self).__init__() super(ZipArchive, self).__init__()
if self.archiver == 'zip': if self.archiver == 'zip':
self._decompess = path.which('unzip') self._decompress = path.which('unzip')
ZipArchive.ADD = ['-r'] ZipArchive.ADD = ['-r']
ZipArchive.EXTRACT = [] ZipArchive.EXTRACT = []
@@ -116,7 +116,7 @@ class RarArchive(Archive):
'supported by unrar.\n') 'supported by unrar.\n')
return False return False
result = subprocess.call([self._compess] + self.ADD + [arch_name] + result = subprocess.call([self._compress] + self.ADD + [arch_name] +
files) files)
if result != 0: if result != 0:
sys.stderr.write("Unable to create archive `%s'\n" % arch_name) sys.stderr.write("Unable to create archive `%s'\n" % arch_name)

11
tests/test_file_archive.py
View File

@@ -143,7 +143,6 @@ class TestArchive(TestCase):
self.assertFalse(arch.create('foo.tar')) self.assertFalse(arch.create('foo.tar'))
call.assert_called_once_with(['tar', 'cf', 'foo.tar', 'bar', 'foo']) call.assert_called_once_with(['tar', 'cf', 'foo.tar', 'bar', 'foo'])
@mock.patch('fs_uae_wrapper.path.which') @mock.patch('fs_uae_wrapper.path.which')
@mock.patch('subprocess.call') @mock.patch('subprocess.call')
def test_lha(self, call, which): def test_lha(self, call, which):
@@ -226,8 +225,8 @@ class TestArchive(TestCase):
which.side_effect = ['zip', 'unzip'] which.side_effect = ['zip', 'unzip']
arch = file_archive.ZipArchive() arch = file_archive.ZipArchive()
self.assertEqual(arch._compess, 'zip') self.assertEqual(arch._compress, 'zip')
self.assertEqual(arch._decompess, 'unzip') self.assertEqual(arch._decompress, 'unzip')
@mock.patch('fs_uae_wrapper.path.which') @mock.patch('fs_uae_wrapper.path.which')
@mock.patch('subprocess.call') @mock.patch('subprocess.call')
@@ -268,7 +267,7 @@ class TestArchive(TestCase):
call.reset_mock() call.reset_mock()
call.return_value = 0 call.return_value = 0
arch._compess = arch._decompess = arch.archiver = 'unrar' arch._compress = arch._decompress = arch.archiver = 'unrar'
self.assertFalse(arch.create('foo')) self.assertFalse(arch.create('foo'))
call.assert_not_called() call.assert_not_called()
@@ -308,11 +307,11 @@ class TestArchivers(TestCase):
archivers = file_archive.Archivers archivers = file_archive.Archivers
self.assertEqual(archivers.get_extension_by_name('tar'), '.tar') self.assertEqual(archivers.get_extension_by_name('tar'), '.tar')
self.assertEqual(archivers.get_extension_by_name('tgz'), '.tar.gz') self.assertEqual(archivers.get_extension_by_name('tgz'), '.tar.gz')
self.assertEqual(archivers.get_extension_by_name('tar.bz2'), '.tar.bz2') self.assertEqual(archivers.get_extension_by_name('tar.bz2'),
'.tar.bz2')
self.assertEqual(archivers.get_extension_by_name('tar.xz'), '.tar.xz') self.assertEqual(archivers.get_extension_by_name('tar.xz'), '.tar.xz')
self.assertEqual(archivers.get_extension_by_name('rar'), '.rar') self.assertEqual(archivers.get_extension_by_name('rar'), '.rar')
self.assertEqual(archivers.get_extension_by_name('7z'), '.7z') self.assertEqual(archivers.get_extension_by_name('7z'), '.7z')
self.assertEqual(archivers.get_extension_by_name('lha'), '.lha') self.assertEqual(archivers.get_extension_by_name('lha'), '.lha')
self.assertEqual(archivers.get_extension_by_name('lzx'), '.lzx') self.assertEqual(archivers.get_extension_by_name('lzx'), '.lzx')
self.assertIsNone(archivers.get_extension_by_name('ace')) self.assertIsNone(archivers.get_extension_by_name('ace'))