Make sure that configuration files are closed when loading them

[saulbein]

Change Description

When configuration is loaded, the current version of the code only does an open call which leaves a dangling file reference that is never closed. While this isn't a major issue, we found that in the project where we use presidio, this causes pytest to issue warnings about unclosed resources (that for us is set up to be converted into test errors).

I'm not sure why, but this happens only with version 2.2.354 but not 2.2.353.

Issue reference

Didn't create an issue as just contributing a fix was faster for me.

Checklist

• I have reviewed the contribution guidelines
• I have signed the CLA (if required)
• My code includes unit tests
• All unit tests and lint checks pass locally

[omri374]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).