TestPathManager test_instantiate_without_file fails with existing `path_config.env about simpa HOT 5 CLOSED

Ok, I see. With the new option to set the SIMPA_SAVE_PATH in environment variables it is unclear which option is prioritized in the order: https://github.com/IMSY-DKFZ/simpa/tree/develop?tab=readme-ov-file#path-management

I would suggest the following order:

1. The optional path you give the PathManager
2. The environment variables
3. Your home directory
4. The current working directory
5. The SIMPA home directory path

from simpa.

Maybe @leoyala knows more? Is there a solution with @patch.dict decorators?

from simpa.

Theoretically, the decorator in that test should set the environment variable for the mcx path to "test_mcx_path". This should be the path that MCX uses for the path manager. I do see that there might be some conflicts if botht eh path in the home directory is set and when not. @TomTomRixRix are you working on this?

from simpa.

That order of priority seems good to me

from simpa.

Fixed by #347

from simpa.