modifs

bzr revid: fp@tinyerp.com-20081229172857-rwpkla4hmala06vh

1 files changed, 27 insertions, 13 deletions

diff --git a/addons/base_module_quality/method_test/method_test.py b/addons/base_module_quality/method_test/method_test.py
index 0ef59cc2b29..1da2e06ad97 100644
--- a/addons/base_module_quality/method_test/method_test.py
+++ b/addons/base_module_quality/method_test/method_test.py
@@ -30,23 +30,24 @@ import pooler
 class quality_test(base_module_quality.abstract_quality_check):
 
     def __init__(self):
-        self.result = """
-===Method Test===:
-
-This test checks if the module classes are raising exception when calling basic methods or no.
-
-"""
+        super(quality_test, self).__init__()
+#        self.result = """
+#===Method Test===:
+#
+#This test checks if the module classes are raising exception when calling basic methods or no.
+#
+#"""
         self.bool_installed_only = True
         return None
 
-    def run_test(self, cr, uid, module_path):
+    def run_test(self, cr, uid, module_path, module_state):
         pool = pooler.get_pool(cr.dbname)
         module_name = module_path.split('/')[-1]
         obj_list = self.get_objects(cr, uid, module_name)
         result = {}
         ok_count = 0
         ex_count = 0
-
+        error = False
         for obj in obj_list:
             temp = []
             try:
@@ -71,13 +72,26 @@ This test checks if the module classes are raising exception when calling basic
                 temp.append('Exception')
                 ex_count += 1
             result[obj] = temp
-        self.result += ('{| border="1" cellspacing="0" cellpadding="5" align="left" \n! %-40s \n! %-16s \n! %-20s \n! %-16s ') % ('Object Name'.ljust(40), 'search()'.ljust(16), 'fields_view_get()'.ljust(20), 'read()'.ljust(16))
+#        self.result += ('{| border="1" cellspacing="0" cellpadding="5" align="left" \n! %-40s \n! %-16s \n! %-20s \n! %-16s ') % ('Object Name'.ljust(40), 'search()'.ljust(16), 'fields_view_get()'.ljust(20), 'read()'.ljust(16))
+        header_list = ['Object Name', 'search()', 'fields_view_get', 'read']
+#        for res in result:
+#            self.result += ('\n|-\n| %s \n| %s \n| %s \n| %s ') % (res, result[res][0],result[res][1], result[res][2])
+#        self.result += '\n|}'
+        self.score = (ok_count + ex_count) and float(ok_count)/float(ok_count + ex_count) or 0.0
+        if not self.bool_installed_only or module_state=="installed":
+            summary = """
+===Method Test===:
 
-        for res in result:
-            self.result += ('\n|-\n| %s \n| %s \n| %s \n| %s ') % (res, result[res][0],result[res][1], result[res][2])
+This test checks if the module classes are raising exception when calling basic methods or no.
 
-        self.result += '\n|}'
-        self.score = (ok_count + ex_count) and float(ok_count)/float(ok_count + ex_count) or 0.0
+""" + "Score: " + str(self.score) + "/10\n"
+        else:
+            summary ="""  \n===Method Test===:
+
+The module has to be installed before running this test.\n\n """
+            header_list = ""
+            error = True
+        self.result = self.format_table(test='method', header=header_list, data_list=[summary,result,error])
         return None