disgord icon indicating copy to clipboard operation
disgord copied to clipboard

feat: simpler integration tests

Open andersfylling opened this issue 2 years ago • 1 comments

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 avatar Dec 27 '21 18:12 andersfylling

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.

andersfylling avatar Mar 15 '22 18:03 andersfylling