Address PR comments

Author: ekouts

reframe/core/pipeline.py

@@ -148,6 +148,17 @@ def disable_hook(cls, hook_name):
         '''
         cls._rfm_disabled_hooks.add(hook_name)
 
+    @classmethod
+    def hooks(cls):
+        ret = {}
+        for c in cls.mro():
+            if hasattr(c, '_rfm_pipeline_hooks'):
+                for kind, hook in c._rfm_pipeline_hooks.items():
+                    ret.setdefault(kind, [])
+                    ret[kind] += hook
+
+        return ret
+
     #: The name of the test.
     #:
     #: :type: string that can contain any character except ``/``
@@ -949,17 +960,6 @@ def stderr(self):
         '''
         return self._job.stderr
 
-    @property
-    def hooks(self):
-        ret = {}
-        for c in inspect.getmro(type(self)):
-            if hasattr(c, '_rfm_pipeline_hooks'):
-                for k, v in c._rfm_pipeline_hooks.items():
-                    ret.setdefault(k, [])
-                    ret[k].extend(v)
-
-        return ret
-
     @property
     def build_job(self):
         return self._build_job

reframe/frontend/cli.py

@@ -71,7 +71,7 @@ def fmt_deps():
         'Node allocation': node_alloc_scheme,
         'Pipeline hooks': {
             k: fmt_list(fn.__name__ for fn in v)
-            for k, v in check.hooks.items()
+            for k, v in type(check).hooks().items()
         },
         'Tags': fmt_list(check.tags),
         'Valid environments': fmt_list(check.valid_prog_environs),