Skip to content

Migrate rdp test to pytest#477

Merged
thinkst-pieter merged 2 commits into
masterfrom
migrate_to_pytest_rdp
May 15, 2026
Merged

Migrate rdp test to pytest#477
thinkst-pieter merged 2 commits into
masterfrom
migrate_to_pytest_rdp

Conversation

@thinkst-pieter
Copy link
Copy Markdown
Contributor

Proposed changes

Migrate the testing to pytest from the python unit test into a dedicated test file for each opencanary module in preparation of adding more testing.

This PR handles the rdp tests

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update

Checklist

  • Lint and unit tests pass locally with my changes (if applicable)
  • I have run pre-commit (pre-commit in the repo)
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Linked to the relevant github issue or github discussion

Comment thread opencanary/test/test_rdp.py Outdated
@thinkst-pieter thinkst-pieter merged commit ed10fed into master May 15, 2026
21 checks passed
@thinkst-pieter thinkst-pieter deleted the migrate_to_pytest_rdp branch May 15, 2026 12:54
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