Skip to content

test: Fix pytest error#2712

Merged
szokeasaurusrex merged 2 commits intomasterfrom
szokeasaurusrex/fix-pytest
Feb 6, 2024
Merged

test: Fix pytest error#2712
szokeasaurusrex merged 2 commits intomasterfrom
szokeasaurusrex/fix-pytest

Conversation

@szokeasaurusrex
Copy link
Copy Markdown
Member

@szokeasaurusrex szokeasaurusrex commented Feb 6, 2024

The ability to pass None to pytest.capture_warnings was removed in pytest version 8.0.0. To validate that this fix, one can run any of the test cases with pytest==8.0.0. Without this change, the test immediately fails with an error; with the change, the test suite runs as expected.

Fixes GH-2693

@szokeasaurusrex szokeasaurusrex enabled auto-merge (squash) February 6, 2024 16:10
Copy link
Copy Markdown
Contributor

@sentrivana sentrivana left a comment

Choose a reason for hiding this comment

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

@szokeasaurusrex szokeasaurusrex merged commit d97e7d7 into master Feb 6, 2024
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/fix-pytest branch February 6, 2024 16:27
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.

1.40.0: pytest fails with deprecation errors

2 participants