Skip to content

MAINT Remove redundant pytest asyncio markers - #2453

Merged
Roman Lutz (romanlutz) merged 1 commit into
microsoft:mainfrom
romanlutz:romanlutz-remove-redundant-asyncio-markers
Aug 21, 2026
Merged

MAINT Remove redundant pytest asyncio markers#2453
Roman Lutz (romanlutz) merged 1 commit into
microsoft:mainfrom
romanlutz:romanlutz-remove-redundant-asyncio-markers

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

PyRIT uses asyncio_mode = "auto", so explicit @pytest.mark.asyncio decorators on async tests are redundant and prohibited by the test instructions. Remove all 33 remaining parameterless function-level decorators across the affected unit and integration tests while preserving skip, fixture, and parametrization decorators.

No parameterized, module-level, class-level, or loop-scope asyncio markers required exceptions.

Tests and Documentation

  • Affected test collection passed.
  • Combined affected test selection: 226 passed, 38 skipped.
  • Ruff check and format checks passed for all changed files.
  • File-scoped pre-commit hooks and git diff --check passed.
  • Documentation and JupyText: N/A.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jbolor21 jbolor21 self-assigned this Aug 21, 2026
@romanlutz
Roman Lutz (romanlutz) added this pull request to the merge queue Aug 21, 2026
Merged via the queue into microsoft:main with commit ca46fb3 Aug 21, 2026
47 checks passed
@romanlutz
Roman Lutz (romanlutz) deleted the romanlutz-remove-redundant-asyncio-markers branch August 21, 2026 19:45
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