Seem quite unnecessary. We haven't found that to be an issue elsewhere in the code where slurp_file is used. And in the present case we know the file exists because we got its name from list_files().Agreed. That's an exchange between a hard failure mid-test and a failure while letting the whole test run. Here, we expect the test to find the log file all the time, so a hard failure does not sound like a bad thing to me either.
Ok, fine with me! -- Fabien.