2016-03-10 09:12:23 +00:00
|
|
|
#!/usr/bin/env python2
|
|
|
|
# -*- coding: utf-8 -*-
|
|
|
|
|
|
|
|
import mock
|
2016-05-25 11:39:02 +00:00
|
|
|
try:
|
|
|
|
import unittest2 as unittest
|
|
|
|
except ImportError:
|
|
|
|
import unittest
|
2016-03-10 09:12:23 +00:00
|
|
|
import os
|
|
|
|
import sys
|
|
|
|
import StringIO
|
|
|
|
|
|
|
|
sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..'))
|
|
|
|
|
|
|
|
from pungi import checks
|
|
|
|
|
|
|
|
|
|
|
|
class CheckDependenciesTestCase(unittest.TestCase):
|
2016-03-10 09:44:21 +00:00
|
|
|
|
|
|
|
def dont_find(self, paths):
|
|
|
|
return lambda path: path not in paths
|
|
|
|
|
2016-03-10 09:12:23 +00:00
|
|
|
def test_all_deps_missing(self):
|
|
|
|
def custom_exists(path):
|
|
|
|
return False
|
|
|
|
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = custom_exists
|
|
|
|
result = checks.check({})
|
|
|
|
|
2016-03-10 09:44:21 +00:00
|
|
|
self.assertGreater(len(out.getvalue().strip().split('\n')), 1)
|
2016-03-10 09:12:23 +00:00
|
|
|
self.assertFalse(result)
|
|
|
|
|
|
|
|
def test_all_deps_ok(self):
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
2016-03-10 17:54:14 +00:00
|
|
|
with mock.patch('platform.machine') as machine:
|
|
|
|
machine.return_value = 'x86_64'
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find([])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check({})
|
2016-03-10 09:12:23 +00:00
|
|
|
|
|
|
|
self.assertEqual('', out.getvalue())
|
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_does_not_require_jigdo_if_not_configured(self):
|
|
|
|
conf = {
|
|
|
|
'create_jigdo': False
|
|
|
|
}
|
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
2016-03-10 17:54:14 +00:00
|
|
|
with mock.patch('platform.machine') as machine:
|
|
|
|
machine.return_value = 'x86_64'
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/jigdo-lite'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 09:44:21 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
self.assertEqual('', out.getvalue())
|
2016-03-10 09:44:21 +00:00
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_isohybrid_not_required_without_productimg_phase(self):
|
|
|
|
conf = {
|
|
|
|
'bootable': True,
|
|
|
|
'productimg': False,
|
|
|
|
'runroot': True,
|
|
|
|
}
|
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 09:44:21 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
self.assertEqual('', out.getvalue())
|
2016-03-10 09:44:21 +00:00
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_isohybrid_not_required_on_not_bootable(self):
|
|
|
|
conf = {
|
|
|
|
'bootable': False,
|
|
|
|
'runroot': True,
|
|
|
|
}
|
2016-03-10 09:12:23 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 09:12:23 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
self.assertEqual('', out.getvalue())
|
2016-03-10 09:12:23 +00:00
|
|
|
self.assertTrue(result)
|
|
|
|
|
2016-03-10 09:44:21 +00:00
|
|
|
def test_isohybrid_not_required_on_arm(self):
|
|
|
|
conf = {
|
|
|
|
'bootable': True,
|
|
|
|
'productimg': True,
|
|
|
|
'runroot': True,
|
|
|
|
}
|
|
|
|
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('platform.machine') as machine:
|
|
|
|
machine.return_value = 'armhfp'
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 09:44:21 +00:00
|
|
|
|
|
|
|
self.assertRegexpMatches(out.getvalue(), r'^Not checking.*Expect failures.*$')
|
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_isohybrid_not_needed_in_runroot(self):
|
|
|
|
conf = {
|
|
|
|
'runroot': True,
|
|
|
|
}
|
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/isohybrid'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 10:23:45 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
self.assertEqual('', out.getvalue())
|
2016-03-10 10:23:45 +00:00
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_genisoimg_not_needed_in_runroot(self):
|
|
|
|
conf = {
|
|
|
|
'runroot': True,
|
|
|
|
}
|
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/genisoimage'])
|
2016-10-25 13:25:56 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 10:23:45 +00:00
|
|
|
|
2016-03-10 11:47:49 +00:00
|
|
|
self.assertEqual('', out.getvalue())
|
2016-03-10 10:23:45 +00:00
|
|
|
self.assertTrue(result)
|
|
|
|
|
|
|
|
def test_genisoimg_needed_for_productimg(self):
|
|
|
|
conf = {
|
|
|
|
'runroot': True,
|
|
|
|
'productimg': True,
|
|
|
|
'bootable': True,
|
|
|
|
}
|
|
|
|
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
2016-03-10 09:44:21 +00:00
|
|
|
with mock.patch('os.path.exists') as exists:
|
2016-03-10 10:23:45 +00:00
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/genisoimage'])
|
2016-10-25 13:43:19 +00:00
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
2016-03-10 09:44:21 +00:00
|
|
|
|
2016-03-10 10:23:45 +00:00
|
|
|
self.assertIn('genisoimage', out.getvalue())
|
|
|
|
self.assertFalse(result)
|
|
|
|
|
2016-10-25 13:43:19 +00:00
|
|
|
def test_requires_modifyrepo(self):
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/modifyrepo'])
|
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check({})
|
|
|
|
|
|
|
|
self.assertIn('createrepo', out.getvalue())
|
|
|
|
self.assertFalse(result)
|
|
|
|
|
|
|
|
def test_requires_createrepo_c(self):
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/createrepo_c'])
|
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check({})
|
|
|
|
|
|
|
|
self.assertIn('createrepo_c', out.getvalue())
|
|
|
|
self.assertFalse(result)
|
|
|
|
|
|
|
|
def test_doesnt_require_createrepo_c_if_configured(self):
|
|
|
|
conf = {
|
|
|
|
'createrepo_c': False,
|
|
|
|
}
|
|
|
|
|
|
|
|
with mock.patch('sys.stdout', new_callable=StringIO.StringIO) as out:
|
|
|
|
with mock.patch('os.path.exists') as exists:
|
|
|
|
exists.side_effect = self.dont_find(['/usr/bin/createrepo_c'])
|
|
|
|
with mock.patch('__builtin__.__import__'):
|
|
|
|
result = checks.check(conf)
|
|
|
|
|
|
|
|
self.assertNotIn('createrepo_c', out.getvalue())
|
|
|
|
self.assertTrue(result)
|
|
|
|
|
2016-03-10 09:12:23 +00:00
|
|
|
|
2016-03-31 07:45:50 +00:00
|
|
|
class TestUmask(unittest.TestCase):
|
|
|
|
def setUp(self):
|
|
|
|
self.orig_umask = os.umask(0)
|
|
|
|
os.umask(self.orig_umask)
|
|
|
|
|
|
|
|
def tearDown(self):
|
|
|
|
os.umask(self.orig_umask)
|
|
|
|
|
|
|
|
def test_no_warning_with_0022(self):
|
|
|
|
os.umask(0o022)
|
|
|
|
logger = mock.Mock()
|
|
|
|
checks.check_umask(logger)
|
|
|
|
self.assertItemsEqual(logger.mock_calls, [])
|
|
|
|
|
|
|
|
def test_no_warning_with_0000(self):
|
|
|
|
os.umask(0o000)
|
|
|
|
logger = mock.Mock()
|
|
|
|
checks.check_umask(logger)
|
|
|
|
self.assertItemsEqual(logger.mock_calls, [])
|
|
|
|
|
|
|
|
def test_warning_with_0044(self):
|
|
|
|
os.umask(0o044)
|
|
|
|
logger = mock.Mock()
|
|
|
|
checks.check_umask(logger)
|
|
|
|
self.assertItemsEqual(
|
|
|
|
logger.mock_calls,
|
|
|
|
[mock.call.warning('Unusually strict umask detected (0%03o), '
|
|
|
|
'expect files with broken permissions.', 0o044)]
|
|
|
|
)
|
|
|
|
|
|
|
|
|
2016-03-10 09:12:23 +00:00
|
|
|
if __name__ == "__main__":
|
|
|
|
unittest.main()
|