chore: Upgrade to pytest-docker v1

[WillDaSilva]

Closes #6496

[netlify]

✅ Deploy Preview for meltano ready!

Name	Link
🔨 Latest commit	d25a65e
🔍 Latest deploy log	https://app.netlify.com/sites/meltano/deploys/62e96b82c2cc20000866a872
😎 Deploy Preview	https://deploy-preview-6547--meltano.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[WillDaSilva]

I updated the Poetry lock file using Poetry 1.1.13, which had a bug that would set all updated entries under [metadata.files] to []. My hope is that updating the lock file with Poetry 1.1.14, then running the tests again should fix the issue in CI.

[edgarrmondragon]

I updated the Poetry lock file using Poetry 1.1.13, which had a bug that would set all updated entries under [metadata.files] to []. My hope is that updating the lock file with Poetry 1.1.14, then running the tests again should fix the issue in CI.

@WillDaSilva this might help python-poetry/poetry#5988 (comment)

[WillDaSilva]

this might help python-poetry/poetry#5988 (comment)

@edgarrmondragon Thanks for the link. I had already cleared my Poetry cache before generating the lock file for 6ade6a7, yet unfortunately the problem remains.

I've been able to reproduce locally. The same failures occur on main after updating the lock file, so it seems like we will have to put a max version restriction on one of the non-dev dependencies that poetry update touched. These include:

• alembic
• astroid
• atomicwrites
• blinker
• cryptography
• distlib
• flask
• flask-login
• identify
• importlib-resources
• markdown
• python-gitlab
• urllib3
• yarl
• zipp

Or we can just get to the bottom of this, and fix whatever bug is causing this if it within our own code.

[WillDaSilva]

Keeping flask-login at 0.6.1 fixes the failing tests.

When the tests are run in isolation, they do not error, regardless of whether I had flask-login at 0.6.1 or 0.6.2, which suggests that resolving #6300 would address the underlying issue here.

Relevant changelog: https://github.com/maxcountryman/flask-login/blob/main/CHANGES.md#version-062

[codecov]

Codecov Report

Merging #6547 (d25a65e) into main (c9c1737) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #6547   +/-   ##
=======================================
  Coverage   96.52%   96.52%           
=======================================
  Files          95       95           
  Lines        8349     8349           
  Branches      398      398           
=======================================
  Hits         8059     8059           
  Misses        227      227           
  Partials       63       63           

Help us with your feedback. Take ten seconds to tell us how you rate us.

[edgarrmondragon]

@WillDaSilva can you enable the Windows x Python 3.10 tests on this PR to see if this fixes them?

[WillDaSilva]

@WillDaSilva can you enable the Windows x Python 3.10 tests on this PR to see if this fixes them?

I was planning to do that after this PR goes through, to avoid introducing errors to this one if something goes wrong.

[edgarrmondragon]

I was planning to do that after this PR goes through, to avoid introducing errors to this one if something goes wrong.

@WillDaSilva sounds good 👍