From 7d57771833bf2c2b2bf3b5f3ba5e15918d88c977 Mon Sep 17 00:00:00 2001 From: "xrg@hellug.gr" <> Date: Fri, 3 Sep 2010 00:04:51 +0300 Subject: [FIX] base_module_quality: log, fix workflow test When no wkf_ids are located for the module, search() must be skipped and have instead an empty list of activities. Otherwise, all the activities for all modules would be selected, breaking the test bzr revid: xrg@hellug.gr-20100902210451-30eaz5vdtplhjlgr --- addons/base_module_quality/workflow_test/workflow_test.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'addons/base_module_quality/workflow_test/workflow_test.py') diff --git a/addons/base_module_quality/workflow_test/workflow_test.py b/addons/base_module_quality/workflow_test/workflow_test.py index f92107e8fbd..11f7bcf12c3 100644 --- a/addons/base_module_quality/workflow_test/workflow_test.py +++ b/addons/base_module_quality/workflow_test/workflow_test.py @@ -74,9 +74,14 @@ class quality_test(base_module_quality.abstract_quality_check): wkf_ids = map(lambda x:x['id'], wkfs) if not wkf_ids: result_dict[module_name] = [module_name, 'No workflow defined on module'] - #Activity of workflow checking... - activity_ids = wkf_activity_obj.search(cr, uid, [('wkf_id', 'in', wkf_ids)]) - activities = wkf_activity_obj.browse(cr, uid, activity_ids) + activity_ids = [] + activities = [] + else: + #Activity of workflow checking... + activity_ids = wkf_activity_obj.search(cr, uid, [('wkf_id', 'in', wkf_ids)]) + activities = wkf_activity_obj.browse(cr, uid, activity_ids) + self.log.debug("quality test: wkf_ids = %r", wkf_ids) + self.log.debug("quality test: activity_ids = %r", activity_ids) for activity in activities: if activity.flow_start: activity_chk[activity.wkf_id.osv]['start'] = 'ok' -- cgit v1.2.3