PR#3688: fix missing clause

Merges #3688
https://pagure.io/koji/pull-request/3688

Relates: #3627
https://pagure.io/koji/issue/3627
checksum API for generate and store signed rpms checksums
This commit is contained in:
Tomas Kopecek 2023-02-14 16:26:39 +01:00
commit 88d9f8cd13
2 changed files with 6 additions and 4 deletions

View file

@ -15592,8 +15592,10 @@ def create_rpm_checksum(rpm_id, sigkey, chsum_dict):
checksum_type_int = [koji.CHECKSUM_TYPES[func] for func, _ in chsum_dict.items()]
query = QueryProcessor(tables=['rpm_checksum'],
columns=['checksum_type', 'checksum', 'sigkey', 'rpm_id'],
clauses=["checksum_type IN %(checksum_types)s", 'sigkey=%(sigkey)s'],
values={'checksum_types': checksum_type_int, 'sigkey': sigkey})
clauses=["checksum_type IN %(checksum_types)s", 'sigkey=%(sigkey)s',
'rpm_id = %(rpm_id)d'],
values={'checksum_types': checksum_type_int, 'sigkey': sigkey,
'rpm_id': rpm_id})
rows = query.execute()
if len(rows) == len(checksum_type_int):
return None

View file

@ -35,5 +35,5 @@ class TestCreateRPMChecksum(unittest.TestCase):
query = self.queries[0]
self.assertEqual(query.tables, ['rpm_checksum'])
self.assertEqual(query.joins, None)
self.assertEqual(query.clauses,
["checksum_type IN %(checksum_types)s", "sigkey=%(sigkey)s"])
self.assertEqual(set(query.clauses), {"checksum_type IN %(checksum_types)s",
"sigkey=%(sigkey)s", "rpm_id = %(rpm_id)d"})