Correctly set applications environment in spawned nodes and start only started applications

whitfin / local-cluster

Easy local cluster creation for Elixir to aid in unit testing

MIT License

223 stars 30 forks source link

Closed xadhoom closed 9 months ago

xadhoom commented 3 years ago

This patch fixes two minor issues:

when setting env, the application must be loaded (and not started) before calling Application.put_env, otherwise if the app is not loaded when env is set, the defaults from the app will overwrite the ones set by local_cluster
start only already started applications: sometimes there're apps that are loaded and not started, like the ones that have runtime: false in mix.exs. Starting them may lead to errors

whitfin commented 9 months ago

This was closed accidentally when changing the main branch, feel free to reopen against main (I can't seem to).