mirror of
https://github.com/clearlinux/autospec.git
synced 2026-09-05 13:21:35 +00:00
Matthew Johnson
7c4e0e9fd6
Use a regular expression in order to exclude all directives at the beginning of the filename, including %doc. Signed-off-by: Matthew Johnson <matthew.johnson@intel.com>
255 lines
9.2 KiB
Python
255 lines
9.2 KiB
Python
import unittest
|
|
import files
|
|
import tempfile
|
|
import os
|
|
from unittest.mock import call, MagicMock
|
|
from files import FileManager
|
|
|
|
|
|
def mock_return(retval):
|
|
"""
|
|
Simple mock method to set return value of a function
|
|
"""
|
|
def mock_fn(_):
|
|
return retval
|
|
|
|
return mock_fn
|
|
|
|
|
|
class TestFiles(unittest.TestCase):
|
|
|
|
def setUp(self):
|
|
self.fm = FileManager()
|
|
|
|
def test_push_package_file(self):
|
|
"""
|
|
Test push_package_file with no package name specified (package name
|
|
should default to 'main'
|
|
"""
|
|
self.assertFalse(self.fm.newfiles_printed)
|
|
self.fm.push_package_file('test-fn')
|
|
self.assertEqual(self.fm.packages['main'], set(['test-fn']))
|
|
self.assertTrue(self.fm.newfiles_printed)
|
|
|
|
def test_push_package_file_dev(self):
|
|
"""
|
|
Test push_package_file with dev package specified
|
|
"""
|
|
self.fm.push_package_file('test-fn', 'dev')
|
|
self.assertEqual(self.fm.packages['dev'], set(['test-fn']))
|
|
self.assertTrue(self.fm.newfiles_printed)
|
|
|
|
def test_file_pat_match(self):
|
|
"""
|
|
Test file_pat_match with good match and no replacement or prefix
|
|
specified.
|
|
"""
|
|
self.fm.push_package_file = MagicMock()
|
|
self.assertTrue(self.fm.file_pat_match('test-fn', r'test-fn', 'main'))
|
|
self.fm.push_package_file.assert_called_with('test-fn', 'main')
|
|
|
|
def test_file_pat_match_exclude(self):
|
|
"""
|
|
Test file_pat_match with good match and filename in excludes list.
|
|
"""
|
|
self.fm.push_package_file = MagicMock()
|
|
self.fm.excludes.append('test-fn')
|
|
self.assertTrue(self.fm.file_pat_match('test-fn', r'test-fn', 'main'))
|
|
self.fm.push_package_file.assert_called_with('%exclude test-fn', 'main')
|
|
|
|
def test_file_pat_match_replacement(self):
|
|
"""
|
|
Test file_pat_match with replacement provided
|
|
"""
|
|
self.fm.push_package_file = MagicMock()
|
|
self.assertTrue(self.fm.file_pat_match('test-fn', r'test-fn', 'main', 'testfn'))
|
|
self.fm.push_package_file.assert_called_with('testfn', 'main')
|
|
|
|
def test_file_pat_match_no_match(self):
|
|
"""
|
|
Test file_pat_match with no match
|
|
"""
|
|
self.fm.push_package_file = MagicMock()
|
|
self.assertFalse(self.fm.file_pat_match('test-fn', r'testfn', 'main'))
|
|
self.fm.push_package_file.assert_not_called()
|
|
|
|
def test_file_is_locale(self):
|
|
"""
|
|
Test file_is_locale with locale filename not present in locale list
|
|
"""
|
|
self.assertEqual(self.fm.locales, [])
|
|
self.assertTrue(self.fm.file_is_locale('/usr/share/locale/a/loc.mo'))
|
|
self.assertEqual(self.fm.locales, ['loc'])
|
|
|
|
def test_file_is_locale_non_locale(self):
|
|
"""
|
|
Test file_is_locale with non-locale filename
|
|
"""
|
|
self.assertFalse(self.fm.file_is_locale('test-fn'))
|
|
self.assertEqual(self.fm.locales, [])
|
|
|
|
def test_file_is_locale_present(self):
|
|
"""
|
|
Test file_is_locale with locale present in locale list
|
|
"""
|
|
self.fm.locales.append('loc')
|
|
self.assertEqual(self.fm.locales, ['loc'])
|
|
self.assertTrue(self.fm.file_is_locale('/usr/share/locale/a/loc.mo'))
|
|
self.assertEqual(self.fm.locales, ['loc'])
|
|
|
|
def test_push_file_autostart(self):
|
|
"""
|
|
Test push_file to autostart package, this excludes the file.
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
autostart = '/usr/lib/systemd/system/some.target.wants/some'
|
|
self.fm.push_file(autostart)
|
|
calls = [call(autostart, 'autostart'), call('%exclude ' + autostart, 'config')]
|
|
self.fm.push_package_file.assert_has_calls(calls)
|
|
|
|
def test_push_file_extras(self):
|
|
"""
|
|
Test push_file to extras package, this excludes the file
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
self.fm.extras.append('test')
|
|
self.fm.push_file('test')
|
|
calls = [call('test', 'extras'), call('%exclude test')]
|
|
self.fm.push_package_file.assert_has_calls(calls)
|
|
|
|
|
|
def test_push_file_setuid(self):
|
|
"""
|
|
Test push_file with fname in setuid list
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
self.fm.setuid.append('test')
|
|
self.fm.push_file('test')
|
|
calls = [call('%attr(4755, root, root) test', 'setuid'), call('%exclude test')]
|
|
self.fm.push_package_file.assert_has_calls(calls)
|
|
|
|
|
|
def test_push_file_match(self):
|
|
"""
|
|
Test push_file with match in pattern list
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
self.fm.push_file('/usr/bin/test')
|
|
self.fm.push_package_file.assert_called_once_with('/usr/bin/test', 'bin')
|
|
|
|
def test_push_file_match_tarball_name_dependency(self):
|
|
"""
|
|
Test push_file with match in the list on the single item that is
|
|
dependent on the tarball name.
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
files.tarball.name = 'testball'
|
|
self.fm.push_file('/usr/share/doc/testball/')
|
|
self.fm.push_package_file.assert_called_once_with('%doc /usr/share/doc/testball/*', 'doc')
|
|
|
|
def test_push_file_no_match(self):
|
|
"""
|
|
Test push_file with no pattern match on the file name. Should just push
|
|
the unmodified filename once.
|
|
"""
|
|
self.fm.file_is_locale = MagicMock(return_value=False)
|
|
self.fm.push_package_file = MagicMock()
|
|
self.fm.push_file('doesntmatcha thing')
|
|
self.fm.push_package_file.assert_called_once_with('doesntmatcha thing')
|
|
|
|
def test_remove_file(self):
|
|
"""
|
|
Test remove_file with filename in files list and main package
|
|
"""
|
|
self.fm.files.append('test')
|
|
self.fm.packages['main'] = ['test']
|
|
self.assertIn('test', self.fm.files)
|
|
self.assertNotIn('test', self.fm.files_blacklist)
|
|
self.assertIn('test', self.fm.packages['main'])
|
|
self.fm.remove_file('test')
|
|
self.assertNotIn('test', self.fm.files)
|
|
self.assertNotIn('test', self.fm.packages['main'])
|
|
self.assertIn('test', self.fm.files_blacklist)
|
|
|
|
def test_remove_file_not_present(self):
|
|
"""
|
|
Test remove_file with filename not in files list.
|
|
"""
|
|
self.assertNotIn('test', self.fm.files)
|
|
self.assertNotIn('test', self.fm.files_blacklist)
|
|
self.fm.remove_file('test')
|
|
self.assertNotIn('test', self.fm.files)
|
|
self.assertNotIn('test', self.fm.files_blacklist)
|
|
|
|
def test_clean_directories(self):
|
|
"""
|
|
Test clean_directories with a directory in the list
|
|
"""
|
|
with tempfile.TemporaryDirectory() as tmpd:
|
|
os.mkdir(os.path.join(tmpd, "directory"))
|
|
with open(os.path.join(tmpd, "file1"), "w") as f:
|
|
f.write(" ")
|
|
|
|
with open(os.path.join(tmpd, "file2"), "w") as f:
|
|
f.write(" ")
|
|
|
|
self.fm.packages["main"] = set()
|
|
self.fm.packages["main"].add("/directory")
|
|
self.fm.packages["main"].add("/file1")
|
|
self.fm.packages["main"].add("/file2")
|
|
self.fm.clean_directories(tmpd)
|
|
self.assertEqual(self.fm.packages["main"], set(["/file1", "/file2"]))
|
|
|
|
|
|
def test_clean_directories_with_dir(self):
|
|
"""
|
|
Test clean_directories with a %dir directory in the list. This should
|
|
remain.
|
|
"""
|
|
with tempfile.TemporaryDirectory() as tmpd:
|
|
os.mkdir(os.path.join(tmpd, "directory"))
|
|
with open(os.path.join(tmpd, "file1"), "w") as f:
|
|
f.write(" ")
|
|
|
|
with open(os.path.join(tmpd, "file2"), "w") as f:
|
|
f.write(" ")
|
|
|
|
self.fm.packages["main"] = set()
|
|
self.fm.packages["main"].add("%dir /directory")
|
|
self.fm.packages["main"].add("/file1")
|
|
self.fm.packages["main"].add("/file2")
|
|
self.fm.clean_directories(tmpd)
|
|
self.assertEqual(self.fm.packages["main"],
|
|
set(["%dir /directory", "/file1", "/file2"]))
|
|
|
|
|
|
def test_clean_directories_with_doc(self):
|
|
"""
|
|
Test clean_directories with a %doc directive in the list. This should
|
|
remain.
|
|
"""
|
|
with tempfile.TemporaryDirectory() as tmpd:
|
|
os.mkdir(os.path.join(tmpd, "directory"))
|
|
with open(os.path.join(tmpd, "file1"), "w") as f:
|
|
f.write(" ")
|
|
|
|
with open(os.path.join(tmpd, "file2"), "w") as f:
|
|
f.write(" ")
|
|
|
|
self.fm.packages["main"] = set()
|
|
self.fm.packages["main"].add("%doc /directory")
|
|
self.fm.packages["main"].add("/file1")
|
|
self.fm.packages["main"].add("/file2")
|
|
self.fm.clean_directories(tmpd)
|
|
self.assertEqual(self.fm.packages["main"],
|
|
set(["%doc /directory", "/file1", "/file2"]))
|
|
|
|
|
|
if __name__ == '__main__':
|
|
unittest.main(buffer=True)
|