From 7615b8aa8505b78046903837af85b7a34c9d49e0 Mon Sep 17 00:00:00 2001
From: Mike McLean <mikem@redhat.com>
Date: Tue, 14 Dec 2021 14:31:27 -0500
Subject: [PATCH] actually execute the insert

---
 hub/kojihub.py                     | 1 +
 tests/test_hub/test_lookup_name.py | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/hub/kojihub.py b/hub/kojihub.py
index 14560a91..2e69bca0 100644
--- a/hub/kojihub.py
+++ b/hub/kojihub.py
@@ -3282,6 +3282,7 @@ def lookup_name(table, info, strict=False, create=False):
         new_id = nextval(f'{table}_id_seq')
         insert = InsertProcessor(table)
         insert.set(id=new_id, name=info)
+        insert.execute()
         return {'id': new_id, 'name': info}
     else:
         # no match and not strict
diff --git a/tests/test_hub/test_lookup_name.py b/tests/test_hub/test_lookup_name.py
index 8b630d47..42346fa4 100644
--- a/tests/test_hub/test_lookup_name.py
+++ b/tests/test_hub/test_lookup_name.py
@@ -20,7 +20,6 @@ class TestLookupName(unittest.TestCase):
         self.InsertProcessor = mock.patch('kojihub.InsertProcessor',
                                           side_effect=self.getInsert).start()
         self.inserts = []
-        self.insert_execute = mock.MagicMock()
         self.nextval = mock.patch('kojihub.nextval').start()
         self.context = mock.patch('kojihub.context').start()
 
@@ -32,7 +31,7 @@ class TestLookupName(unittest.TestCase):
 
     def getInsert(self, *args, **kwargs):
         insert = IP(*args, **kwargs)
-        insert.execute = self.insert_execute
+        insert.execute = mock.MagicMock()
         self.inserts.append(insert)
         return insert
 
@@ -128,6 +127,7 @@ class TestLookupName(unittest.TestCase):
         self.assertEqual(insert.table, 'package')
         self.assertEqual(insert.data, expected)
         self.assertEqual(insert.rawdata, {})
+        insert.execute.assert_called_once()
 
     def test_lookup_name_create_wrong_type(self):
         self.query_executeOne.return_value = None