Skip to content

Commit

Permalink
Merge pull request #10357 from NREL/IsolateNonASCIITests
Browse files Browse the repository at this point in the history
Protect a few Plugin tests with LINK_WITH_PYTHON
Myoldmopar authored Jan 4, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
2 parents 8c0e3da + b1214c3 commit fb265cd
Showing 1 changed file with 11 additions and 10 deletions.
21 changes: 11 additions & 10 deletions src/EnergyPlus/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1111,16 +1111,17 @@ if(BUILD_TESTING)
set(PY_FILE "${EXAMPLES_FILES_DIR}/${TEST_CASE}.py")
configure_file(${PY_FILE} "${INPUT_TEST_DIR}/${TEST_CASE}.py" COPYONLY)

add_test(NAME energyplus.TestNonASCIIDirsAndFiles.PythonPlugin.FromWithin
COMMAND energyplus -D -d "${CLI_TEST_DIR}/PythonPlugin.FromWithin/out-${NON_ASCII_DIRNAME}" ${TEST_CASE}.idf
WORKING_DIRECTORY "${CLI_TEST_DIR}/${NON_ASCII_DIRNAME}"
)

add_test(NAME energyplus.TestNonASCIIDirsAndFiles.PythonPlugin.FromOutside
COMMAND energyplus -D -d "${CLI_TEST_DIR}/PythonPlugin.FromOutside/out-${NON_ASCII_DIRNAME}" ${NON_ASCII_DIRNAME}/${TEST_CASE}.idf
WORKING_DIRECTORY "${CLI_TEST_DIR}"
)

if(LINK_WITH_PYTHON)
add_test(NAME energyplus.TestNonASCIIDirsAndFiles.PythonPlugin.FromWithin
COMMAND energyplus -D -d "${CLI_TEST_DIR}/PythonPlugin.FromWithin/out-${NON_ASCII_DIRNAME}" ${TEST_CASE}.idf
WORKING_DIRECTORY "${CLI_TEST_DIR}/${NON_ASCII_DIRNAME}"
)

add_test(NAME energyplus.TestNonASCIIDirsAndFiles.PythonPlugin.FromOutside
COMMAND energyplus -D -d "${CLI_TEST_DIR}/PythonPlugin.FromOutside/out-${NON_ASCII_DIRNAME}" ${NON_ASCII_DIRNAME}/${TEST_CASE}.idf
WORKING_DIRECTORY "${CLI_TEST_DIR}"
)
endif()
endif()

if(UNIX AND NOT APPLE)

5 comments on commit fb265cd

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (2778 of 2778 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (Myoldmopar) - Win64-Windows-10-VisualStudio-16: OK (2756 of 2756 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1969 of 1969 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (790 of 790 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

develop (Myoldmopar) - x86_64-MacOS-10.17-clang-14.0.0: OK (2757 of 2757 tests passed, 0 test warnings)

Build Badge Test Badge

Please sign in to comment.