on xenial help sign in
Fix issues raised in MR; removed un-needed function, fixed docstring, fix typo for universal_newlines
69c36aeChristopher Lee
tests/autopilot/unity8/process_helpers.py+2−6
@@ -82,12 +82,8 @@ def restart_unity_with_testability(*args):
82 return restart_unity(*args)
82 return restart_unity(*args)
83
83
84
84
85−def restart_unity_normally():
86− restart_unity()
87−
88−
89def restart_unity(*args):
85def restart_unity(*args):
90− """Restarts (or starts) unity8 with the testability driver loaded.
86+ """Restarts (or starts) unity8 using the provided arguments.
91
87
92 Passes *args arguments to the launched process.
88 Passes *args arguments to the launched process.
93
89
@@ -111,7 +107,7 @@ def restart_unity(*args):
111 output = subprocess.check_output(
107 output = subprocess.check_output(
112 command,
108 command,
113 stderr=subprocess.STDOUT,
109 stderr=subprocess.STDOUT,
114− universal_newlines=False,
110+ universal_newlines=True,
115 )
111 )
116 logger.info(output)
112 logger.info(output)
117 pid = _get_unity_pid()
113 pid = _get_unity_pid()