469f71144e
This commit is two fold. For one side does a small refactor to move the deletion of custom RPM macros to package manager level inside a cleaning method. This way only RPM based package managers run RPM specific code and each package manager can apply its own specific logic. On the other hand for the zypper package manager the deletion of /var/lib/zypp/AnonymousUniqueId file has been added as part of the new cleaning method. Fixes #1396
160 lines
5.6 KiB
Python
160 lines
5.6 KiB
Python
from mock import patch
|
|
from pytest import raises
|
|
import mock
|
|
|
|
from kiwi.package_manager.dnf import PackageManagerDnf
|
|
|
|
from kiwi.exceptions import KiwiRequestError
|
|
|
|
|
|
class TestPackageManagerDnf:
|
|
def setup(self):
|
|
repository = mock.Mock()
|
|
repository.root_dir = '/root-dir'
|
|
|
|
repository.runtime_config = mock.Mock(
|
|
return_value={
|
|
'dnf_args': ['-c', '/root-dir/dnf.conf', '-y'],
|
|
'command_env': ['env']
|
|
}
|
|
)
|
|
self.manager = PackageManagerDnf(repository)
|
|
|
|
def test_request_package(self):
|
|
self.manager.request_package('name')
|
|
assert self.manager.package_requests == ['name']
|
|
|
|
def test_request_collection(self):
|
|
self.manager.request_collection('name')
|
|
assert self.manager.collection_requests == ['"name"']
|
|
|
|
def test_request_product(self):
|
|
self.manager.request_product('name')
|
|
assert self.manager.product_requests == []
|
|
|
|
def test_request_package_exclusion(self):
|
|
self.manager.request_package_exclusion('name')
|
|
assert self.manager.exclude_requests == ['name']
|
|
|
|
@patch('kiwi.command.Command.call')
|
|
@patch('kiwi.command.Command.run')
|
|
def test_process_install_requests_bootstrap(self, mock_run, mock_call):
|
|
self.manager.request_package('vim')
|
|
self.manager.request_collection('collection')
|
|
self.manager.process_install_requests_bootstrap()
|
|
mock_run.assert_called_once_with(
|
|
['dnf', '-c', '/root-dir/dnf.conf', '-y', 'makecache']
|
|
)
|
|
mock_call.assert_called_once_with(
|
|
[
|
|
'bash', '-c',
|
|
'dnf -c /root-dir/dnf.conf -y '
|
|
'--installroot /root-dir install vim && '
|
|
'dnf -c /root-dir/dnf.conf -y '
|
|
'--installroot /root-dir group install '
|
|
'"collection"'
|
|
], ['env']
|
|
)
|
|
|
|
@patch('kiwi.command.Command.call')
|
|
def test_process_install_requests(self, mock_call):
|
|
self.manager.request_package('vim')
|
|
self.manager.request_collection('collection')
|
|
self.manager.request_package_exclusion('skipme')
|
|
self.manager.process_install_requests()
|
|
mock_call.assert_called_once_with(
|
|
[
|
|
'bash', '-c',
|
|
'chroot /root-dir dnf -c /dnf.conf -y '
|
|
'--exclude=skipme install vim && '
|
|
'chroot /root-dir dnf -c /dnf.conf -y '
|
|
'--exclude=skipme group install '
|
|
'"collection"'
|
|
], ['env']
|
|
)
|
|
|
|
@patch('kiwi.command.Command.call')
|
|
@patch('kiwi.command.Command.run')
|
|
def test_process_delete_requests_force(self, mock_run, mock_call):
|
|
self.manager.request_package('vim')
|
|
self.manager.process_delete_requests(True)
|
|
mock_call.assert_called_once_with(
|
|
[
|
|
'chroot', '/root-dir', 'rpm', '-e',
|
|
'--nodeps', '--allmatches', '--noscripts', 'vim'
|
|
],
|
|
[
|
|
'env'
|
|
]
|
|
)
|
|
|
|
@patch('kiwi.command.Command.call')
|
|
@patch('kiwi.command.Command.run')
|
|
def test_process_delete_requests_no_force(self, mock_run, mock_call):
|
|
self.manager.request_package('vim')
|
|
self.manager.process_delete_requests()
|
|
mock_call.assert_called_once_with(
|
|
[
|
|
'chroot', '/root-dir', 'dnf',
|
|
'-c', '/dnf.conf', '-y', 'autoremove', 'vim'
|
|
],
|
|
['env']
|
|
)
|
|
|
|
@patch('kiwi.command.Command.run')
|
|
@patch('kiwi.command.Command.call')
|
|
def test_process_delete_requests_package_missing(
|
|
self, mock_call, mock_run
|
|
):
|
|
mock_run.side_effect = Exception
|
|
self.manager.request_package('vim')
|
|
with raises(KiwiRequestError):
|
|
self.manager.process_delete_requests()
|
|
mock_run.assert_called_once_with(
|
|
['chroot', '/root-dir', 'rpm', '-q', 'vim']
|
|
)
|
|
|
|
@patch('kiwi.command.Command.call')
|
|
def test_update(self, mock_call):
|
|
self.manager.update()
|
|
mock_call.assert_called_once_with(
|
|
[
|
|
'chroot', '/root-dir', 'dnf',
|
|
'-c', '/dnf.conf', '-y', 'upgrade'
|
|
], ['env']
|
|
)
|
|
|
|
def test_process_only_required(self):
|
|
self.manager.process_only_required()
|
|
assert self.manager.custom_args == ['--setopt=install_weak_deps=False']
|
|
|
|
def test_process_plus_recommended(self):
|
|
self.manager.process_only_required()
|
|
assert self.manager.custom_args == ['--setopt=install_weak_deps=False']
|
|
self.manager.process_plus_recommended()
|
|
assert \
|
|
'--setopt=install_weak_deps=False' not in self.manager.custom_args
|
|
|
|
def test_match_package_installed(self):
|
|
assert self.manager.match_package_installed('foo', 'Installing : foo')
|
|
|
|
def test_match_package_deleted(self):
|
|
assert self.manager.match_package_deleted('foo', 'Removing: foo')
|
|
|
|
@patch('kiwi.package_manager.dnf.RpmDataBase')
|
|
def test_post_process_install_requests_bootstrap(self, mock_RpmDataBase):
|
|
rpmdb = mock.Mock()
|
|
rpmdb.has_rpm.return_value = True
|
|
mock_RpmDataBase.return_value = rpmdb
|
|
self.manager.post_process_install_requests_bootstrap()
|
|
rpmdb.set_database_to_image_path.assert_called_once_with()
|
|
|
|
@patch('kiwi.package_manager.dnf.Rpm')
|
|
def test_clean_leftovers(self, mock_rpm):
|
|
mock_rpm.return_value = mock.Mock()
|
|
self.manager.clean_leftovers()
|
|
mock_rpm.assert_called_once_with(
|
|
'/root-dir', 'macros.kiwi-image-config'
|
|
)
|
|
mock_rpm.return_value.wipe_config.assert_called_once_with()
|