Skip to content
This repository has been archived by the owner on Jul 11, 2024. It is now read-only.

feat: simpler integration tests #454

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

Conversation

andersfylling
Copy link
Owner

@andersfylling andersfylling commented Dec 27, 2021

It's currently a lot of work getting integration tests to work locally. While there is a need to verify error handling, I believe it's currently more productive to remove all the setup requirements for integration tests to run. Instead the bot will create a guild such that there is no need to verify permissions or set anything up.

Meaning the only requirement should be one discord token for testing, and then all integration tests should work. After this, integration tests can run per PR.

Might have to talk to the discord staff to reduce some rate limits if I get a lot of PRs.

New flow:

  • delete all previous guilds for test bot (this sounds fairly dangerous)
  • create new guild
  • run integration tests
  • delete guild

@andersfylling
Copy link
Owner Author

Current blocker is adding members. We need at least one member in any server created. This can be a bot, just need to figure out how to add one automatically on creation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant