A while back a bugfix was implemented to address the leading zero issue with cycle/pass that resulted in the value being converted to octal. That fix has been working well, but we found one spot where this change is missing and causes issues for any granule where the cycle or pass contains invalid octal characters (8 or 9). In those cases, the validity check (see diff) raises an exception, so the cycle/pass extraction processing does not occur.
In this PR, I've added the UMMUtils.removeStrLeadingZeros call around the cycle and pass in the validity check as well. Also added a unit test.
Note; this has not yet been tested in a full Cumulus end-to-end test.
A while back a bugfix was implemented to address the leading zero issue with cycle/pass that resulted in the value being converted to octal. That fix has been working well, but we found one spot where this change is missing and causes issues for any granule where the cycle or pass contains invalid octal characters (8 or 9). In those cases, the validity check (see diff) raises an exception, so the cycle/pass extraction processing does not occur.
In this PR, I've added the
UMMUtils.removeStrLeadingZeroscall around the cycle and pass in the validity check as well. Also added a unit test.Note; this has not yet been tested in a full Cumulus end-to-end test.