Skip to content

Commit 0b666e0

Browse files
committed
Fixed the way in which process are stopped, now it shows the full log
1 parent 8ea07b2 commit 0b666e0

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

interoperability_report.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,16 +51,19 @@ def stop_process(child_process, timeout=30, poll_interval=0.2):
5151
else:
5252
return True # Process already exited
5353

54-
start_time = time.time()
54+
return_value = True
5555

56+
start_time = time.time()
5657
while child_process.isalive() and (time.time() - start_time < timeout):
5758
time.sleep(poll_interval)
5859

5960
if child_process.isalive():
6061
child_process.terminate(force=True)
61-
return False # Process was forcefully terminated
62+
return_value = False # Process was forcefully terminated
63+
64+
child_process.expect(pexpect.EOF, timeout=5)
6265

63-
return True
66+
return return_value
6467

6568
def run_subscriber_shape_main(
6669
name_executable: str,

test_suite.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@
122122

123123
# RELIABILITY
124124
'Test_Reliability_0' : {
125-
'apps' : ['-P -t Square -b -z 0', '-S -t Square -b -z 0'],
125+
'apps' : ['-P -t Square -b -z 0', '-S -t Square -b'],
126126
'expected_codes' : [ReturnCode.OK, ReturnCode.OK],
127127
'check_function' : tsf.test_reliability_order,
128128
'title' : 'Communication between BEST_EFFORT publisher and subscriber',

0 commit comments

Comments
 (0)