diff --git a/module_build_service/scheduler/handlers/modules.py b/module_build_service/scheduler/handlers/modules.py index 2875dd51..c4e1499d 100644 --- a/module_build_service/scheduler/handlers/modules.py +++ b/module_build_service/scheduler/handlers/modules.py @@ -160,4 +160,7 @@ def wait(config, session, msg): # If this build already exists and is done, then fake the repo change event # back to the scheduler if state == koji.BUILD_STATES['COMPLETE']: - return [module_build_service.messaging.KojiRepoChange('fake msg', build.koji_tag)] + # TODO: builder.module_build_tag only works for Koji, figure out if + # other backends need this implemented (e.g. COPR) + return [module_build_service.messaging.KojiRepoChange( + 'fake msg', builder.module_build_tag['name'])] diff --git a/tests/test_scheduler/test_module_wait.py b/tests/test_scheduler/test_module_wait.py index da3f0bd1..c07b1f2f 100644 --- a/tests/test_scheduler/test_module_wait.py +++ b/tests/test_scheduler/test_module_wait.py @@ -40,6 +40,7 @@ class TestModuleWait(unittest.TestCase): builder = mock.Mock() builder.get_disttag_srpm.return_value = 'some srpm disttag' builder.build.return_value = 1234, 1, "", None + builder.module_build_tag = {'name': 'some-tag-build'} KojiModuleBuilder.return_value = builder mocked_module_build = mock.Mock() mocked_module_build.json.return_value = {