The last two lines duplicate the results from the first two, except they also capitalize the REV prefix. When enumerating version files, this results in things like:
There is code to de-dup lists like this using the Python set() constructor, but this de-dup is case-sensitive and thus both examples of the file end up being present (see, e.g. PdsFile.all_versions()). Usually this is caught in a later phase of PdsFile, but it causes a warning to be logged (which we don't usually see because we don't have PdsFile logging turned on).
The reason I found this is it changes the code coverage for the PdsFile tests when they are run against Linux-vs-Mac filesystems.
There is no other case where we have this problem, leading me to believe the VERSIONS for CORSS are incorrect in this instance.
From rms-webtools created by rfrenchseti: SETI/rms-webtools#56
The VERSIONS part of
rules/CORSS_8xxx.py
has the following code:The last two lines duplicate the results from the first two, except they also capitalize the REV prefix. When enumerating version files, this results in things like:
There is code to de-dup lists like this using the Python
set()
constructor, but this de-dup is case-sensitive and thus both examples of the file end up being present (see, e.g.PdsFile.all_versions()
). Usually this is caught in a later phase of PdsFile, but it causes a warning to be logged (which we don't usually see because we don't have PdsFile logging turned on).The reason I found this is it changes the code coverage for the PdsFile tests when they are run against Linux-vs-Mac filesystems.
There is no other case where we have this problem, leading me to believe the VERSIONS for CORSS are incorrect in this instance.