super-cooper / memebot

A custom Discord bot For The Boys
GNU General Public License v3.0
8 stars 0 forks source link

Add automated testing infrastructure #31

Open super-cooper opened 4 years ago

super-cooper commented 4 years ago

Is your feature request related to a problem? Please describe. We keep running into simple bugs because we have no unit tests or anything of the sort. We should have a series of tests that are required to pass before a change is merged.

Describe the solution you'd like For now, something simple like the python unittest framework should suffice. If our needs become more complex, we can use something more robust later.

There is also a library specifically made for testing Discord bots that @nhawke has mentioned to me before. We should absolutely incorporate this into our unit tests.

Describe alternatives you've considered The only other Python unit testing framework I know of is PyTest, but I've never used that before.

Additional context We should probably use continuous integration to prevent things from being merged without testing. This should be a separate, future ticket that depends on this one.

nhawke commented 4 years ago

Using the discord bot testing library probably isn't necessary. This will be easier with #19 done.

super-cooper commented 4 years ago

We should do it anyway so we can test what output looks like

super-cooper commented 3 years ago

https://github.com/JakeCover/distest

super-cooper commented 3 years ago

http://mypy-lang.org/

super-cooper commented 2 years ago

Re-assigning this to myself since it's blocking a bunch of stuff and I feel like working on the bot