Address PR comments

Author: Vasileios Karakasis

docs/tutorial_deps.rst

@@ -150,7 +150,7 @@ As a result, its immediate dependency :class:`OSUBuildTest` will be skipped, whi
 Listing Dependencies
 --------------------
 
-As shown in the listing of :class:`OSULatencyTest` before, the full dependency chain of the test is listed along with test.
+As shown in the listing of :class:`OSULatencyTest` before, the full dependency chain of the test is listed along with the test.
 Each target dependency is printed in a new line prefixed by the ``^`` character and indented proportionally to its level.
 If a target dependency appears in multiple paths, it will only be listed once.
 

reframe/core/meta.py

@@ -482,9 +482,6 @@ def __call__(cls, *args, **kwargs):
         if not isinstance(fixt_vars, collections.abc.Mapping):
             raise TypeError("'fixt_vars' argument must be a mapping")
 
-        # Intercept is_fixture argument to flag an instance as a fixture
-        # is_fixture = kwargs.pop('is_fixture', False)
-
         obj = cls.__new__(cls, *args, **kwargs)
 
         # Insert the var and param spaces

reframe/schemas/runreport.json

@@ -98,10 +98,7 @@
                 "time_total": {"type": ["number", "null"]},
                 "unique_name": {"type": "string"}
             },
-            "required": [
-                "environment", "fail_phase", "fail_reason", "filename",
-                "result", "stagedir", "system", "time_total", "unique_name"
-            ]
+            "required": ["environment", "stagedir", "system", "unique_name"]
         }
     },
     "type": "object",
@@ -125,7 +122,7 @@
                 "version": {"type": "string"},
                 "workdir": {"type": "string"}
             },
-            "required": ["data_version", "hostname", "time_elapsed", "time_start"]
+            "required": ["data_version"]
         },
         "restored_cases": {
             "type": "array",
@@ -145,9 +142,7 @@
                         "items": {"$ref": "#/defs/testcase_type"}
                     }
                 },
-                "required": [
-                    "num_cases", "num_failures", "num_aborted", "runid", "testcases"
-                ]
+                "required": ["testcases"]
             }
         }
     },

tutorials/basics/stream/stream3.py

@@ -23,7 +23,7 @@ class StreamWithRefTest(rfm.RegressionTest):
     }
     reference = {
         'catalina': {
-            'Copy':  (55200, -0.05, 0.05, 'MB/s'),
+            'Copy':  (25200, -0.05, 0.05, 'MB/s'),
             'Scale': (16800, -0.05, 0.05, 'MB/s'),
             'Add':   (18500, -0.05, 0.05, 'MB/s'),
             'Triad': (18800, -0.05, 0.05, 'MB/s')

unittests/test_parameters.py

@@ -50,7 +50,6 @@ def test_abstract_param():
     class MyTest(Abstract):
         pass
 
-    print(MyTest.param_space)
     assert MyTest.param_space['P0'] == ()
     assert MyTest.param_space['P1'] == ('b',)