diff options
| author | xrg@hellug.gr <> | 2010-09-03 00:04:51 +0300 |
|---|---|---|
| committer | xrg@hellug.gr <> | 2010-09-03 00:04:51 +0300 |
| commit | 7d57771833bf2c2b2bf3b5f3ba5e15918d88c977 (patch) | |
| tree | c6b9638d5060c26dddc74ef9bf1eb7a9435e077a /addons/base_module_quality/workflow_test | |
| parent | d67d284e003c28a56ab4c2e23731158001d24f78 (diff) | |
[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
Diffstat (limited to 'addons/base_module_quality/workflow_test')
| -rw-r--r-- | addons/base_module_quality/workflow_test/workflow_test.py | 11 |
1 files changed, 8 insertions, 3 deletions
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' |