Project

General

Profile

Bug #5377

stats default app test fails depending on length of FROM string

Added by Ian Humphrey 9 months ago. Updated about 2 months ago.

Status:
Acknowledged
Priority:
Normal
Assignee:
-
Category:
Applications
Target version:
-
Impact:
Software Version:
Test Reviewer:

Description

If you look at the Makefile for the stats "default" test case, the last command run is:

$(APPNAME) from=$(INPUT)/isisTruth.cub \
           | grep -v stats \
           | grep -v Processed \
           >& $(OUTPUT)/output.pvl;

The top of the output.pvl file looks like:

Group = Results
  From                    = /usgs/cpkgs/isis3/testData/isis/src/base/apps/sta-
                            ts/tsts/default/input/isisTruth.cub

Note that there is a grep -v stats command that occurs. It just so happens that the "stats" word in the path is actually separated (to not exceed a certain character length in the PVL).

If you happen to provide your own testData symlink next to ISISROOT and the "stats" word is not separated by the character length limit, the grep -v stats command will remove all lines that contain the word "stats". This can cause an issue if th

This can be replicated by doing the following:

cd /usgs/pkgs/isis3.5.2/isis/src/base/apps/stats/tsts/default
make release
make test
make output

The output/output.pvl file looks like:

Group = Results
  From                    = /usgs/pkgs/isis3.5.2/isis/../testData/isis/src/ba-
  Band                    = 1

This will cause the following error to occur:

*ERROR** Unable to read PVL file [output/output.pvl].
**ERROR** Error in PVL file on line [3].
**ERROR** Unable to read PVL keyword [From                    = /usgs/pkgs/isis3.5.2/isis/../testData/isis/src/baBand                    = 1].
**ERROR** Keyword has extraneous data [= 1] at the end.

I think this is happening because the end of the "From" keyword has "ba-", so the PVL parser is expecting the next line to continue the "From" keyword. So, it tries to attach "Band = 1" to the path and fails.

History

#1 Updated by Makayla Shepherd 9 months ago

  • Status changed from New to Acknowledged

Also available in: Atom PDF