Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add cmake configuration for the location of the test data #513

Closed
wants to merge 2 commits into from

Conversation

ianmccul
Copy link
Contributor

Added CYTNX_TEST_DATA_DIR variable for the location of the test data root directory

Update all tests that reference the test data directory to load data from CYTNX_TEST_DATA_DIR

tests/CMakeLists.txt: add CYTNX_TEST_DATA_DIR variable for the location of the test data root directory

Update all tests that reference the test data directory to load data from CYTNX_TEST_DATA_DIR
@ianmccul
Copy link
Contributor Author

Updated version passes all tests locally

@yingjerkao
Copy link
Collaborator

In the old uni10 days, the tests and related files are installed in a test subdirectory so that the CTEST can be run separately. This install location can be set in the main CmakeLIst.txt.

@ianmccul
Copy link
Contributor Author

I have no idea about the codecov test, it doesn't appear to be anything to do with the changes in the patch

tests/CMakeLists.txt Outdated Show resolved Hide resolved
@IvanaGyro
Copy link
Collaborator

In the old uni10 days, the tests and related files are installed in a test subdirectory so that the CTEST can be run separately. This install location can be set in the main CmakeLIst.txt.

What does CTEST can be run separately mean?

Confirmed works for the CPU tests and GPU tests
@IvanaGyro
Copy link
Collaborator

I merged this PR into the dev branch at #523.

@IvanaGyro IvanaGyro closed this Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants