I just sync'd apex_launchtest from github into our internal repo, and these three commits represent places where our internal repo was ahead of the public apex_rostest
abf801d - If you have a long running test that takes more than 60 seconds, you need to give a longer timeout to ament_add_test. This commit plumbs that argument up to the add_apex_rostest cmake macro
9ca67ef - This was a for-real bug. When we try to print extra information when the processes under test die out from under us, the code would fail if one or more of the processes generated no output.
TODO
[x] Add a regression test for the issue fixed in 9ca67ef. Right now the except KeyError: part below is not covered. A regression test will cover these lines
def _print_process_output_summary(self):
failed_procs = [proc for proc in self.proc_info if proc.returncode != 0]
for process in failed_procs:
print("Process '{}' exited with {}".format(process.process_name, process.returncode))
print("##### '{}' output #####".format(process.process_name))
try:
for io in self.proc_output[process.action]:
print("{}".format(io.text.decode('ascii')))
except KeyError:
pass # Process generated no output
print("#" * (len(process.process_name) + 21))
@jpsamper2009 Any chance you can approve this? It's a PR for the areas where the internal apex_launchtest was ahead of the github version. This changes have already been approved once
I just sync'd apex_launchtest from github into our internal repo, and these three commits represent places where our internal repo was ahead of the public apex_rostest
TODO
[x] Add a regression test for the issue fixed in 9ca67ef. Right now the
except KeyError:
part below is not covered. A regression test will cover these lines