Address PR remarks

Author: victorusu

reframe/core/logging.py

@@ -175,8 +175,8 @@ def formatTime(self, record, datefmt=None):
 
     def format(self, record):
         datefmt = self.datefmt or self.default_time_format
-        if record.check_job_unix_completion_time is not None:
-            ct = self.converter(record.check_job_unix_completion_time)
+        if record.check_job_completion_time_unix is not None:
+            ct = self.converter(record.check_job_completion_time_unix)
             record.check_job_completion_time = _format_time_rfc3339(
                 ct, datefmt
             )
@@ -419,7 +419,7 @@ def __init__(self, logger=None, check=None):
                 'check_name': 'reframe',
                 'check_jobid': '-1',
                 'check_job_completion_time': None,
-                'check_job_unix_completion_time': None,
+                'check_job_completion_time_unix': None,
                 'check_info': 'reframe',
                 'check_system': None,
                 'check_partition': None,
@@ -481,7 +481,7 @@ def _update_check_extras(self):
             self.extra['check_jobid'] = self.check.job.jobid
             if self.check.job.completion_time:
                 ct = self.check.job.completion_time
-                self.extra['check_job_unix_completion_time'] = ct
+                self.extra['check_job_completion_time_unix'] = ct
 
     def log_performance(self, level, tag, value, ref,
                         low_thres, upper_thres, unit=None, *, msg=None):

unittests/test_cli.py

@@ -473,6 +473,15 @@ def test_verbosity(self):
         assert 'Traceback' not in stderr
         assert 0 == returncode
 
+    def test_verbosity_with_check(self):
+        self.more_options = ['-vvvvv']
+        self.checkpath = ['unittests/resources/checks/hellocheck.py']
+        returncode, stdout, stderr = self._run_reframe()
+        assert '' != stdout
+        assert 'Traceback' not in stdout
+        assert 'Traceback' not in stderr
+        assert 0 == returncode
+
     @fixtures.switch_to_user_runtime
     def test_unload_module(self):
         # This test is mostly for ensuring coverage. `_run_reframe()` restores