move get_version to child process

[javierggt]

move get_version to child process to prevent environment from being changed as side effect

Description

This issue arised in the context of sot/testr/pull/21.

At the very top of get_version, the module whose version we want is imported. Some modules change the environment, and that can cause undesired effects. One way around it is to get the version in a separate process.

This is not that common. One instance I have seen was importing dea_check, which causes kadi to be imported and adds DJANGO_SETTINGS_MODULE to the environment. The trouble is during testing, where the main process has this import, and then in child processes we get the error:

E           AttributeError: module 'kadi.events' has no attribute 'dwells'

Testing

• Passes unit tests on MacOS, linux, Windows (at least one required)
• Functional testing

Fixes #

[taldcroft]

I assume this is overtaken by #12?

[javierggt]

Yes. overtaken by #12