Test suite expects `/data1` to not exists

[cfv-project]

Test suite expects `/data1` to not exists

[lxp]

@Terry-Kennedy Can you verify, if the current python3 branch fixes the issue on your system too?

[Terry-Kennedy]

It does fix the issue and the test suite now runs cleanly.

But I'm confused - if the file it wants to test against is in test/testdata/data1 why does it seem to be passing /data1 (in the root of the filesystem) instead of test/testdata/data1? Note that I have a /data2 and /data3 and tests don't trip over those...

[lxp]

The affected test case tests the option --strippaths=none with the checksum file https://github.com/cfv-project/cfv/blob/python3/test/testdata/teststrip-none.csv4, which references /data1. This option changes the default behavior (--strippaths=0), so that the root folder or drive is not stripped.

As it is impossible to write a static positive test case for this, because the temp directory path changes with every run and we do not want to write anywhere outside that temp directory, this test case is implemented as negative test case.
If the option is applied correctly, it previously expected the result not found. I changed it to check if /data1 actually exists and to expect the result bad crc in such a case.

If there would be a major implementation error with the --strippaths=none option, then one outcome could be that it strips the root folder and finds the file in test/testdata/data1, which would result in OK and, therefore, in a failed test case.

The --strippaths test cases only use data1, and not the other test files. Therefore, you don't see a problem with the others.

One additional note: If I would want to rename the test file data1, this would basically affect all tests and I would have to change or regenerate nearly all static test checksum files located in test/testdata.

I hope this explains the issue well enough. I am going to close the issue, if you still have questions regarding it, you can just write another comment or if that doesn't work just re-open the issue.