On Fri, 12 Jul 2019, 6:09 pm Chris Lamb, <la...@debian.org> wrote: > * DIFFOSCOPE_FAIL_TESTS_ON_MISSING_TOOLS='' > → Fails; the required version is missing and unlisted. > > * DIFFOSCOPE_FAIL_TESTS_ON_MISSING_TOOLS="foo bar" > → Fails; the required version is missing and unlisted. >
These shouldn't fail. As in, the program is present, but the required version is missing. That variable isn't holding versions after all. At least, having that fail in this case wasn't my intention when I wrote that thing. If you went down that route it would make that variable/setting unusable in any circumstance where simply the required version is missing. Having said all of this, I think we should make the failing test able to run with either of the file versions. `file` keeps changing and breaking our tests, but it would be awful to only support running the testsuite with the latest version available. > >
