204 lines
7.9 KiB
Python
204 lines
7.9 KiB
Python
import copy
|
|
from unittest import mock
|
|
import os
|
|
import unittest
|
|
|
|
import koji
|
|
from kojihub import kojihub
|
|
|
|
|
|
class TestRPMDiff(unittest.TestCase):
|
|
|
|
@mock.patch('koji.rpmdiff.Rpmdiff')
|
|
def test_rpmdiff_empty_invocation(self, Rpmdiff):
|
|
kojihub.rpmdiff('basepath', [], hashes={})
|
|
Rpmdiff.assert_not_called()
|
|
kojihub.rpmdiff('basepath', ['foo'], hashes={})
|
|
Rpmdiff.assert_not_called()
|
|
|
|
@mock.patch('koji.rpmdiff.Rpmdiff')
|
|
def test_rpmdiff_simple_success(self, Rpmdiff):
|
|
d = mock.MagicMock()
|
|
d.differs.return_value = False
|
|
Rpmdiff.return_value = d
|
|
self.assertFalse(kojihub.rpmdiff('basepath', ['12/1234/foo', '23/2345/bar'], hashes={}))
|
|
Rpmdiff.assert_called_once_with(
|
|
'basepath/12/1234/foo', 'basepath/23/2345/bar', ignore='S5TN')
|
|
|
|
@mock.patch('koji.rpmdiff.Rpmdiff')
|
|
def test_rpmdiff_simple_failure(self, Rpmdiff):
|
|
d = mock.MagicMock()
|
|
d.differs.return_value = True
|
|
Rpmdiff.return_value = d
|
|
with self.assertRaises(koji.BuildError):
|
|
kojihub.rpmdiff('basepath', ['12/1234/foo', '13/1345/bar'], hashes={})
|
|
Rpmdiff.assert_called_once_with(
|
|
'basepath/12/1234/foo', 'basepath/13/1345/bar', ignore='S5TN')
|
|
d.textdiff.assert_called_once_with()
|
|
|
|
def test_rpmdiff_real_target(self):
|
|
data_path = os.path.abspath("tests/test_hub/data/rpms")
|
|
|
|
# the only differences between rpm1 and rpm2 are 1) create time 2) file name
|
|
rpm1 = os.path.join(data_path, 'test-pkg-1.0.0-1.el7.noarch.rpm')
|
|
rpm2 = os.path.join(data_path, 'test-pkg-1.0.0-1.fc24.noarch.rpm')
|
|
|
|
diff_output = "..........T /usr/share/test-pkg/test-doc01.txt\n" + \
|
|
"..........T /usr/share/test-pkg/test-doc02.txt\n" + \
|
|
"..........T /usr/share/test-pkg/test-doc03.txt\n" + \
|
|
"..........T /usr/share/test-pkg/test-doc04.txt"
|
|
|
|
# case 1. no ignore option, timestamp is different
|
|
# perform twice check to verify issue: #994
|
|
for _ in range(0, 2):
|
|
d = koji.rpmdiff.Rpmdiff(rpm1, rpm2)
|
|
self.assertEqual(d.textdiff(), diff_output)
|
|
|
|
# case 2. ignore timestamp, two rpms should be the same
|
|
# perform twice check to verify issue: #994
|
|
for r in range(0, 2):
|
|
d = koji.rpmdiff.Rpmdiff(rpm1, rpm2, ignore='S5TN')
|
|
self.assertEqual(d.textdiff(), '')
|
|
|
|
def test_rpmdiff_size(self):
|
|
data_path = os.path.abspath("tests/test_hub/data/rpms")
|
|
|
|
# the only differences between rpm1 and rpm2 are 1) create time 2) file name
|
|
rpm1 = os.path.join(data_path, 'different_size_a.noarch.rpm')
|
|
rpm2 = os.path.join(data_path, 'different_size_b.noarch.rpm')
|
|
|
|
diff_output = "S.5.......T /bin/test"
|
|
|
|
# case 1. no ignore option, timestamp is different
|
|
# perform twice check to verify issue: #994
|
|
for _ in range(0, 2):
|
|
d = koji.rpmdiff.Rpmdiff(rpm1, rpm2)
|
|
self.assertEqual(d.textdiff(), diff_output)
|
|
|
|
# case 2. ignore timestamp, two rpms should be the same
|
|
# perform twice check to verify issue: #994
|
|
for r in range(0, 2):
|
|
d = koji.rpmdiff.Rpmdiff(rpm1, rpm2, ignore='S5TN')
|
|
self.assertEqual(d.textdiff(), '')
|
|
|
|
def test_rpmdiff_kojihash(self):
|
|
data_path = os.path.abspath("tests/test_hub/data/rpms")
|
|
|
|
# the only differences between rpm1 and rpm2 are 1) create time 2) file name
|
|
rpm1 = os.path.join(data_path, 'different_size_a.noarch.rpm')
|
|
rpm2 = os.path.join(data_path, 'different_size_b.noarch.rpm')
|
|
|
|
hash1 = 'ed0bae957653692a7d2ff9d90dbc3eaf08486994af315e11a9e80929092f6d0e'
|
|
hash2 = '8cdd40b738ac156af09f31445831f324cedacd028bf1dc3e12cc48b2137ed190'
|
|
for _ in range(2):
|
|
# double check that kojihash is deterministic
|
|
d = koji.rpmdiff.Rpmdiff(rpm1, rpm2)
|
|
self.assertEqual(d.kojihash(), hash1)
|
|
self.assertEqual(d.kojihash(new=True), hash2)
|
|
|
|
def test_rpmdiff_ignore_test(self):
|
|
data_path = os.path.abspath("tests/test_hub/data/rpms")
|
|
|
|
# a template file for parsing rpm header
|
|
rpm = os.path.join(data_path, 'test-pkg-1.0.0-1.el7.noarch.rpm')
|
|
|
|
# dummy file info
|
|
defattr = [19, 33188, 1531970408, 0, 0, 2, 1, -1, -1, 'root', 'root', '02d2c91b']
|
|
|
|
rpm_dict_old = {'a_file': defattr}
|
|
|
|
def check_diff_result(opt, idx, value, textdiff):
|
|
orig_init = koji.rpmdiff.Rpmdiff.__init__
|
|
|
|
def init_mock(*args, **kwargs):
|
|
# need to init rpm_dict every time
|
|
attr = defattr[:]
|
|
attr[idx] = value
|
|
rpm_dict_new = {'a_file': attr}
|
|
|
|
args[0]._Rpmdiff__getFilesDict = mock.MagicMock()
|
|
args[0]._Rpmdiff__getFilesDict.side_effect = [rpm_dict_old, rpm_dict_new]
|
|
orig_init(*args, **kwargs)
|
|
|
|
# compare with every option
|
|
with mock.patch('koji.rpmdiff.Rpmdiff.__init__', new=init_mock):
|
|
for token in 'SM5DNLVUGFT':
|
|
diff = koji.rpmdiff.Rpmdiff(rpm, rpm, ignore=token)
|
|
self.assertEqual(diff.textdiff(), textdiff if token not in opt else '')
|
|
|
|
# case 1 size diffrerent
|
|
check_diff_result('S', 0, 99, "S.......... a_file")
|
|
|
|
# case 2 mode different
|
|
check_diff_result('M', 1, 22188, ".M......... a_file")
|
|
|
|
# case 3 time different
|
|
check_diff_result('T', 2, 1531976666, "..........T a_file")
|
|
|
|
# case 4 flag different
|
|
check_diff_result('F', 3, 3, ".........F. a_file")
|
|
|
|
# case 5 device different
|
|
check_diff_result('D', 4, 4, "...D....... a_file")
|
|
|
|
# case 6 inode different
|
|
check_diff_result('N', 6, 6, "....N...... a_file")
|
|
|
|
# case 7 number of links different
|
|
check_diff_result('L', 7, 7, ".....L..... a_file")
|
|
|
|
# case 8 vflag different
|
|
check_diff_result('V', 8, 8, "......V.... a_file")
|
|
|
|
# case 9 user different
|
|
check_diff_result('U', 9, 'tester', ".......U... a_file")
|
|
|
|
# case 10 group different
|
|
check_diff_result('G', 10, 'tester', "........G.. a_file")
|
|
|
|
# case 11 checksum different
|
|
check_diff_result('5', 11, 'aabbccdd', "..5........ a_file")
|
|
|
|
|
|
class TestCheckNoarchRpms(unittest.TestCase):
|
|
@mock.patch('kojihub.kojihub.rpmdiff')
|
|
def test_check_noarch_rpms_empty_invocation(self, rpmdiff):
|
|
originals = ['foo', 'bar']
|
|
result = kojihub.check_noarch_rpms('basepath', copy.copy(originals))
|
|
self.assertEqual(result, originals)
|
|
|
|
@mock.patch('kojihub.kojihub.rpmdiff')
|
|
def test_check_noarch_rpms_simple_invocation(self, rpmdiff):
|
|
originals = ['12/1234/foo.noarch.rpm', '23/2345/foo.noarch.rpm']
|
|
result = kojihub.check_noarch_rpms('basepath', copy.copy(originals))
|
|
self.assertEqual(result, originals[0:1])
|
|
self.assertEqual(len(rpmdiff.mock_calls), 1)
|
|
|
|
@mock.patch('kojihub.kojihub.rpmdiff')
|
|
def test_check_noarch_rpms_with_duplicates(self, rpmdiff):
|
|
originals = [
|
|
'bar.noarch.rpm',
|
|
'bar.noarch.rpm',
|
|
'bar.noarch.rpm',
|
|
]
|
|
result = kojihub.check_noarch_rpms('basepath', copy.copy(originals))
|
|
self.assertEqual(result, ['bar.noarch.rpm'])
|
|
rpmdiff.assert_called_once_with('basepath', originals, hashes={})
|
|
|
|
@mock.patch('kojihub.kojihub.rpmdiff')
|
|
def test_check_noarch_rpms_with_mixed(self, rpmdiff):
|
|
originals = [
|
|
'foo.x86_64.rpm',
|
|
'bar.x86_64.rpm',
|
|
'bar.noarch.rpm',
|
|
'bar.noarch.rpm',
|
|
]
|
|
result = kojihub.check_noarch_rpms('basepath', copy.copy(originals))
|
|
self.assertEqual(result, [
|
|
'foo.x86_64.rpm', 'bar.x86_64.rpm', 'bar.noarch.rpm'
|
|
])
|
|
rpmdiff.assert_called_once_with(
|
|
'basepath',
|
|
['bar.noarch.rpm', 'bar.noarch.rpm'],
|
|
hashes={}
|
|
)
|