Skip to content

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

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

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

Conversation

@BeyondEvil

@BeyondEvil BeyondEvil commented Mar 10, 2023 •

Copy link
Copy Markdown
Contributor

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