Tests: convert more left-over type hint to type comment
authorMichael Jeanson <mjeanson@efficios.com>
Mon, 1 May 2023 20:50:57 +0000 (16:50 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Mon, 1 May 2023 21:27:44 +0000 (17:27 -0400)
Use type comments to support older python3 interpreters that can't
handle type hints (such as 3.4).

Change-Id: I3599311dfcc172344c759f8c0e22af640759f1f5
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
tests/regression/ust/ust-constructor/test_ust_constructor.py
tests/utils/lttngtest/environment.py

index 1682071e2fcbbe82d6d0c3ecf2dccd0e694a4732..06939742ea5c891b5f7c82a70feab28ff737a725 100755 (executable)
@@ -196,7 +196,7 @@ def capture_trace(tap, test_env):
     return session_output_location
 
 
-def validate_trace(trace_location, tap) -> bool:
+def validate_trace(trace_location, tap):
     # type: (pathlib.Path, lttngtest.TapGenerator) -> bool
     success = True
     unknown_event_count = 0
index 282e95643c33a6ece9cd2f815bc1d28f5f6f69f0..0efb2b5f526ea5edba9ea5199e87772645d3db4d 100644 (file)
@@ -192,8 +192,9 @@ class TraceTestApplication:
     Create an application to trace.
     """
 
-    def __init__(self, binary_path: pathlib.Path, environment: "Environment"):
-        self._environment: Environment = environment
+    def __init__(self, binary_path, environment):
+        # type: (pathlib.Path, Environment)
+        self._environment = environment  # type: Environment
         self._has_returned = False
 
         test_app_env = os.environ.copy()
@@ -208,7 +209,8 @@ class TraceTestApplication:
             test_app_args, env=test_app_env
         )
 
-    def wait_for_exit(self) -> None:
+    def wait_for_exit(self):
+        # type: () -> None
         if self._process.wait() != 0:
             raise RuntimeError(
                 "Test application has exit with return code `{return_code}`".format(
@@ -412,9 +414,8 @@ class _Environment(logger._Logger):
             self,
         )
 
-    def launch_trace_test_constructor_application(
-        self
-    ) -> TraceTestApplication:
+    def launch_trace_test_constructor_application(self):
+        # type () -> TraceTestApplication
         """
         Launch an application that will trace from within constructors.
         """
This page took 0.035587 seconds and 4 git commands to generate.