Merge branch 'topic/fix-testsuite-debug-mode' into 'master'

Fix testsuite.py --debug

See merge request eng/ide/ada_language_server!1815

Author: eliericha

testsuite/drivers/basic.py

@@ -1,7 +1,11 @@
 import glob
+import logging
 import os
 import sys
 
+from e3.os.process import Run, command_line_image
+
+
 from drivers import ALSTestDriver
 from e3.testsuite.driver.classic import TestAbortWithFailure
 
@@ -29,17 +33,36 @@ def run(self):
             if self.env.options.format:
                 cmd.append("--format=%s" % self.env.options.format)
 
+            env = {
+                "ALS": self.env.als,
+                "ALS_HOME": self.env.als_home,
+                "ALS_WAIT_FACTOR": str(self.env.wait_factor),
+                "PYTHON": sys.executable,
+            }
+
             if self.env.main_options.debug:
                 cmd.append("--debug")
 
-            self.shell(
-                cmd,
-                cwd=wd,
-                env={
-                    "ALS": self.env.als,
-                    "ALS_HOME": self.env.als_home,
-                    "ALS_WAIT_FACTOR": str(self.env.wait_factor),
-                    "PYTHON": sys.executable,
-                },
-                ignore_environ=False,
-            )
+                # In debug mode, we want to run the subprocess without piping so that it
+                # can communicate the PID to debug and wait for input. So we use
+                # e3.process.Run directly instead of self.shell.
+                logging.info(f"Running: cd {wd}; {command_line_image(cmd)}")
+                p = Run(
+                    cmd,
+                    cwd=wd,
+                    env=env,
+                    ignore_environ=False,
+                    output=None,
+                )
+                if p.status != 0:
+                    raise TestAbortWithFailure("non-zero status code")
+            else:
+                p = self.shell(
+                    cmd,
+                    cwd=wd,
+                    env=env,
+                    ignore_environ=False,
+                )
+
+                if self.env.main_options.verbose:
+                    print(p.out)