Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: create initial test setup #1130

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

test: create initial test setup #1130

wants to merge 8 commits into from

Conversation

ooliver1
Copy link
Member

@ooliver1 ooliver1 commented Sep 30, 2023

Summary

Note
This is branched from #1126 as to not have to redo dependencies when that is merged. View the tests-only diff here

I am yet to add this to CONTRIBUTING.md, it probably won't be done in this PR as there are too few tests to enforce test styles right now.

This PR adds the initial setup for unit testing. I have added test_abc.py which does shallow testing of abc.GuildChannel's _edit and _move methods as an example of sorts for adding more tests.

These have already fixed a couple of issues - forum tags using strings in outbound payloads and audioop-dependent code needing to be changed as it will be removed in a future Python version.

@ooliver1 ooliver1 added p: medium Priority: medium - should be worked on in the near future s: awaiting review Status: the issue or PR is awaiting reviews 3.0 The issue/PR should go for the 3.0 release labels Sep 30, 2023
@ooliver1 ooliver1 marked this pull request as draft October 3, 2023 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.0 The issue/PR should go for the 3.0 release p: medium Priority: medium - should be worked on in the near future s: awaiting review Status: the issue or PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant