search

TopEFT / topeft

15 stars 24 forks source link

fail test_futures if subprocess fails #375

Closed btovar closed 1 year ago

btovar commented 1 year ago

Previously the test only checked for the presence of an output file. If the file was there from a previous run, the test would be reported as successful even if the python subprocess failed. This pr makes it so that the subprocess call is checked for correct termination.

kmohrman commented 1 year ago

Looks fine to me as well (sorry I did not see this till now).