Skip to content

Commit 45ac44e

Browse files
committed
Set 60 second timeout on proc.communicate
1 parent d39401a commit 45ac44e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/test_device.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -164,17 +164,17 @@ def do_command(self, args):
164164
cli_args.append(shlex.quote(arg))
165165
if self.interface == 'cli':
166166
proc = subprocess.Popen(['hwi ' + ' '.join(cli_args)], stdout=subprocess.PIPE, shell=True)
167-
result = proc.communicate()
167+
result = proc.communicate(timeout=60)
168168
return json.loads(result[0].decode())
169169
elif self.interface == 'bindist':
170170
proc = subprocess.Popen(['../dist/hwi ' + ' '.join(cli_args)], stdout=subprocess.PIPE, shell=True)
171-
result = proc.communicate()
171+
result = proc.communicate(timeout=60)
172172
return json.loads(result[0].decode())
173173
elif self.interface == 'stdin':
174174
args = [f'"{arg}"' for arg in args]
175175
input_str = '\n'.join(args) + '\n'
176176
proc = subprocess.Popen(['hwi', '--stdin'], stdin=subprocess.PIPE, stdout=subprocess.PIPE)
177-
result = proc.communicate(input_str.encode())
177+
result = proc.communicate(input_str.encode(), timeout=60)
178178
return json.loads(result[0].decode())
179179
else:
180180
return process_commands(args)

0 commit comments

Comments
 (0)