diff --git a/rida/scheduler/handlers/modules.py b/rida/scheduler/handlers/modules.py index 4e4d4191..b3c7bb2a 100644 --- a/rida/scheduler/handlers/modules.py +++ b/rida/scheduler/handlers/modules.py @@ -52,7 +52,7 @@ def init(config, session, msg): log.debug("Received module_info=%s from pdc" % module_info) tag = rida.pdc.get_module_tag(pdc, module_info) - log.info("Found tag=%s for module %s-%s-%s" % (tag, build['name'], build['version'], build['release'])) + log.info("Found tag=%s for module %s-%s-%s" % (tag, build.name, build.version, build.release)) dependencies = rida.pdc.get_module_dependencies(pdc, module_info) builder = rida.builder.KojiModuleBuilder(build.name, config) diff --git a/tests/test_scheduler/test_modules/test_init.py b/tests/test_scheduler/test_modules/test_init.py index 294e1be8..9ca15c7b 100644 --- a/tests/test_scheduler/test_modules/test_init.py +++ b/tests/test_scheduler/test_modules/test_init.py @@ -35,7 +35,7 @@ class TestInit(unittest.TestCase): @mock.patch('rida.builder.KojiModuleBuilder') @mock.patch('rida.database.ModuleBuild.from_fedmsg') - @mock.patch('rida.pdc.get_pdc_client_session') + @mock.patch('rida.pdc') def test_init_basic(self, pdc, from_fedmsg, KojiModuleBuilder): builder = mock.Mock() KojiModuleBuilder.return_value = builder @@ -45,6 +45,11 @@ class TestInit(unittest.TestCase): 'version': 1, } from_fedmsg.return_value = mocked_module_build + pdc.get_module.return_value = { + 'name': 'foo', + 'version': 1, + 'release': 1, + } msg = { 'topic': 'org.fedoraproject.prod.rida.module.state.change',