Tests: refactored working with processes.

4 files changed, 20 insertions, 14 deletions

diff --git a/test/unit/applications/lang/go.py b/test/unit/applications/lang/go.py
index 6be1667b..367059e6 100644
--- a/test/unit/applications/lang/go.py
+++ b/test/unit/applications/lang/go.py
@@ -40,13 +40,12 @@ class TestApplicationGo(TestApplicationProto):
             print("\n$ GOPATH=" + env['GOPATH'] + " " + " ".join(args))
 
         try:
-            process = subprocess.Popen(args, env=env)
-            process.communicate()
+            process = subprocess.run(args, env=env)
 
         except KeyboardInterrupt:
             raise
 
-        except:
+        except subprocess.CalledProcessError:
             return None
 
         return process
diff --git a/test/unit/applications/lang/java.py b/test/unit/applications/lang/java.py
index 53b27b07..50998978 100644
--- a/test/unit/applications/lang/java.py
+++ b/test/unit/applications/lang/java.py
@@ -64,10 +64,17 @@ class TestApplicationJava(TestApplicationProto):
 
             javac = [
                 'javac',
-                '-target', '8', '-source', '8', '-nowarn',
-                '-encoding',   'utf-8',
-                '-d',          classes_path,
-                '-classpath',  classpath + ':' + ws_jars[0],
+                '-target',
+                '8',
+                '-source',
+                '8',
+                '-nowarn',
+                '-encoding',
+                'utf-8',
+                '-d',
+                classes_path,
+                '-classpath',
+                classpath + ':' + ws_jars[0],
             ]
             javac.extend(src)
 
@@ -75,13 +82,12 @@ class TestApplicationJava(TestApplicationProto):
                 print("\n$ " + " ".join(javac))
 
             try:
-                process = subprocess.Popen(javac, stderr=subprocess.STDOUT)
-                process.communicate()
+                subprocess.check_output(javac, stderr=subprocess.STDOUT)
 
             except KeyboardInterrupt:
                 raise
 
-            except:
+            except subprocess.CalledProcessError:
                 pytest.fail('Can\'t run javac process.')
 
     def load(self, script, **kwargs):
diff --git a/test/unit/applications/tls.py b/test/unit/applications/tls.py
index 583b618f..c7254235 100644
--- a/test/unit/applications/tls.py
+++ b/test/unit/applications/tls.py
@@ -15,7 +15,7 @@ class TestApplicationTLS(TestApplicationProto):
     def certificate(self, name='default', load=True):
         self.openssl_conf()
 
-        subprocess.call(
+        subprocess.check_output(
             [
                 'openssl',
                 'req',
diff --git a/test/unit/check/go.py b/test/unit/check/go.py
index 309091c0..cc17f0fe 100644
--- a/test/unit/check/go.py
+++ b/test/unit/check/go.py
@@ -11,7 +11,7 @@ def check_go(current_dir, temp_dir, test_dir):
     env['GO111MODULE'] = 'auto'
 
     try:
-        process = subprocess.Popen(
+        process = subprocess.run(
             [
                 'go',
                 'build',
@@ -20,8 +20,9 @@ def check_go(current_dir, temp_dir, test_dir):
                 test_dir + '/go/empty/app.go',
             ],
             env=env,
+            stderr=subprocess.STDOUT,
+            stdout=subprocess.PIPE,
         )
-        process.communicate()
 
         if process.returncode == 0:
             return True
@@ -29,5 +30,5 @@ def check_go(current_dir, temp_dir, test_dir):
     except KeyboardInterrupt:
         raise
 
-    except:
+    except subprocess.CalledProcessError:
         return None