Adds pytest-xdist as dependency

[sklarsa]

I installed pytest-terraform using a fresh poetry environment, ran a simple testing script, and got this output:

file /Users/steven/Library/Caches/pypoetry/virtualenvs/terraform-SLjsQr8Z-py3.7/lib/python3.7/site-packages/pytest_terraform/tf.py, line 373: source code not available
E       fixture 'worker_id' not found
>       available fixtures: cache, capfd, capfdbinary, caplog, capsys, capsysbinary, doctest_namespace, monkeypatch, pytestconfig, record_property, record_testsuite_property, record_xml_attribute, recwarn, role, tmp_path, tmp_path_factory, tmpdir, tmpdir_factory
>       use 'pytest --fixtures [testpath]' for help on them.

When I remove the worker_id arg, the tests run

[codecov]

Codecov Report

Merging #29 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   92.64%   92.64%           
=======================================
  Files           8        8           
  Lines         462      462           
=======================================
  Hits          428      428           
  Misses         34       34           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 760efee...fe0b09f. Read the comment docs.

[kapilt]

do you have pytest-xdist installed and if so what version?

[kapilt]

we probably need to add pytest-xdist to our requires its currently only setup as a dev-dependency

[sklarsa]

no did not have pytest-xdist installed

[llavin]

I had the same experience. Installing python-xdist solved it. Everything was cool after that. Thanks.

[sklarsa]

I just updated this PR to revert the previous change and move pytest-xdist from dev to prod dependencies

[marcoceppi]

Thanks for moving this up. Just one request before releasing.

[marcoceppi]

Can we stick with ^1.31.0? That will do >= 1.31.0 but not 2.0 so we don't accidentally break things if upstream changes faster than we're ready.

[kapilt]

we work on xdist 2.0, and thats already out so we don't want to pin to old versions, we want to catch breakage proactively, we should setup a nightly github action to catch any issues against these pins. the lock file already captures repeatable install per se albeit thats source install.

[sklarsa]

I'm not sure what you want me to do here, keep the >= or switch to ^?

[marcoceppi]

Fair enough. I'll take up an item to expand our testing on latest deps for our installs