From 4abde33785ac85ae7e60e71767cd149640f1e6bd Mon Sep 17 00:00:00 2001
From: Xavier Morel <xmo@openerp.com>
Date: Wed, 6 Nov 2013 14:07:43 +0100
Subject: [PATCH] [IMP] always print a recap of found test modules before
 running the tests

bzr revid: xmo@openerp.com-20131106130743-xpy9i34b5xceo8ee
---
 openerpcommand/run_tests.py | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/openerpcommand/run_tests.py b/openerpcommand/run_tests.py
index a3aa39a05ca..fbcbfa398e6 100644
--- a/openerpcommand/run_tests.py
+++ b/openerpcommand/run_tests.py
@@ -63,8 +63,8 @@ def get_test_modules(module, submodule, explode):
         ms = submodules
     elif submodule == '__fast_suite__':
         # `suite` was used before the 6.1 release instead of `fast_suite`.
-        ms = fast_suite if fast_suite else getattr(m, 'suite', None)
-        if ms is None:
+        ms = fast_suite if hasattr(m, 'fast_suite') else getattr(m, 'suite', None)
+        if not ms:
             if explode:
                 print 'The module `%s` has no defined test suite.' % (module,)
                 show_submodules_and_exit()
@@ -72,7 +72,7 @@ def get_test_modules(module, submodule, explode):
                 ms = []
     elif submodule == '__sanity_checks__':
         ms = checks
-        if ms is None:
+        if not ms:
             if explode:
                 print 'The module `%s` has no defined sanity checks.' % (module,)
                 show_submodules_and_exit()
@@ -135,7 +135,11 @@ def run(args):
     else:
         test_modules = get_test_modules(module, submodule, explode=True)
 
-    # Run the test suite.
+    print 'Test modules:'
+    for test_module in test_modules:
+        print '    ', test_module.__name__
+    print
+
     if not args.dry_run:
         suite = unittest2.TestSuite()
         for test_module in test_modules:
@@ -143,10 +147,6 @@ def run(args):
         r = unittest2.TextTestRunner(verbosity=2).run(suite)
         if r.errors or r.failures:
             sys.exit(1)
-    else:
-        print 'Test modules:'
-        for test_module in test_modules:
-            print ' ', test_module.__name__
 
 def add_parser(subparsers):
     parser = subparsers.add_parser('run-tests',