Skip to content

Commit

Permalink
Fix unit test failures associated with base data area update. (#3753)
Browse files Browse the repository at this point in the history
* Fix missed ProcessImportPds base references and update CMakelists to move translation files from the ISIS3/isis/src/missionName/translations level

* Update classes and tests with base-area changes that were uncovered via CI tests failing
  • Loading branch information
krlberry authored Mar 12, 2020
1 parent c040fb3 commit b77e8dd
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion isis/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -519,7 +519,7 @@ add_custom_target(appdata ALL COMMAND ${CMAKE_COMMAND} -E copy_if_different
${CMAKE_SOURCE_DIR}/appdata/serialnumbers/*.trn ${CMAKE_SOURCE_DIR}/appdata/translations/*.trn
${CMAKE_SOURCE_DIR}/appdata/translations/*.typ ${CMAKE_SOURCE_DIR}/src/*/apps/*/*.typ
${CMAKE_SOURCE_DIR}/src/*/apps/*/*.trn ${CMAKE_SOURCE_DIR}/src/*/translations/*.trn
${CMAKE_SOURCE_DIR}/src/*/apps/*/*.pft
${CMAKE_SOURCE_DIR}/src/*/apps/*/*.pft ${CMAKE_SOURCE_DIR}/src/*/apps/*/*.def
${CMAKE_BINARY_DIR}/appdata/translations)
add_dependencies(isis3 appdata)

Expand Down
2 changes: 1 addition & 1 deletion isis/src/base/objs/EmbreeTargetManager/unitTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ int main(int argc, char *argv[]) {
qDebug() << "";

qDebug() << "Create a new target shape for the same file";
QString copyDSKFile = "$ISIS3DATA/base/testData/hay_a_amica_5_itokawashape_v1_0_64q.bds";
QString copyDSKFile = "$base/testData/hay_a_amica_5_itokawashape_v1_0_64q.bds";
EmbreeTargetShape *copyTargetShape = manager->create(copyDSKFile);
qDebug() << "Target shape status:";
qDebug() << " Number of polygons: " << copyTargetShape->numberOfPolygons();
Expand Down
2 changes: 1 addition & 1 deletion isis/src/base/objs/Histogram/unitTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ int main(int argc, char *argv[]) {
Isis::Progress progress;
Isis::FileName netName("$base/testData/enceladus_sp-Jig.net");
flist.append(netName.expanded());
cout << netName.toString().replace(QRegularExpression("(\\/[\\w\\-\\. ]*)+\\/data"), "data").toStdString() << endl;
cout << netName.toString().replace(QRegularExpression("(\\/[\\w\\-\\. ]*)+\\/*data"), "data").toStdString() << endl;

Isis::ControlNet net(flist[0], &progress);

Expand Down
2 changes: 1 addition & 1 deletion isis/src/base/objs/ProcessExportPds4/ProcessExportPds4.h
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ namespace Isis {
void setImageType(ImageType imageType);
void setPixelDescription(QString description);
static void translateUnits(QDomDocument &label,
QString transMapFile = "$base/translations/pds4ExportUnits.pvl");
QString transMapFile = "$ISISROOT/appdata/translations/pds4ExportUnits.pvl");
void reorder();
void addSchema(QString sch, QString xsd, QString xmlns, QString xmlnsURI);
void addSchema(QString xsd, QString xmlns, QString xmlnsURI);
Expand Down
2 changes: 1 addition & 1 deletion isis/src/base/objs/XmlToPvlTranslationManager/unitTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -370,7 +370,7 @@ int main(void) {
}

try {
FileName pvlFile("$base/translations/pdsImage.trn");
FileName pvlFile("$ISISROOT/appdata/translations/pdsImage.trn");
stringstream simpleTrans;
simpleTrans << "Group = Version" << endl;
simpleTrans << " Auto" << endl;
Expand Down

0 comments on commit b77e8dd

Please sign in to comment.