Bug: JSON unsupported

[Elijas]

To keep it DRY, I'll put a reference to the original issue:

MobileDynasty/pytest-env#11

There's is also a PR with a fix:

MobileDynasty/pytest-env#12

[gaborbernat]

A pull request is welcomed to address it. 👍

[Elijas]

A pull request is welcomed to address it. 👍

Thanks for a quick reply! Happy to see the project support revived.

The PR is available here:

MobileDynasty/pytest-env#12

Unfortunately I'm away from my computer (responding over phone) and will be for the foreseeable future, so it's really cumbersome to recreate the PR and all.

Could you maybe git cherry-pick the commit or just copy-paste the changes?

[gaborbernat]

It's alright. We'll wait until you're back. Also need to add tests now.

[Elijas]

It's alright. We'll wait until you're back. Also need to add tests now.

Completed it: #5

I did not see any tests written for the package so far, so covering the whole package with unit tests is out of scope for this PR. Once the package has tests, adding tests for this package will be super straighforward.

[gaborbernat]

I did not see any tests written for the package so far, so covering the whole package with unit tests is out of scope for this PR.

Strongly disagree.

[Elijas]

Thanks for providing an expected framework for unit testing with #6 ! Now adding tests for the package is really straightforward. I did add the tests, and also tried to improve a little more on the framework itself, as the current framework introduced a lot of boilerplate code and files. Please take a look: #9

[Elijas]

Fixed with #9