search

ACCESS-NRI / model-config-tests

Tests for checking model configurations
Apache License 2.0
0 stars 1 forks source link

Remove general config test for exe paths #29

Open jo-basevi opened 2 weeks ago

jo-basevi commented 2 weeks ago

In the eventual next release of payu, model executables can be set using modules. The current QA config tests check for absolute paths - and also that the exe paths are in the manifest:

https://github.com/ACCESS-NRI/model-config-tests/blob/154705ee830c9a046dee5eb6903fb26126fc91c1/src/model_config_tests/test_config.py#L78-L109

These tests will either need to be removed or updated

jo-basevi commented 2 weeks ago

Note relevant issue here for updating these tests: https://github.com/ACCESS-NRI/model-config-tests/issues/18

aidanheerdegen commented 2 weeks ago

We're specify the versions of tests we use right? So we could make this change without need for backwards compatibility?

jo-basevi commented 2 weeks ago

Yes, we specify the version in the <model>-configs repository. So we should only now test that exe paths are only set using modules?

aidanheerdegen commented 2 weeks ago

So we should only now test that exe paths are only set using modules?

Yes.