radiasoft / sirepo

Sirepo is a framework for scientific cloud computing. Try it out!
https://sirepo.com
Apache License 2.0
64 stars 31 forks source link

animation_test should set env before start #5088

Open moellep opened 1 year ago

moellep commented 1 year ago

sirepo/tests/animation_test.py only works with a certain environment. For example if SIREPO_MPI_CORES is set to 5, the test_opal case will fail. If SIREPO_AUTH_METHODS='email' is set, then all cases fail. if SIREPO_FEATURE_CONFIG_SIM_TYPES='elegant' or similar, the other cases will fail.

If possible, the test should unset all those variables prior to running. Most likely test/report_test.py has similar issues.

robnagler commented 1 year ago

conftest.py should ensure a clean environment for all those variables.