Skip to content

fix: Report fails to render with pytest-xdist (2)#592

Closed
BeyondEvil wants to merge 1 commit intopytest-dev:next-genfrom
BeyondEvil:beyondevil/fix-xdist-2
Closed

fix: Report fails to render with pytest-xdist (2)#592
BeyondEvil wants to merge 1 commit intopytest-dev:next-genfrom
BeyondEvil:beyondevil/fix-xdist-2

Conversation

@BeyondEvil
Copy link
Copy Markdown
Contributor

@BeyondEvil BeyondEvil commented Mar 10, 2023

See #591 for a detailed explanation.

This is proposed solution 2.

@BeyondEvil BeyondEvil changed the title fix: Report fails to render with pytest-xdist fix: Report fails to render with pytest-xdist (2) Mar 10, 2023
)

# The WorkerController (from xdist) object is unserializable, so we remove it
data.pop("node", None)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we ought to use pytest-dev/pytest-xdist#887 later on

@BeyondEvil
Copy link
Copy Markdown
Contributor Author

Fixed by: #598

@BeyondEvil BeyondEvil closed this Mar 19, 2023
@BeyondEvil BeyondEvil deleted the beyondevil/fix-xdist-2 branch March 19, 2023 17:58
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.

2 participants