Change test name from test to precice-aste-test

precice / aste

Artificial Solver Testing Environment

GNU General Public License v3.0

10 stars 11 forks source link

Change test name from test to precice-aste-test #175

Closed kursatyurt closed 3 months ago

kursatyurt commented 1 year ago

Main changes of this PR

The test executable is called testing is clashes with Cmake testing directory. test is located for Cmake itself. This PR changes the test executable to precice-aste-test

Author's checklist

[ ] I used the pre-commit hook and used pre-commit run --all to apply all available hooks.
[ ] I added a test to cover the proposed changes in our test suite.
[ ] I updated the documentation in docs/README.md.
[ ] I updated potential breaking changes in the tutorial precice/tutorials/aste-turbine.

davidscn commented 1 year ago

The test executable is called testing is clashes with Cmake testing directory.

@kursatyurt could you elaborate this more in detail? Naming an executable and directory the same is usually no problem and works on my system.

kursatyurt commented 1 year ago

The test executable is called testing is clashes with Cmake testing directory.

@kursatyurt could you elaborate this more in detail? Naming an executable and directory the same is usually no problem and works on my system.

It throws an error in case insensitive file system. Cannot compile the test.

More specifically

ld: can't write output file to 'testing' because that path is a directory
clang: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.