The .exe file is no longer produced in a dev build

Since we switched to pip install --editable instead of calling
setup.py directly, we no longer get an executable file in the build
output, so this test needed adjusting. It was also attempting to do a
for loop and returning at the first entry so we fixed that.

Co-authored-by: Jonathan Haigh <jonathan.haigh@stfc.ac.uk>

Author: jclarkeSTFC

Testing/SystemTests/tests/qt/WorkbenchStartupTest.py

@@ -10,11 +10,14 @@
 import sys
 import systemtesting
 
-from mantid.kernel import ConfigService
 from tempfile import NamedTemporaryFile
 
 TEST_MESSAGE = "Hello Mantid!"
-EXECUTABLE_SWITCHER = {"linux": ["launch_mantidworkbench.sh", "mantidworkbench"], "darwin": ["workbench"], "win32": ["workbench.exe"]}
+EXECUTABLE_SWITCHER = {
+    "linux": ["launch_mantidworkbench.sh", "mantidworkbench", "workbench"],
+    "darwin": ["workbench"],
+    "win32": ["workbench"],
+}
 SUBPROCESS_TIMEOUT_SECS = 300
 
 
@@ -25,15 +28,6 @@ def get_mantid_executables_for_platform(platform):
     return workbench_executables
 
 
-def get_mantid_executable_path(platform):
-    directory = ConfigService.getPropertiesDir().replace("\\", "/")
-    for executable in get_mantid_executables_for_platform(platform):
-        workbench_exe = os.path.join(directory, executable)
-        if os.path.exists(workbench_exe):
-            return workbench_exe
-    raise RuntimeError(f"Could not find path to {workbench_exe}. Tried {get_mantid_executables_for_platform(platform)}")
-
-
 def start_and_wait_for_completion(args_list):
     pids_before_open = set(psutil.pids())
     process = subprocess.Popen(args_list, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
@@ -78,18 +72,18 @@ def __init__(self):
 
         self._test_file = NamedTemporaryFile(suffix=".txt", delete=False).name.replace("\\", "/")
         self._test_script = NamedTemporaryFile(suffix=".py", delete=False).name.replace("\\", "/")
-        self._executable = get_mantid_executable_path(sys.platform)
+        self._executables = get_mantid_executables_for_platform(sys.platform)
         write_test_script(self._test_script, self._test_file)
 
     def runTest(self):
-        exitcode = start_and_wait_for_completion([self._executable, "--execute", self._test_script, "--quit"])
-
-        # Was the process successful
-        self.assertEqual(0, exitcode)
-        # Assert that the test script runs successfully by writing to a .txt file
-        with open(self._test_file, "r") as file:
-            self.assertEqual(TEST_MESSAGE, file.readline())
+        for executable in self._executables:
+            exitcode = start_and_wait_for_completion([executable, "--execute", self._test_script, "--quit"])
+            # Was the process successful
+            self.assertEqual(0, exitcode)
+            # Assert that the test script runs successfully by writing to a .txt file
+            with open(self._test_file, "r") as file:
+                self.assertEqual(TEST_MESSAGE, file.readline())
+            remove_file(self._test_file)
 
     def cleanup(self):
         remove_file(self._test_script)
-        remove_file(self._test_file)