Skip to content

Conversation

@moorec-aws
Copy link
Contributor

@moorec-aws moorec-aws commented May 14, 2025

What was the problem/requirement? (What/Why)

xdist and parallel test workers are not able to import our example modules AdaptorExample and CommandAdaptorExample on Windows, seems the subprocesses cannot access PYTHONPATH in the latest version of pytest.

What was the solution? (How)

Set PYTEST_XDIST_WORKER_PYTHONPATH to allow workers to access the modules

What is the impact of this change?

Support latest version of pytest

How was this change tested?

hatch run lint
hatch run test

Was this change documented?

N/A

Is this a breaking change?

No

Does this change impact security?

No

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@moorec-aws moorec-aws force-pushed the moorec/update-pytest branch 3 times, most recently from d4ab847 to 947f37a Compare May 14, 2025 15:49
@sonarqubecloud
Copy link

@moorec-aws moorec-aws force-pushed the moorec/update-pytest branch 2 times, most recently from d63ea65 to 93623a4 Compare May 14, 2025 16:23
@moorec-aws moorec-aws force-pushed the moorec/update-pytest branch from 93623a4 to c650df1 Compare May 14, 2025 17:06
@moorec-aws moorec-aws marked this pull request as ready for review May 14, 2025 17:11
@moorec-aws moorec-aws requested a review from a team as a code owner May 14, 2025 17:11
@moorec-aws moorec-aws changed the title chore(deps): update pytest to 8.* chore(deps): update pytest to 8.3.* May 14, 2025
@moorec-aws moorec-aws merged commit b000328 into OpenJobDescription:mainline May 14, 2025
17 checks passed
@moorec-aws moorec-aws deleted the moorec/update-pytest branch May 14, 2025 18:11
jblagden pushed a commit to jblagden/openjd-adaptor-runtime-for-python that referenced this pull request Aug 21, 2025
Signed-off-by: Charles Moore <[email protected]>
Signed-off-by: Justin Blagden <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants