Properly terminate supervisor following test run

Author: bnaul

tools/test_frontend.py

@@ -1,14 +1,15 @@
 #!/usr/bin/env python
 
-import sys
 import os
+from os.path import join as pjoin
 import pathlib
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), '..'))
+import sys
+sys.path.insert(0, pjoin(os.path.dirname(__file__), '..'))
 
+import signal
+import socket
 import subprocess
-from os.path import join as pjoin
 import time
-import socket
 from baselayer.tools.supervisor_status import supervisor_status
 try:
     import http.client as http
@@ -73,7 +74,8 @@ def delete_test_yaml():
 
     clear_tables()
 
-    web_client = subprocess.Popen(['make', 'testrun'], cwd=base_dir)
+    web_client = subprocess.Popen(['make', 'testrun'], cwd=base_dir,
+                                  preexec_fn=os.setsid)
 
     print('[test_frontend] Waiting for supervisord to launch all server processes...')
 
@@ -116,5 +118,5 @@ def delete_test_yaml():
         raise
     finally:
         print('[test_frontend] Terminating supervisord...')
-        web_client.terminate()
+        os.killpg(os.getpgid(web_client.pid), signal.SIGTERM)
         delete_test_yaml()