Improve message and enhance implementation

Author: Vasileios Karakasis

reframe/core/decorators.py

@@ -7,17 +7,26 @@
 
 import collections
 import inspect
+import sys
 import traceback
 
 import reframe
-from reframe.core.exceptions import ReframeSyntaxError
+from reframe.core.exceptions import ReframeSyntaxError, user_frame
 from reframe.core.logging import getlogger
 from reframe.core.pipeline import RegressionTest
 from reframe.utility.versioning import Version, VersionValidator
 
 
 def _register_test(cls, args=None):
-    def _instantiate():
+    def _instantiate(cls, args):
+        if isinstance(args, collections.Sequence):
+            return cls(*args)
+        elif isinstance(args, collections.Mapping):
+            return cls(**args)
+        elif args is None:
+            return cls()
+
+    def _instantiate_all():
         ret = []
         for cls, args in mod.__rfm_test_registry:
             try:
@@ -28,23 +37,19 @@ def _instantiate():
                 mod.__rfm_skip_tests = set()
 
             try:
-                if isinstance(args, collections.Sequence):
-                    ret.append(cls(*args))
-                elif isinstance(args, collections.Mapping):
-                    ret.append(cls(**args))
-                elif args is None:
-                    ret.append(cls())
+                _instantiate(cls, args)
             except Exception as e:
-                getlogger().error('%s:%s: skipping due to errors; check log'
-                                  'file for more information.' %
-                                  (inspect.getfile(cls), cls.__name__))
-                getlogger().debug(traceback.format_exc())
+                frame = user_frame(sys.exc_info()[2])
+                msg = 'skipping test due to errors: %s\n' % cls.__name__
+                msg += '  %s:%s' % (frame.filename, frame.lineno)
+                getlogger().warning(msg)
+                getlogger().verbose(traceback.format_exc())
 
         return ret
 
     mod = inspect.getmodule(cls)
     if not hasattr(mod, '_rfm_gettests'):
-        mod._rfm_gettests = _instantiate
+        mod._rfm_gettests = _instantiate_all
 
     try:
         mod.__rfm_test_registry.append((cls, args))